Title: [153356] trunk/Source
Revision
153356
Author
[email protected]
Date
2013-07-25 16:57:40 -0700 (Thu, 25 Jul 2013)

Log Message

ChromeClient::focusedNodeChanged() should be focusedElementChanged().
<http://webkit.org/b/119110>

Reviewed by Anders Carlsson.

Source/WebCore:

Because only Elements can be focused.

* dom/Document.cpp:
(WebCore::Document::setFocusedElement):
* loader/EmptyClients.h:
(WebCore::EmptyChromeClient::focusedElementChanged):
* page/Chrome.cpp:
(WebCore::Chrome::focusedElementChanged):
* page/Chrome.h:
* page/ChromeClient.h:

Source/WebKit/blackberry:

* WebCoreSupport/ChromeClientBlackBerry.cpp:
(WebCore::ChromeClientBlackBerry::focusedElementChanged):
* WebCoreSupport/ChromeClientBlackBerry.h:

Source/WebKit/efl:

* WebCoreSupport/ChromeClientEfl.cpp:
(WebCore::ChromeClientEfl::focusedElementChanged):
* WebCoreSupport/ChromeClientEfl.h:

Source/WebKit/gtk:

* WebCoreSupport/ChromeClientGtk.cpp:
(WebKit::ChromeClient::focusedElementChanged):
* WebCoreSupport/ChromeClientGtk.h:

Source/WebKit/mac:

* WebCoreSupport/WebChromeClient.h:
* WebCoreSupport/WebChromeClient.mm:
(WebChromeClient::focusedElementChanged):

Source/WebKit/qt:

* WebCoreSupport/ChromeClientQt.cpp:
(WebCore::ChromeClientQt::focusedElementChanged):
* WebCoreSupport/ChromeClientQt.h:

Source/WebKit/win:

* WebCoreSupport/WebChromeClient.cpp:
(WebChromeClient::focusedElementChanged):
* WebCoreSupport/WebChromeClient.h:

Source/WebKit/wince:

* WebCoreSupport/ChromeClientWinCE.cpp:
(WebKit::ChromeClientWinCE::focusedElementChanged):
* WebCoreSupport/ChromeClientWinCE.h:

Source/WebKit2:

* WebProcess/WebCoreSupport/WebChromeClient.cpp:
(WebKit::WebChromeClient::focusedElementChanged):
* WebProcess/WebCoreSupport/WebChromeClient.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (153355 => 153356)


--- trunk/Source/WebCore/ChangeLog	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebCore/ChangeLog	2013-07-25 23:57:40 UTC (rev 153356)
@@ -1,3 +1,21 @@
+2013-07-25  Andreas Kling  <[email protected]>
+
+        ChromeClient::focusedNodeChanged() should be focusedElementChanged().
+        <http://webkit.org/b/119110>
+
+        Reviewed by Anders Carlsson.
+
+        Because only Elements can be focused.
+
+        * dom/Document.cpp:
+        (WebCore::Document::setFocusedElement):
+        * loader/EmptyClients.h:
+        (WebCore::EmptyChromeClient::focusedElementChanged):
+        * page/Chrome.cpp:
+        (WebCore::Chrome::focusedElementChanged):
+        * page/Chrome.h:
+        * page/ChromeClient.h:
+
 2013-07-25  Kwang Yul Seo  <[email protected]>
 
         [WK2][Soup] Add private browsing support

Modified: trunk/Source/WebCore/dom/Document.cpp (153355 => 153356)


--- trunk/Source/WebCore/dom/Document.cpp	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebCore/dom/Document.cpp	2013-07-25 23:57:40 UTC (rev 153356)
@@ -3407,7 +3407,7 @@
     }
 
     if (!focusChangeBlocked)
-        page()->chrome().focusedNodeChanged(m_focusedElement.get());
+        page()->chrome().focusedElementChanged(m_focusedElement.get());
 
 SetFocusedNodeDone:
     updateStyleIfNeeded();

Modified: trunk/Source/WebCore/loader/EmptyClients.h (153355 => 153356)


--- trunk/Source/WebCore/loader/EmptyClients.h	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebCore/loader/EmptyClients.h	2013-07-25 23:57:40 UTC (rev 153356)
@@ -77,7 +77,7 @@
     virtual bool canTakeFocus(FocusDirection) { return false; }
     virtual void takeFocus(FocusDirection) { }
 
