Changeset 107992 in webkit

Timestamp:

Feb 16, 2012 4:04:35 PM (12 years ago)

Author:

tonikitoo@webkit.org

Message:

2012-02-16 Antonio Gomes <​agomes@rim.com>

[BlackBerry] Implemented a way to defer client navigation change client notifications
​https://bugs.webkit.org/show_bug.cgi?id=78848

Reviewed by Rob Buis.

Currently we postpone touch_down till touch_up if user touches
the screen and an input field has the WebKit focus. This is done
so we can scroll the page without hidding the vkb needlessly.

However, it breaks the conversion of touch to mouse events
if an input field has the focus in the following scenario:
an <input type=text> is focused and an user grab and-drag
a <input type=range> knob/slide. It does not work until the
user unfocuses the currently focused edit field.

Patch introduces a way to unfocus a currently focused input field,
without requesting the client to show or hide the virtual keyboard
right way. Instead it gets a delayed notification of the vkb mode requested
at either touch_released/mouse_up or touch_cancel time.

For now, due to content side issues with major web sites,
only delay navigation mode notification changes if we are not dealing with
input modes.

• WebKitSupport/InputHandler.cpp: (BlackBerry::WebKit::InputHandler::InputHandler): (BlackBerry::WebKit::InputHandler::nodeFocused): (BlackBerry::WebKit::InputHandler::setElementFocused): (BlackBerry::WebKit::InputHandler::setNavigationMode): (WebKit): (BlackBerry::WebKit::InputHandler::setDelayClientNotificationOfNavigationModeChange): (BlackBerry::WebKit::InputHandler::processPendingClientNavigationModeChangeNotification): (BlackBerry::WebKit::InputHandler::notifyClientOfNavigationModeChange): (BlackBerry::WebKit::InputHandler::willOpenPopupForNode): (BlackBerry::WebKit::InputHandler::setPopupListIndexes): (BlackBerry::WebKit::InputHandler::spannableTextInRange): (BlackBerry::WebKit::InputHandler::removeComposedText): (BlackBerry::WebKit::InputHandler::firstSpanInString): (BlackBerry::WebKit::InputHandler::setText): (BlackBerry::WebKit::InputHandler::setRelativeCursorPosition):
• WebKitSupport/InputHandler.h: (InputHandler):

Location:

trunk/Source/WebKit/blackberry

Files:

• ChangeLog (modified) (1 diff)
• WebKitSupport/InputHandler.cpp (modified) (3 diffs)
• WebKitSupport/InputHandler.h (modified) (3 diffs)

trunk/Source/WebKit/blackberry/ChangeLog

@@ +1 @@
+2012-02-16  Antonio Gomes  <agomes@rim.com>
+
+        [BlackBerry] Implemented a way to defer client navigation change client notifications
+        https://bugs.webkit.org/show_bug.cgi?id=78848
+
+        Reviewed by Rob Buis.
+
+        Currently we postpone touch_down till touch_up if user touches
+        the screen and an input field has the WebKit focus. This is done
+        so we can scroll the page without hidding the vkb needlessly.
+
+        However, it breaks the conversion of touch to mouse events
+        if an input field has the focus in the following scenario:
+        an <input type=text> is focused and an user grab and-drag
+        a <input type=range> knob/slide. It does not work until the
+        user unfocuses the currently focused edit field.
+
+        Patch introduces a way to unfocus a currently focused input field,
+        without requesting the client to show or hide the virtual keyboard
+        right way. Instead it gets a delayed notification of the vkb mode requested
+        at either touch_released/mouse_up or touch_cancel time.
+
+        For now, due to content side issues with major web sites,
+        only delay navigation mode notification changes if we are not dealing with
+        input modes.
+
+        * WebKitSupport/InputHandler.cpp:
+        (BlackBerry::WebKit::InputHandler::InputHandler):
+        (BlackBerry::WebKit::InputHandler::nodeFocused):
+        (BlackBerry::WebKit::InputHandler::setElementFocused):
+        (BlackBerry::WebKit::InputHandler::setNavigationMode):
+        (WebKit):
+        (BlackBerry::WebKit::InputHandler::setDelayClientNotificationOfNavigationModeChange):
+        (BlackBerry::WebKit::InputHandler::processPendingClientNavigationModeChangeNotification):
+        (BlackBerry::WebKit::InputHandler::notifyClientOfNavigationModeChange):
+        (BlackBerry::WebKit::InputHandler::willOpenPopupForNode):
+        (BlackBerry::WebKit::InputHandler::setPopupListIndexes):
+        (BlackBerry::WebKit::InputHandler::spannableTextInRange):
+        (BlackBerry::WebKit::InputHandler::removeComposedText):
+        (BlackBerry::WebKit::InputHandler::firstSpanInString):
+        (BlackBerry::WebKit::InputHandler::setText):
+        (BlackBerry::WebKit::InputHandler::setRelativeCursorPosition):
+        * WebKitSupport/InputHandler.h:
+        (InputHandler):
+
 2012-02-16  Antonio Gomes  <agomes@rim.com>
 

