Clean up logic for updating caps lock indicator state
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 29 May 2015 23:59:26 +0000 (23:59 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 29 May 2015 23:59:26 +0000 (23:59 +0000)
https://bugs.webkit.org/show_bug.cgi?id=145329

Reviewed by Sam Weinig.

Remove logic to explicitly update the state of the caps lock indicator, which was used
when the caps logic indicator was implemented as part of RenderTextControl. Currently,
the caps logic indicator is implemented as part of a shadow DOM and we have existing
logic to forward DOM focus and blur events to it so that it may update its state.

No change in behavior.

* editing/FrameSelection.cpp:
(WebCore::FrameSelection::focusedOrActiveStateChanged): Deleted.
* editing/FrameSelection.h:
* page/EventHandler.cpp:
(WebCore::EventHandler::keyEvent): Inline the code from EventHandler::capsLockStateMayHaveChanged()
into this function because it is the only caller of EventHandler::capsLockStateMayHaveChanged() after
we removed the call site in FrameSelection::focusedOrActiveStateChanged().
(WebCore::EventHandler::capsLockStateMayHaveChanged): Deleted.
* page/EventHandler.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@185020 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/editing/FrameSelection.cpp
Source/WebCore/editing/FrameSelection.h
Source/WebCore/page/EventHandler.cpp
Source/WebCore/page/EventHandler.h

index c3d40a0..fe0d0b6 100644 (file)
@@ -1,3 +1,27 @@
+2015-05-29  Daniel Bates  <dabates@apple.com>
+
+        Clean up logic for updating caps lock indicator state
+        https://bugs.webkit.org/show_bug.cgi?id=145329
+
+        Reviewed by Sam Weinig.
+
+        Remove logic to explicitly update the state of the caps lock indicator, which was used
+        when the caps logic indicator was implemented as part of RenderTextControl. Currently,
+        the caps logic indicator is implemented as part of a shadow DOM and we have existing
+        logic to forward DOM focus and blur events to it so that it may update its state.
+
+        No change in behavior.
+
+        * editing/FrameSelection.cpp:
+        (WebCore::FrameSelection::focusedOrActiveStateChanged): Deleted.
+        * editing/FrameSelection.h:
+        * page/EventHandler.cpp:
+        (WebCore::EventHandler::keyEvent): Inline the code from EventHandler::capsLockStateMayHaveChanged()
+        into this function because it is the only caller of EventHandler::capsLockStateMayHaveChanged() after
+        we removed the call site in FrameSelection::focusedOrActiveStateChanged().
+        (WebCore::EventHandler::capsLockStateMayHaveChanged): Deleted.
+        * page/EventHandler.h:
+
 2015-05-29  Zalan Bujtas  <zalan@apple.com>
 
         Text disappears shortly after page load on Nexus 7 site.
index 9e9414f..02264d5 100644 (file)
@@ -1839,9 +1839,6 @@ void FrameSelection::focusedOrActiveStateChanged()
         setSelectionFromNone();
     setCaretVisibility(activeAndFocused ? Visible : Hidden);
 
-    // Update for caps lock state
-    m_frame->eventHandler().capsLockStateMayHaveChanged();
-
     // Because StyleResolver::checkOneSelector() and
     // RenderTheme::isFocused() check if the frame is active, we have to
     // update style and theme state that depended on those.
index 2b7f234..db7d667 100644 (file)
@@ -200,7 +200,7 @@ public:
     bool isCaretBlinkingSuspended() const { return m_isCaretBlinkingSuspended; }
 
     // Focus
-    WEBCORE_EXPORT void setFocused(bool);
+    void setFocused(bool);
     bool isFocused() const { return m_focused; }
     WEBCORE_EXPORT bool isFocusedAndActive() const;
     void pageActivationChanged();
index 8ac66dc..dba2df6 100644 (file)
@@ -3078,8 +3078,12 @@ bool EventHandler::keyEvent(const PlatformKeyboardEvent& initialKeyEvent)
         return false;
 #endif
 
-    if (initialKeyEvent.windowsVirtualKeyCode() == VK_CAPITAL)
-        capsLockStateMayHaveChanged();
+    if (initialKeyEvent.windowsVirtualKeyCode() == VK_CAPITAL) {
+        if (auto* element = m_frame.document()->focusedElement()) {
+            if (is<HTMLInputElement>(*element))
+                downcast<HTMLInputElement>(*element).capsLockStateMayHaveChanged();
+        }
+    }
 
 #if ENABLE(PAN_SCROLLING)
     if (m_frame.mainFrame().eventHandler().panScrollInProgress()) {
@@ -3750,15 +3754,6 @@ void EventHandler::defaultTabEventHandler(KeyboardEvent* event)
         event->setDefaultHandled();
 }
 
-void EventHandler::capsLockStateMayHaveChanged()
-{
-    Document* document = m_frame.document();
-    if (auto* element = document->focusedElement()) {
-        if (is<HTMLInputElement>(*element))
-            downcast<HTMLInputElement>(*element).capsLockStateMayHaveChanged();
-    }
-}
-
 void EventHandler::sendScrollEvent()
 {
     setFrameWasScrolledByUser();
index 866199c..164b930 100644 (file)
@@ -259,8 +259,6 @@ public:
 #endif
 
     void focusDocumentView();
-
-    void capsLockStateMayHaveChanged(); // Only called by FrameSelection
     
     WEBCORE_EXPORT void sendScrollEvent(); // Ditto