WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
[patch]
WIP patch for ews testing
0001-setautofilled.patch (text/plain), 24.08 KB, created by
Jesus Sanchez-Palencia
on 2012-06-04 14:33:08 PDT
(
hide
)
Description:
WIP patch for ews testing
Filename:
MIME Type:
Creator:
Jesus Sanchez-Palencia
Created:
2012-06-04 14:33:08 PDT
Size:
24.08 KB
patch
obsolete
>From 10092611b4591b2e1232589abf6bd1e00e6fa0a6 Mon Sep 17 00:00:00 2001 >From: Jesus Sanchez-Palencia <jesus.palencia@openbossa.org> >Date: Mon, 4 Jun 2012 11:50:32 -0300 >Subject: [PATCH] setautofilled > >--- > LayoutTests/fast/forms/input-autofilled.html | 8 ++++---- > LayoutTests/fast/forms/reset-autofilled.html | 4 ++-- > Source/WebCore/testing/Internals.cpp | 10 ++++++++++ > Source/WebCore/testing/Internals.h | 1 + > Source/WebCore/testing/Internals.idl | 2 ++ > Source/WebKit/chromium/public/WebInputElement.h | 4 ++++ > .../WebCoreSupport/DumpRenderTreeSupportEfl.cpp | 13 ------------ > .../efl/WebCoreSupport/DumpRenderTreeSupportEfl.h | 1 - > .../WebCoreSupport/DumpRenderTreeSupportGtk.cpp | 13 ------------ > .../gtk/WebCoreSupport/DumpRenderTreeSupportGtk.h | 1 - > Source/WebKit/mac/DOM/WebDOMOperationsPrivate.h | 2 ++ > .../qt/WebCoreSupport/DumpRenderTreeSupportQt.cpp | 12 ----------- > .../qt/WebCoreSupport/DumpRenderTreeSupportQt.h | 1 - > Source/WebKit/win/DOMHTMLClasses.h | 2 ++ > Tools/DumpRenderTree/LayoutTestController.cpp | 12 ----------- > Tools/DumpRenderTree/LayoutTestController.h | 1 - > .../blackberry/LayoutTestControllerBlackBerry.cpp | 13 ------------ > .../chromium/LayoutTestController.cpp | 18 ----------------- > .../DumpRenderTree/chromium/LayoutTestController.h | 4 ---- > .../DumpRenderTree/efl/LayoutTestControllerEfl.cpp | 5 ----- > .../DumpRenderTree/gtk/LayoutTestControllerGtk.cpp | 5 ----- > .../DumpRenderTree/mac/LayoutTestControllerMac.mm | 9 --------- > Tools/DumpRenderTree/qt/LayoutTestControllerQt.cpp | 5 ----- > Tools/DumpRenderTree/qt/LayoutTestControllerQt.h | 1 - > .../DumpRenderTree/win/LayoutTestControllerWin.cpp | 21 -------------------- > Tools/DumpRenderTree/wx/LayoutTestControllerWx.cpp | 5 ----- > 26 files changed, 27 insertions(+), 146 deletions(-) > >diff --git a/LayoutTests/fast/forms/input-autofilled.html b/LayoutTests/fast/forms/input-autofilled.html >index 61ae360..3b2c58e 100644 >--- a/LayoutTests/fast/forms/input-autofilled.html >+++ b/LayoutTests/fast/forms/input-autofilled.html >@@ -12,8 +12,8 @@ > var originalForeground = computedStyle.color; > var originalBackground = computedStyle.backgroundColor; > >- if (window.layoutTestController) { >- layoutTestController.setAutofilled(tf, true); >+ if (window.internals) { >+ internals.setAutofilled(tf, true); > } > > // Both the foreground and background colors should change. >@@ -29,8 +29,8 @@ > return; > } > >- if (window.layoutTestController) { >- layoutTestController.setAutofilled(tf, false); >+ if (window.internals) { >+ internals.setAutofilled(tf, false); > } > > // Colors should be restored. >diff --git a/LayoutTests/fast/forms/reset-autofilled.html b/LayoutTests/fast/forms/reset-autofilled.html >index d3fbe7f..49f8354 100644 >--- a/LayoutTests/fast/forms/reset-autofilled.html >+++ b/LayoutTests/fast/forms/reset-autofilled.html >@@ -12,8 +12,8 @@ > } > > var field = document.getElementById('field'); >- if (window.layoutTestController) { >- layoutTestController.setAutofilled(field, true); >+ if (window.internals) { >+ internals.setAutofilled(field, true); > } else { > testFailed('This test requires a LayoutTestController.'); > return; >diff --git a/Source/WebCore/testing/Internals.cpp b/Source/WebCore/testing/Internals.cpp >index f4ad3f6..6fdb25c 100644 >--- a/Source/WebCore/testing/Internals.cpp >+++ b/Source/WebCore/testing/Internals.cpp >@@ -1092,4 +1092,14 @@ void Internals::webkitDidExitFullScreenForElement(Document* document, Element* e > document->webkitDidExitFullScreenForElement(element); > } > #endif >+ >+void Internals::setAutofilled(Element* element, bool isAutofilled, ExceptionCode& ec) >+{ >+ if (!element && !element->toInputElement()) { >+ ec = INVALID_ACCESS_ERR; >+ return; >+ } >+ >+ element->toInputElement()->setAutofilled(isAutofilled); >+} > } >diff --git a/Source/WebCore/testing/Internals.h b/Source/WebCore/testing/Internals.h >index 6fa4a45..3e989eb 100644 >--- a/Source/WebCore/testing/Internals.h >+++ b/Source/WebCore/testing/Internals.h >@@ -182,6 +182,7 @@ public: > void webkitWillExitFullScreenForElement(Document*, Element*); > void webkitDidExitFullScreenForElement(Document*, Element*); > #endif >+ void setAutofilled(Element*, bool isAutofilled, ExceptionCode&); > > private: > explicit Internals(Document*); >diff --git a/Source/WebCore/testing/Internals.idl b/Source/WebCore/testing/Internals.idl >index aa10858..5f07cf6 100644 >--- a/Source/WebCore/testing/Internals.idl >+++ b/Source/WebCore/testing/Internals.idl >@@ -157,6 +157,8 @@ module window { > void webkitWillExitFullScreenForElement(in Document document, in Element element); > void webkitDidExitFullScreenForElement(in Document document, in Element element); > #endif >+ >+ void setAutofilled(in Element element, in boolean isAutofilled) raises (DOMException); > }; > } > >diff --git a/Source/WebKit/chromium/public/WebInputElement.h b/Source/WebKit/chromium/public/WebInputElement.h >index fbc3500..ef2b668 100644 >--- a/Source/WebKit/chromium/public/WebInputElement.h >+++ b/Source/WebKit/chromium/public/WebInputElement.h >@@ -85,8 +85,12 @@ namespace WebKit { > WEBKIT_EXPORT WebString suggestedValue() const; > WEBKIT_EXPORT void setPlaceholder(const WebString&); > WEBKIT_EXPORT WebString placeholder() const; >+ > WEBKIT_EXPORT bool isAutofilled() const; >+ // FIXME: This can be removed if there are no Chromium-internal calls to it. >+ // See http://webkit.org/b/88239. > WEBKIT_EXPORT void setAutofilled(bool); >+ > WEBKIT_EXPORT void setSelectionRange(int, int); > WEBKIT_EXPORT int selectionStart() const; > WEBKIT_EXPORT int selectionEnd() const; >diff --git a/Source/WebKit/efl/WebCoreSupport/DumpRenderTreeSupportEfl.cpp b/Source/WebKit/efl/WebCoreSupport/DumpRenderTreeSupportEfl.cpp >index b45aef1..9456199 100644 >--- a/Source/WebKit/efl/WebCoreSupport/DumpRenderTreeSupportEfl.cpp >+++ b/Source/WebKit/efl/WebCoreSupport/DumpRenderTreeSupportEfl.cpp >@@ -335,19 +335,6 @@ void DumpRenderTreeSupportEfl::setValueForUser(JSContextRef context, JSValueRef > inputElement->setValueForUser(value); > } > >-void DumpRenderTreeSupportEfl::setAutofilled(JSContextRef context, JSValueRef nodeObject, bool autofilled) >-{ >- JSC::ExecState* exec = toJS(context); >- WebCore::Element* element = WebCore::toElement(toJS(exec, nodeObject)); >- if (!element) >- return; >- WebCore::HTMLInputElement* inputElement = element->toInputElement(); >- if (!inputElement) >- return; >- >- inputElement->setAutofilled(autofilled); >-} >- > void DumpRenderTreeSupportEfl::setDefersLoading(Evas_Object* ewkView, bool defers) > { > WebCore::Page* page = EWKPrivate::corePage(ewkView); >diff --git a/Source/WebKit/efl/WebCoreSupport/DumpRenderTreeSupportEfl.h b/Source/WebKit/efl/WebCoreSupport/DumpRenderTreeSupportEfl.h >index 2ae3431..bfd3e9d 100644 >--- a/Source/WebKit/efl/WebCoreSupport/DumpRenderTreeSupportEfl.h >+++ b/Source/WebKit/efl/WebCoreSupport/DumpRenderTreeSupportEfl.h >@@ -69,7 +69,6 @@ public: > static WebCore::IntRect selectionRectangle(const Evas_Object* ewkFrame); > static String suitableDRTFrameName(const Evas_Object* ewkFrame); > static void setValueForUser(JSContextRef, JSValueRef nodeObject, const String& value); >- static void setAutofilled(JSContextRef, JSValueRef nodeObject, bool autofilled); > static void setDefersLoading(Evas_Object* ewkView, bool defers); > static void setLoadsSiteIconsIgnoringImageLoadingSetting(Evas_Object* ewkView, bool loadsSiteIconsIgnoringImageLoadingPreferences); > >diff --git a/Source/WebKit/gtk/WebCoreSupport/DumpRenderTreeSupportGtk.cpp b/Source/WebKit/gtk/WebCoreSupport/DumpRenderTreeSupportGtk.cpp >index 3dd6b4d..6792c94 100644 >--- a/Source/WebKit/gtk/WebCoreSupport/DumpRenderTreeSupportGtk.cpp >+++ b/Source/WebKit/gtk/WebCoreSupport/DumpRenderTreeSupportGtk.cpp >@@ -719,19 +719,6 @@ CString DumpRenderTreeSupportGtk::accessibilityHelpText(AtkObject* axObject) > return coreObject->helpText().utf8(); > } > >-void DumpRenderTreeSupportGtk::setAutofilled(JSContextRef context, JSValueRef nodeObject, bool autofilled) >-{ >- JSC::ExecState* exec = toJS(context); >- Element* element = toElement(toJS(exec, nodeObject)); >- if (!element) >- return; >- HTMLInputElement* inputElement = element->toInputElement(); >- if (!inputElement) >- return; >- >- inputElement->setAutofilled(autofilled); >-} >- > void DumpRenderTreeSupportGtk::setValueForUser(JSContextRef context, JSValueRef nodeObject, JSStringRef value) > { > JSC::ExecState* exec = toJS(context); >diff --git a/Source/WebKit/gtk/WebCoreSupport/DumpRenderTreeSupportGtk.h b/Source/WebKit/gtk/WebCoreSupport/DumpRenderTreeSupportGtk.h >index 34f7bde..2f0ab28 100644 >--- a/Source/WebKit/gtk/WebCoreSupport/DumpRenderTreeSupportGtk.h >+++ b/Source/WebKit/gtk/WebCoreSupport/DumpRenderTreeSupportGtk.h >@@ -76,7 +76,6 @@ public: > static AtkObject* getFocusedAccessibleElement(WebKitWebFrame*); > static AtkObject* getRootAccessibleElement(WebKitWebFrame*); > static void layoutFrame(WebKitWebFrame*); >- static void setAutofilled(JSContextRef, JSValueRef, bool); > static void setValueForUser(JSContextRef, JSValueRef, JSStringRef); > static bool shouldClose(WebKitWebFrame*); > static bool elementDoesAutoCompleteForElementWithId(WebKitWebFrame*, JSStringRef); >diff --git a/Source/WebKit/mac/DOM/WebDOMOperationsPrivate.h b/Source/WebKit/mac/DOM/WebDOMOperationsPrivate.h >index d3bf68c..e4ef7d1 100644 >--- a/Source/WebKit/mac/DOM/WebDOMOperationsPrivate.h >+++ b/Source/WebKit/mac/DOM/WebDOMOperationsPrivate.h >@@ -39,6 +39,8 @@ > @end > > @interface DOMHTMLInputElement (WebDOMHTMLInputElementOperationsPrivate) >+// FIXME: This can be removed if there are no Apple-internal calls to it. >+// See http://webkit.org/b/88239. > - (void)_setAutofilled:(BOOL)autofilled; > - (void)_setValueForUser:(NSString *)value; > @end >diff --git a/Source/WebKit/qt/WebCoreSupport/DumpRenderTreeSupportQt.cpp b/Source/WebKit/qt/WebCoreSupport/DumpRenderTreeSupportQt.cpp >index e70bf00..bb3ba86 100644 >--- a/Source/WebKit/qt/WebCoreSupport/DumpRenderTreeSupportQt.cpp >+++ b/Source/WebKit/qt/WebCoreSupport/DumpRenderTreeSupportQt.cpp >@@ -289,18 +289,6 @@ bool DumpRenderTreeSupportQt::hasDocumentElement(QWebFrame* frame) > return QWebFramePrivate::core(frame)->document()->documentElement(); > } > >-void DumpRenderTreeSupportQt::setAutofilled(const QWebElement& element, bool isAutofilled) >-{ >- WebCore::Element* webElement = element.m_element; >- if (!webElement) >- return; >- HTMLInputElement* inputElement = webElement->toInputElement(); >- if (!inputElement) >- return; >- >- inputElement->setAutofilled(isAutofilled); >-} >- > void DumpRenderTreeSupportQt::setValueForUser(const QWebElement& element, const QString& value) > { > WebCore::Element* webElement = element.m_element; >diff --git a/Source/WebKit/qt/WebCoreSupport/DumpRenderTreeSupportQt.h b/Source/WebKit/qt/WebCoreSupport/DumpRenderTreeSupportQt.h >index 41800fa..dd63076 100644 >--- a/Source/WebKit/qt/WebCoreSupport/DumpRenderTreeSupportQt.h >+++ b/Source/WebKit/qt/WebCoreSupport/DumpRenderTreeSupportQt.h >@@ -124,7 +124,6 @@ public: > > static void garbageCollectorCollect(); > static void garbageCollectorCollectOnAlternateThread(bool waitUntilDone); >- static void setAutofilled(const QWebElement&, bool enabled); > static void setValueForUser(const QWebElement&, const QString& value); > static int javaScriptObjectsCount(); > static void clearScriptWorlds(); >diff --git a/Source/WebKit/win/DOMHTMLClasses.h b/Source/WebKit/win/DOMHTMLClasses.h >index 34c7951..96d920b 100644 >--- a/Source/WebKit/win/DOMHTMLClasses.h >+++ b/Source/WebKit/win/DOMHTMLClasses.h >@@ -2015,6 +2015,8 @@ public: > /* [out] */ int *start, > /* [out] */ int *end); > >+ // FIXME: This can be removed if there are no Apple-internal calls to it. >+ // See http://webkit.org/b/88239. > virtual HRESULT STDMETHODCALLTYPE setAutofilled( > /* [in] */ BOOL filled); > >diff --git a/Tools/DumpRenderTree/LayoutTestController.cpp b/Tools/DumpRenderTree/LayoutTestController.cpp >index 2fa80cf..9d148e0 100644 >--- a/Tools/DumpRenderTree/LayoutTestController.cpp >+++ b/Tools/DumpRenderTree/LayoutTestController.cpp >@@ -1152,17 +1152,6 @@ static JSValueRef setAuthorAndUserStylesEnabledCallback(JSContextRef context, JS > return JSValueMakeUndefined(context); > } > >-static JSValueRef setAutofilledCallback(JSContextRef context, JSObjectRef function, JSObjectRef thisObject, size_t argumentCount, const JSValueRef arguments[], JSValueRef* exception) >-{ >- if (argumentCount != 2 || !arguments[0]) >- return JSValueMakeUndefined(context); >- >- LayoutTestController* controller = static_cast<LayoutTestController*>(JSObjectGetPrivate(thisObject)); >- controller->setAutofilled(context, arguments[0], JSValueToBoolean(context, arguments[1])); >- >- return JSValueMakeUndefined(context); >-} >- > static JSValueRef setCacheModelCallback(JSContextRef context, JSObjectRef function, JSObjectRef thisObject, size_t argumentCount, const JSValueRef arguments[], JSValueRef* exception) > { > // Has Mac implementation. >@@ -2375,7 +2364,6 @@ JSStaticFunction* LayoutTestController::staticFunctions() > { "setAuthenticationPassword", setAuthenticationPasswordCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete }, > { "setAuthenticationUsername", setAuthenticationUsernameCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete }, > { "setAuthorAndUserStylesEnabled", setAuthorAndUserStylesEnabledCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete }, >- { "setAutofilled", setAutofilledCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete }, > { "setCacheModel", setCacheModelCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete }, > { "setCallCloseOnWebViews", setCallCloseOnWebViewsCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete }, > { "setCanOpenWindows", setCanOpenWindowsCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete }, >diff --git a/Tools/DumpRenderTree/LayoutTestController.h b/Tools/DumpRenderTree/LayoutTestController.h >index 640d1c1..92d5a03 100644 >--- a/Tools/DumpRenderTree/LayoutTestController.h >+++ b/Tools/DumpRenderTree/LayoutTestController.h >@@ -94,7 +94,6 @@ public: > void setAppCacheMaximumSize(unsigned long long quota); > void setApplicationCacheOriginQuota(unsigned long long quota); > void setAuthorAndUserStylesEnabled(bool); >- void setAutofilled(JSContextRef, JSValueRef nodeObject, bool autofilled); > void setCacheModel(int); > void setCustomPolicyDelegate(bool setDelegate, bool permissive); > void setDatabaseQuota(unsigned long long quota); >diff --git a/Tools/DumpRenderTree/blackberry/LayoutTestControllerBlackBerry.cpp b/Tools/DumpRenderTree/blackberry/LayoutTestControllerBlackBerry.cpp >index 4f043d9..643263f 100644 >--- a/Tools/DumpRenderTree/blackberry/LayoutTestControllerBlackBerry.cpp >+++ b/Tools/DumpRenderTree/blackberry/LayoutTestControllerBlackBerry.cpp >@@ -737,19 +737,6 @@ void LayoutTestController::setAsynchronousSpellCheckingEnabled(bool) > notImplemented(); > } > >-void LayoutTestController::setAutofilled(JSContextRef context, JSValueRef nodeObject, bool autofilled) >-{ >- JSC::ExecState* exec = toJS(context); >- WebCore::Element* element = toElement(toJS(exec, nodeObject)); >- if (!element) >- return; >- WebCore::HTMLInputElement* inputElement = element->toInputElement(); >- if (!inputElement) >- return; >- >- inputElement->setAutofilled(autofilled); >-} >- > int LayoutTestController::numberOfPendingGeolocationPermissionRequests() > { > return DumpRenderTreeSupport::numberOfPendingGeolocationPermissionRequests(BlackBerry::WebKit::DumpRenderTree::currentInstance()->page()); >diff --git a/Tools/DumpRenderTree/chromium/LayoutTestController.cpp b/Tools/DumpRenderTree/chromium/LayoutTestController.cpp >index c739143..51f86fb 100644 >--- a/Tools/DumpRenderTree/chromium/LayoutTestController.cpp >+++ b/Tools/DumpRenderTree/chromium/LayoutTestController.cpp >@@ -185,7 +185,6 @@ LayoutTestController::LayoutTestController(TestShell* shell) > bindMethod("setAllowUniversalAccessFromFileURLs", &LayoutTestController::setAllowUniversalAccessFromFileURLs); > bindMethod("setAlwaysAcceptCookies", &LayoutTestController::setAlwaysAcceptCookies); > bindMethod("setAuthorAndUserStylesEnabled", &LayoutTestController::setAuthorAndUserStylesEnabled); >- bindMethod("setAutofilled", &LayoutTestController::setAutofilled); > bindMethod("setCanOpenWindows", &LayoutTestController::setCanOpenWindows); > bindMethod("setCloseRemainingWindowsWhenComplete", &LayoutTestController::setCloseRemainingWindowsWhenComplete); > bindMethod("setCustomPolicyDelegate", &LayoutTestController::setCustomPolicyDelegate); >@@ -2020,23 +2019,6 @@ void LayoutTestController::setMinimumTimerInterval(const CppArgumentList& argume > m_shell->webView()->settings()->setMinimumTimerInterval(arguments[0].toDouble()); > } > >-void LayoutTestController::setAutofilled(const CppArgumentList& arguments, CppVariant* result) >-{ >- result->setNull(); >- if (arguments.size() != 2 || !arguments[1].isBool()) >- return; >- >- WebElement element; >- if (!WebBindings::getElement(arguments[0].value.objectValue, &element)) >- return; >- >- WebInputElement* input = toWebInputElement(&element); >- if (!input) >- return; >- >- input->setAutofilled(arguments[1].value.boolValue); >-} >- > void LayoutTestController::setValueForUser(const CppArgumentList& arguments, CppVariant* result) > { > result->setNull(); >diff --git a/Tools/DumpRenderTree/chromium/LayoutTestController.h b/Tools/DumpRenderTree/chromium/LayoutTestController.h >index 3797865..88ded4e 100644 >--- a/Tools/DumpRenderTree/chromium/LayoutTestController.h >+++ b/Tools/DumpRenderTree/chromium/LayoutTestController.h >@@ -386,10 +386,6 @@ public: > > void setMinimumTimerInterval(const CppArgumentList&, CppVariant*); > >- // Expects the first argument to be an input element and the second argument to be a boolean. >- // Forwards the setAutofilled() call to the element. >- void setAutofilled(const CppArgumentList&, CppVariant*); >- > // Expects the first argument to be an input element and the second argument to be a string value. > // Forwards the setValueForUser() call to the element. > void setValueForUser(const CppArgumentList&, CppVariant*); >diff --git a/Tools/DumpRenderTree/efl/LayoutTestControllerEfl.cpp b/Tools/DumpRenderTree/efl/LayoutTestControllerEfl.cpp >index 0e8a66e..c77de4a 100644 >--- a/Tools/DumpRenderTree/efl/LayoutTestControllerEfl.cpp >+++ b/Tools/DumpRenderTree/efl/LayoutTestControllerEfl.cpp >@@ -364,11 +364,6 @@ void LayoutTestController::setAuthorAndUserStylesEnabled(bool flag) > DumpRenderTreeSupportEfl::setAuthorAndUserStylesEnabled(browser->mainView(), flag); > } > >-void LayoutTestController::setAutofilled(JSContextRef context, JSValueRef nodeObject, bool autofilled) >-{ >- DumpRenderTreeSupportEfl::setAutofilled(context, nodeObject, autofilled); >-} >- > void LayoutTestController::disableImageLoading() > { > ewk_view_setting_auto_load_images_set(browser->mainView(), EINA_FALSE); >diff --git a/Tools/DumpRenderTree/gtk/LayoutTestControllerGtk.cpp b/Tools/DumpRenderTree/gtk/LayoutTestControllerGtk.cpp >index 275c2e0..2886ad1 100644 >--- a/Tools/DumpRenderTree/gtk/LayoutTestControllerGtk.cpp >+++ b/Tools/DumpRenderTree/gtk/LayoutTestControllerGtk.cpp >@@ -453,11 +453,6 @@ void LayoutTestController::setAuthorAndUserStylesEnabled(bool flag) > // FIXME: implement > } > >-void LayoutTestController::setAutofilled(JSContextRef context, JSValueRef nodeObject, bool isAutofilled) >-{ >- DumpRenderTreeSupportGtk::setAutofilled(context, nodeObject, isAutofilled); >-} >- > void LayoutTestController::disableImageLoading() > { > // FIXME: Implement for testing fix for https://bugs.webkit.org/show_bug.cgi?id=27896 >diff --git a/Tools/DumpRenderTree/mac/LayoutTestControllerMac.mm b/Tools/DumpRenderTree/mac/LayoutTestControllerMac.mm >index 72c0cea..56ea465 100644 >--- a/Tools/DumpRenderTree/mac/LayoutTestControllerMac.mm >+++ b/Tools/DumpRenderTree/mac/LayoutTestControllerMac.mm >@@ -457,15 +457,6 @@ void LayoutTestController::setAuthorAndUserStylesEnabled(bool flag) > [[[mainFrame webView] preferences] setAuthorAndUserStylesEnabled:flag]; > } > >-void LayoutTestController::setAutofilled(JSContextRef context, JSValueRef nodeObject, bool autofilled) >-{ >- DOMElement *element = [DOMElement _DOMElementFromJSContext:context value:nodeObject]; >- if (!element || ![element isKindOfClass:[DOMHTMLInputElement class]]) >- return; >- >- [(DOMHTMLInputElement *)element _setAutofilled:autofilled]; >-} >- > void LayoutTestController::setCustomPolicyDelegate(bool setDelegate, bool permissive) > { > if (setDelegate) { >diff --git a/Tools/DumpRenderTree/qt/LayoutTestControllerQt.cpp b/Tools/DumpRenderTree/qt/LayoutTestControllerQt.cpp >index 9bbf439..1abea11 100644 >--- a/Tools/DumpRenderTree/qt/LayoutTestControllerQt.cpp >+++ b/Tools/DumpRenderTree/qt/LayoutTestControllerQt.cpp >@@ -472,11 +472,6 @@ void LayoutTestController::setAppCacheMaximumSize(unsigned long long quota) > m_drt->webPage()->settings()->setOfflineWebApplicationCacheQuota(quota); > } > >-void LayoutTestController::setAutofilled(const QWebElement& element, bool isAutofilled) >-{ >- return DumpRenderTreeSupportQt::setAutofilled(element, isAutofilled); >-} >- > void LayoutTestController::setValueForUser(const QWebElement& element, const QString& value) > { > DumpRenderTreeSupportQt::setValueForUser(element, value); >diff --git a/Tools/DumpRenderTree/qt/LayoutTestControllerQt.h b/Tools/DumpRenderTree/qt/LayoutTestControllerQt.h >index ef68b36..8ec871d 100644 >--- a/Tools/DumpRenderTree/qt/LayoutTestControllerQt.h >+++ b/Tools/DumpRenderTree/qt/LayoutTestControllerQt.h >@@ -158,7 +158,6 @@ public slots: > void setAllowUniversalAccessFromFileURLs(bool enable); > void setAllowFileAccessFromFileURLs(bool enable); > void setAppCacheMaximumSize(unsigned long long quota); >- void setAutofilled(const QWebElement&, bool enable); > void setValueForUser(const QWebElement&, const QString& value); > void setFixedContentsSize(int width, int height); > void setPrivateBrowsingEnabled(bool enable); >diff --git a/Tools/DumpRenderTree/win/LayoutTestControllerWin.cpp b/Tools/DumpRenderTree/win/LayoutTestControllerWin.cpp >index 1239449..6f56819 100644 >--- a/Tools/DumpRenderTree/win/LayoutTestControllerWin.cpp >+++ b/Tools/DumpRenderTree/win/LayoutTestControllerWin.cpp >@@ -374,27 +374,6 @@ void LayoutTestController::setAuthorAndUserStylesEnabled(bool flag) > prefsPrivate->setAuthorAndUserStylesEnabled(flag); > } > >-void LayoutTestController::setAutofilled(JSContextRef context, JSValueRef nodeObject, bool autofilled) >-{ >- COMPtr<IWebView> webView; >- if (FAILED(frame->webView(&webView))) >- return; >- >- COMPtr<IWebViewPrivate> webViewPrivate(Query, webView); >- if (!webViewPrivate) >- return; >- >- COMPtr<IDOMElement> element; >- if (FAILED(webViewPrivate->elementFromJS(context, nodeObject, &element))) >- return; >- >- COMPtr<IFormsAutoFillTransition> autofillElement(Query, element); >- if (!autofillElement) >- return; >- >- autofillElement->setAutofilled(autofilled); >-} >- > void LayoutTestController::setCustomPolicyDelegate(bool setDelegate, bool permissive) > { > COMPtr<IWebView> webView; >diff --git a/Tools/DumpRenderTree/wx/LayoutTestControllerWx.cpp b/Tools/DumpRenderTree/wx/LayoutTestControllerWx.cpp >index 08d2832..3f9a72b 100644 >--- a/Tools/DumpRenderTree/wx/LayoutTestControllerWx.cpp >+++ b/Tools/DumpRenderTree/wx/LayoutTestControllerWx.cpp >@@ -203,11 +203,6 @@ void LayoutTestController::setAuthorAndUserStylesEnabled(bool flag) > // FIXME: implement > } > >-void LayoutTestController::setAutofilled(JSContextRef, JSValueRef element, bool isAutofilled) >-{ >- // FIXME: implement >-} >- > void LayoutTestController::setPopupBlockingEnabled(bool popupBlockingEnabled) > { > // FIXME: implement >-- >1.7.10.2 >
<b>You cannot view the attachment while viewing its details because your browser does not support IFRAMEs. <a href="attachment.cgi?id=145628">View the attachment on a separate page</a>.</b>
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Formatted Diff
|
Diff
Attachments on
bug 88239
:
145628
|
145790