-    virtual void focusedNodeChanged(Node*) { }
+    virtual void focusedElementChanged(Element*) { }
     virtual void focusedFrameChanged(Frame*) { }
 
     virtual Page* createWindow(Frame*, const FrameLoadRequest&, const WindowFeatures&, const NavigationAction&) { return 0; }

Modified: trunk/Source/WebCore/page/Chrome.cpp (153355 => 153356)


--- trunk/Source/WebCore/page/Chrome.cpp	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebCore/page/Chrome.cpp	2013-07-25 23:57:40 UTC (rev 153356)
@@ -178,9 +178,9 @@
     m_client->takeFocus(direction);
 }
 
-void Chrome::focusedNodeChanged(Node* node) const
+void Chrome::focusedElementChanged(Element* element) const
 {
-    m_client->focusedNodeChanged(node);
+    m_client->focusedElementChanged(element);
 }
 
 void Chrome::focusedFrameChanged(Frame* frame) const

Modified: trunk/Source/WebCore/page/Chrome.h (153355 => 153356)


--- trunk/Source/WebCore/page/Chrome.h	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebCore/page/Chrome.h	2013-07-25 23:57:40 UTC (rev 153356)
@@ -44,12 +44,12 @@
 class FileChooser;
 class FileIconLoader;
 class FloatRect;
+class Element;
 class Frame;
 class Geolocation;
 class HitTestResult;
 class IntRect;
 class NavigationAction;
-class Node;
 class Page;
 class PopupMenu;
 class PopupMenuClient;
@@ -107,7 +107,7 @@
     bool canTakeFocus(FocusDirection) const;
     void takeFocus(FocusDirection) const;
 
-    void focusedNodeChanged(Node*) const;
+    void focusedElementChanged(Element*) const;
     void focusedFrameChanged(Frame*) const;
 
     Page* createWindow(Frame*, const FrameLoadRequest&, const WindowFeatures&, const NavigationAction&) const;

Modified: trunk/Source/WebCore/page/ChromeClient.h (153355 => 153356)


--- trunk/Source/WebCore/page/ChromeClient.h	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebCore/page/ChromeClient.h	2013-07-25 23:57:40 UTC (rev 153356)
@@ -93,7 +93,7 @@
     virtual bool canTakeFocus(FocusDirection) = 0;
     virtual void takeFocus(FocusDirection) = 0;
 
-    virtual void focusedNodeChanged(Node*) = 0;
+    virtual void focusedElementChanged(Element*) = 0;
     virtual void focusedFrameChanged(Frame*) = 0;
 
     // The Frame pointer provides the ChromeClient with context about which

Modified: trunk/Source/WebKit/blackberry/ChangeLog (153355 => 153356)


--- trunk/Source/WebKit/blackberry/ChangeLog	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/blackberry/ChangeLog	2013-07-25 23:57:40 UTC (rev 153356)
@@ -1,3 +1,14 @@
+2013-07-25  Andreas Kling  <[email protected]>
+
+        ChromeClient::focusedNodeChanged() should be focusedElementChanged().
+        <http://webkit.org/b/119110>
+
+        Reviewed by Anders Carlsson.
+
+        * WebCoreSupport/ChromeClientBlackBerry.cpp:
+        (WebCore::ChromeClientBlackBerry::focusedElementChanged):
+        * WebCoreSupport/ChromeClientBlackBerry.h:
+
 2013-07-17  Kangil Han  <[email protected]>
 
         Use toHTMLMediaElement

Modified: trunk/Source/WebKit/blackberry/WebCoreSupport/ChromeClientBlackBerry.cpp (153355 => 153356)


--- trunk/Source/WebKit/blackberry/WebCoreSupport/ChromeClientBlackBerry.cpp	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/blackberry/WebCoreSupport/ChromeClientBlackBerry.cpp	2013-07-25 23:57:40 UTC (rev 153356)
@@ -202,7 +202,7 @@
     notImplemented();
 }
 
-void ChromeClientBlackBerry::focusedNodeChanged(Node*)
+void ChromeClientBlackBerry::focusedElementChanged(Element*)
 {
     m_webPagePrivate->m_inputHandler->focusedNodeChanged();
 }

Modified: trunk/Source/WebKit/blackberry/WebCoreSupport/ChromeClientBlackBerry.h (153355 => 153356)