trunk/Source/WebKit/blackberry/WebKitSupport/InputHandler.cpp

@@ -117 +117 @@
     , m_composingTextStart(0)
     , m_composingTextEnd(0)
+    , m_pendingKeyboardStateChange(NoChange)
+    , m_delayClientNotificationOfNavigationModeChange(false)
 {
 }
@@ -475 +477 @@
     FocusLog(BlackBerry::Platform::LogLevelInfo, "InputHandler::setElementFocused, Type=%d, Style=%d", type, m_currentFocusElementTextEditMask);
 
-    m_webPage->m_client->inputFocusGained(type, m_currentFocusElementTextEditMask);
+    m_webPage->m_client->inputFocusGained(type,
+                                          m_currentFocusElementTextEditMask,
+                                          m_delayClientNotificationOfNavigationModeChange /* wait an explicit keyboard show call */);
     m_navigationMode = true;
 
@@ -749 +753 @@
 
     if (sendMessage)
+        notifyClientOfNavigationModeChange(active);
+}
+
+void InputHandler::setDelayClientNotificationOfNavigationModeChange(bool value)
+{
+    m_delayClientNotificationOfNavigationModeChange = value;
+    m_pendingKeyboardStateChange = NoChange;
+}
+
+void InputHandler::processPendingClientNavigationModeChangeNotification()
+{
+    if (!m_delayClientNotificationOfNavigationModeChange) {
+        ASSERT(m_pendingKeyboardStateChange == NoChange);
+        return;
+    }
+
+    m_delayClientNotificationOfNavigationModeChange = false;
+
+    if (m_pendingKeyboardStateChange == NoChange)
+        return;
+
+    notifyClientOfNavigationModeChange(m_pendingKeyboardStateChange == Visible);
+    m_pendingKeyboardStateChange = NoChange;
+}
+
+void InputHandler::notifyClientOfNavigationModeChange(bool active)
+{
+    if (!m_delayClientNotificationOfNavigationModeChange) {
         m_webPage->m_client->inputSetNavigationMode(active);
+        return;
+    }
+
+    m_pendingKeyboardStateChange = active ? Visible : NotVisible;
 }
 

trunk/Source/WebKit/blackberry/WebKitSupport/InputHandler.h

@@ -78 +78 @@
     void setInputValue(const WTF::String&);
 
+    void setDelayClientNotificationOfNavigationModeChange(bool value);
+    void processPendingClientNavigationModeChangeNotification();
+
+    void notifyClientOfNavigationModeChange(bool active);
+
     bool isInputMode() const { return isActiveTextEdit(); }
     bool isMultilineInputMode() const { return isActiveTextEdit() && elementType(m_currentFocusElement.get()) == BlackBerry::Platform::InputTypeTextArea; }
@@ -118 +123 @@
 
 private:
+    enum PendingKeyboardStateChange { NoChange, Visible, NotVisible };
+
     void setElementFocused(WebCore::Element*);
     void setPluginFocused(WebCore::Element*);
@@ -184 +191 @@
     int m_composingTextStart;
     int m_composingTextEnd;
+
+    PendingKeyboardStateChange m_pendingKeyboardStateChange;
+    bool m_delayClientNotificationOfNavigationModeChange;
 };