--- trunk/Source/WebKit/blackberry/WebCoreSupport/ChromeClientBlackBerry.h	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/blackberry/WebCoreSupport/ChromeClientBlackBerry.h	2013-07-25 23:57:40 UTC (rev 153356)
@@ -42,7 +42,7 @@
     virtual void unfocus();
     virtual bool canTakeFocus(FocusDirection);
     virtual void takeFocus(FocusDirection);
-    virtual void focusedNodeChanged(Node*);
+    virtual void focusedElementChanged(Element*);
     virtual void focusedFrameChanged(Frame*);
     virtual bool shouldForceDocumentStyleSelectorUpdate();
     virtual Page* createWindow(Frame*, const FrameLoadRequest&, const WindowFeatures&, const NavigationAction&);

Modified: trunk/Source/WebKit/efl/ChangeLog (153355 => 153356)


--- trunk/Source/WebKit/efl/ChangeLog	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/efl/ChangeLog	2013-07-25 23:57:40 UTC (rev 153356)
@@ -1,3 +1,14 @@
+2013-07-25  Andreas Kling  <[email protected]>
+
+        ChromeClient::focusedNodeChanged() should be focusedElementChanged().
+        <http://webkit.org/b/119110>
+
+        Reviewed by Anders Carlsson.
+
+        * WebCoreSupport/ChromeClientEfl.cpp:
+        (WebCore::ChromeClientEfl::focusedElementChanged):
+        * WebCoreSupport/ChromeClientEfl.h:
+
 2013-07-15  Kwang Yul Seo  <[email protected]>
 
         NetworkStorageSession::createDefaultSession is only implemented by Soup

Modified: trunk/Source/WebKit/efl/WebCoreSupport/ChromeClientEfl.cpp (153355 => 153356)


--- trunk/Source/WebKit/efl/WebCoreSupport/ChromeClientEfl.cpp	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/efl/WebCoreSupport/ChromeClientEfl.cpp	2013-07-25 23:57:40 UTC (rev 153356)
@@ -107,7 +107,7 @@
     delete this;
 }
 
-void ChromeClientEfl::focusedNodeChanged(Node*)
+void ChromeClientEfl::focusedElementChanged(Element*)
 {
     notImplemented();
 }

Modified: trunk/Source/WebKit/efl/WebCoreSupport/ChromeClientEfl.h (153355 => 153356)


--- trunk/Source/WebKit/efl/WebCoreSupport/ChromeClientEfl.h	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/efl/WebCoreSupport/ChromeClientEfl.h	2013-07-25 23:57:40 UTC (rev 153356)
@@ -56,7 +56,7 @@
     virtual bool canTakeFocus(FocusDirection);
     virtual void takeFocus(FocusDirection);
 
-    virtual void focusedNodeChanged(Node*);
+    virtual void focusedElementChanged(Element*);
     virtual void focusedFrameChanged(Frame*);
 
     virtual Page* createWindow(Frame*, const FrameLoadRequest&, const WindowFeatures&, const NavigationAction&);

Modified: trunk/Source/WebKit/gtk/ChangeLog (153355 => 153356)


--- trunk/Source/WebKit/gtk/ChangeLog	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/gtk/ChangeLog	2013-07-25 23:57:40 UTC (rev 153356)
@@ -1,3 +1,14 @@
+2013-07-25  Andreas Kling  <[email protected]>
+
+        ChromeClient::focusedNodeChanged() should be focusedElementChanged().
+        <http://webkit.org/b/119110>
+
+        Reviewed by Anders Carlsson.
+
+        * WebCoreSupport/ChromeClientGtk.cpp:
+        (WebKit::ChromeClient::focusedElementChanged):
+        * WebCoreSupport/ChromeClientGtk.h:
+
 2013-07-23  Carlos Garcia Campos  <[email protected]>
 
         [GTK] GtkAdjustmentWatcher idle source not correctly handled

Modified: trunk/Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp (153355 => 153356)


--- trunk/Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp	2013-07-25 23:57:40 UTC (rev 153356)
@@ -343,7 +343,7 @@
     unfocus();
 }
 
-void ChromeClient::focusedNodeChanged(Node*)
+void ChromeClient::focusedElementChanged(Element*)
 {
 }
 

Modified: trunk/Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.h (153355 => 153356)


--- trunk/Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.h	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.h	2013-07-25 23:57:40 UTC (rev 153356)
@@ -58,7 +58,7 @@
         virtual bool canTakeFocus(FocusDirection);
         virtual void takeFocus(FocusDirection);
 
-        virtual void focusedNodeChanged(Node*);
+        virtual void focusedElementChanged(Element*);
         virtual void focusedFrameChanged(Frame*);
 
         virtual Page* createWindow(Frame*, const FrameLoadRequest&, const WindowFeatures&, const NavigationAction&);

Modified: trunk/Source/WebKit/mac/ChangeLog (153355 => 153356)


--- trunk/Source/WebKit/mac/ChangeLog	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/mac/ChangeLog	2013-07-25 23:57:40 UTC (rev 153356)
@@ -1,3 +1,14 @@
+2013-07-25  Andreas Kling  <[email protected]>
+
+        ChromeClient::focusedNodeChanged() should be focusedElementChanged().
+        <http://webkit.org/b/119110>
+
+        Reviewed by Anders Carlsson.
+
+        * WebCoreSupport/WebChromeClient.h:
+        * WebCoreSupport/WebChromeClient.mm:
+        (WebChromeClient::focusedElementChanged):
+
 2013-05-05  Geoffrey Garen  <[email protected]>
 
         Rolled back in r149527 with crash fixed.

Modified: trunk/Source/WebKit/mac/WebCoreSupport/WebChromeClient.h (153355 => 153356)


--- trunk/Source/WebKit/mac/WebCoreSupport/WebChromeClient.h	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/mac/WebCoreSupport/WebChromeClient.h	2013-07-25 23:57:40 UTC (rev 153356)
@@ -50,7 +50,7 @@
     virtual bool canTakeFocus(WebCore::FocusDirection) OVERRIDE;
     virtual void takeFocus(WebCore::FocusDirection) OVERRIDE;
 
-    virtual void focusedNodeChanged(WebCore::Node*) OVERRIDE;
+    virtual void focusedElementChanged(WebCore::Element*) OVERRIDE;
     virtual void focusedFrameChanged(WebCore::Frame*) OVERRIDE;
 
     virtual WebCore::Page* createWindow(WebCore::Frame*, const WebCore::FrameLoadRequest&, const WebCore::WindowFeatures&, const WebCore::NavigationAction&) OVERRIDE;

Modified: trunk/Source/WebKit/mac/WebCoreSupport/WebChromeClient.mm (153355 => 153356)


--- trunk/Source/WebKit/mac/WebCoreSupport/WebChromeClient.mm	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/mac/WebCoreSupport/WebChromeClient.mm	2013-07-25 23:57:40 UTC (rev 153356)
@@ -196,14 +196,14 @@
     }
 }
 
-void WebChromeClient::focusedNodeChanged(Node* node)
+void WebChromeClient::focusedElementChanged(Element* element)
 {
-    if (!node)
+    if (!element)
         return;
-    if (!isHTMLInputElement(node))
+    if (!isHTMLInputElement(element))
         return;
 
-    HTMLInputElement* inputElement = toHTMLInputElement(node);
+    HTMLInputElement* inputElement = toHTMLInputElement(element);
     if (!inputElement->isText())
         return;
 

Modified: trunk/Source/WebKit/qt/ChangeLog (153355 => 153356)


--- trunk/Source/WebKit/qt/ChangeLog	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/qt/ChangeLog	2013-07-25 23:57:40 UTC (rev 153356)
@@ -1,3 +1,14 @@
+2013-07-25  Andreas Kling  <[email protected]>
+
+        ChromeClient::focusedNodeChanged() should be focusedElementChanged().
+        <http://webkit.org/b/119110>
+
+        Reviewed by Anders Carlsson.
+
+        * WebCoreSupport/ChromeClientQt.cpp:
+        (WebCore::ChromeClientQt::focusedElementChanged):
+        * WebCoreSupport/ChromeClientQt.h:
+
 2013-07-23  Allan Sandfeld Jensen  <[email protected]>
 
         Remove deprecated and unused compositing settings

Modified: trunk/Source/WebKit/qt/WebCoreSupport/ChromeClientQt.cpp (153355 => 153356)


--- trunk/Source/WebKit/qt/WebCoreSupport/ChromeClientQt.cpp	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/qt/WebCoreSupport/ChromeClientQt.cpp	2013-07-25 23:57:40 UTC (rev 153356)
@@ -213,7 +213,7 @@
 }
 
 
-void ChromeClientQt::focusedNodeChanged(Node*)
+void ChromeClientQt::focusedElementChanged(Element*)
 {
 }
 

Modified: trunk/Source/WebKit/qt/WebCoreSupport/ChromeClientQt.h (153355 => 153356)


--- trunk/Source/WebKit/qt/WebCoreSupport/ChromeClientQt.h	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/qt/WebCoreSupport/ChromeClientQt.h	2013-07-25 23:57:40 UTC (rev 153356)
@@ -77,7 +77,7 @@
     virtual bool canTakeFocus(FocusDirection);
     virtual void takeFocus(FocusDirection);
 
-    virtual void focusedNodeChanged(Node*);
+    virtual void focusedElementChanged(Element*);
     virtual void focusedFrameChanged(Frame*);
 
     virtual Page* createWindow(Frame*, const FrameLoadRequest&, const WindowFeatures&, const NavigationAction&);

Modified: trunk/Source/WebKit/win/ChangeLog (153355 => 153356)


--- trunk/Source/WebKit/win/ChangeLog	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/win/ChangeLog	2013-07-25 23:57:40 UTC (rev 153356)
@@ -1,3 +1,14 @@
+2013-07-25  Andreas Kling  <[email protected]>
+
+        ChromeClient::focusedNodeChanged() should be focusedElementChanged().
+        <http://webkit.org/b/119110>
+
+        Reviewed by Anders Carlsson.
+
+        * WebCoreSupport/WebChromeClient.cpp:
+        (WebChromeClient::focusedElementChanged):
+        * WebCoreSupport/WebChromeClient.h:
+
 2013-07-23  [email protected]  <[email protected]>
 
         [Win] WebView didReceiveIcon delegate call doesn't have the icon in it

Modified: trunk/Source/WebKit/win/WebCoreSupport/WebChromeClient.cpp (153355 => 153356)


--- trunk/Source/WebKit/win/WebCoreSupport/WebChromeClient.cpp	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/win/WebCoreSupport/WebChromeClient.cpp	2013-07-25 23:57:40 UTC (rev 153356)
@@ -163,7 +163,7 @@
     }
 }
 
-void WebChromeClient::focusedNodeChanged(Node*)
+void WebChromeClient::focusedElementChanged(Element*)
 {
 }
 

Modified: trunk/Source/WebKit/win/WebCoreSupport/WebChromeClient.h (153355 => 153356)


--- trunk/Source/WebKit/win/WebCoreSupport/WebChromeClient.h	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/win/WebCoreSupport/WebChromeClient.h	2013-07-25 23:57:40 UTC (rev 153356)
@@ -54,7 +54,7 @@
     virtual bool canTakeFocus(WebCore::FocusDirection);
     virtual void takeFocus(WebCore::FocusDirection);
 
-    virtual void focusedNodeChanged(WebCore::Node*);
+    virtual void focusedElementChanged(WebCore::Element*);
     virtual void focusedFrameChanged(WebCore::Frame*);
 
     virtual WebCore::Page* createWindow(WebCore::Frame*, const WebCore::FrameLoadRequest&, const WebCore::WindowFeatures&, const WebCore::NavigationAction&);

Modified: trunk/Source/WebKit/wince/ChangeLog (153355 => 153356)


--- trunk/Source/WebKit/wince/ChangeLog	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/wince/ChangeLog	2013-07-25 23:57:40 UTC (rev 153356)
@@ -1,3 +1,14 @@
+2013-07-25  Andreas Kling  <[email protected]>
+
+        ChromeClient::focusedNodeChanged() should be focusedElementChanged().
+        <http://webkit.org/b/119110>
+
+        Reviewed by Anders Carlsson.
+
+        * WebCoreSupport/ChromeClientWinCE.cpp:
+        (WebKit::ChromeClientWinCE::focusedElementChanged):
+        * WebCoreSupport/ChromeClientWinCE.h:
+
 2013-06-26  Patrick Gansterer  <[email protected]>
 
         Unreviewed WinCE build fix after r152065.

Modified: trunk/Source/WebKit/wince/WebCoreSupport/ChromeClientWinCE.cpp (153355 => 153356)


--- trunk/Source/WebKit/wince/WebCoreSupport/ChromeClientWinCE.cpp	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/wince/WebCoreSupport/ChromeClientWinCE.cpp	2013-07-25 23:57:40 UTC (rev 153356)
@@ -165,7 +165,7 @@
     unfocus();
 }
 
-void ChromeClientWinCE::focusedNodeChanged(Node*)
+void ChromeClientWinCE::focusedElementChanged(Element*)
 {
     notImplemented();
 }

Modified: trunk/Source/WebKit/wince/WebCoreSupport/ChromeClientWinCE.h (153355 => 153356)


--- trunk/Source/WebKit/wince/WebCoreSupport/ChromeClientWinCE.h	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit/wince/WebCoreSupport/ChromeClientWinCE.h	2013-07-25 23:57:40 UTC (rev 153356)
@@ -48,7 +48,7 @@
     virtual bool canTakeFocus(WebCore::FocusDirection);
     virtual void takeFocus(WebCore::FocusDirection);
 
-    virtual void focusedNodeChanged(WebCore::Node*);
+    virtual void focusedElementChanged(WebCore::Element*);
     virtual void focusedFrameChanged(WebCore::Frame*);
 
     // The Frame pointer provides the ChromeClient with context about which

Modified: trunk/Source/WebKit2/ChangeLog (153355 => 153356)


--- trunk/Source/WebKit2/ChangeLog	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit2/ChangeLog	2013-07-25 23:57:40 UTC (rev 153356)
@@ -1,3 +1,14 @@
+2013-07-25  Andreas Kling  <[email protected]>
+
+        ChromeClient::focusedNodeChanged() should be focusedElementChanged().
+        <http://webkit.org/b/119110>
+
+        Reviewed by Anders Carlsson.
+
+        * WebProcess/WebCoreSupport/WebChromeClient.cpp:
+        (WebKit::WebChromeClient::focusedElementChanged):
+        * WebProcess/WebCoreSupport/WebChromeClient.h:
+
 2013-07-25  Kwang Yul Seo  <[email protected]>
 
         [WK2][Soup] Add private browsing support

Modified: trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebChromeClient.cpp (153355 => 153356)


--- trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebChromeClient.cpp	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebChromeClient.cpp	2013-07-25 23:57:40 UTC (rev 153356)
@@ -160,18 +160,18 @@
     m_page->send(Messages::WebPageProxy::TakeFocus(direction));
 }
 
-void WebChromeClient::focusedNodeChanged(Node* node)
+void WebChromeClient::focusedElementChanged(Element* element)
 {
-    if (!node)
+    if (!element)
         return;
-    if (!isHTMLInputElement(node))
+    if (!isHTMLInputElement(element))
         return;
 
-    HTMLInputElement* inputElement = toHTMLInputElement(node);
+    HTMLInputElement* inputElement = toHTMLInputElement(element);
     if (!inputElement->isText())
         return;
 
-    WebFrameLoaderClient* webFrameLoaderClient = toWebFrameLoaderClient(node->document()->frame()->loader()->client());
+    WebFrameLoaderClient* webFrameLoaderClient = toWebFrameLoaderClient(element->document()->frame()->loader()->client());
     WebFrame* webFrame = webFrameLoaderClient ? webFrameLoaderClient->webFrame() : 0;
     ASSERT(webFrame);
     m_page->injectedBundleFormClient().didFocusTextField(m_page, inputElement, webFrame);

Modified: trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebChromeClient.h (153355 => 153356)


--- trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebChromeClient.h	2013-07-25 23:23:42 UTC (rev 153355)
+++ trunk/Source/WebKit2/WebProcess/WebCoreSupport/WebChromeClient.h	2013-07-25 23:57:40 UTC (rev 153356)
@@ -63,7 +63,7 @@
     virtual bool canTakeFocus(WebCore::FocusDirection) OVERRIDE;
     virtual void takeFocus(WebCore::FocusDirection) OVERRIDE;
 
-    virtual void focusedNodeChanged(WebCore::Node*) OVERRIDE;
+    virtual void focusedElementChanged(WebCore::Element*) OVERRIDE;
     virtual void focusedFrameChanged(WebCore::Frame*) OVERRIDE;
 
     // The Frame pointer provides the ChromeClient with context about which
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to