[BLACKBERRY] Add notification if an element does not have touch move or mouse move...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 21 Aug 2012 00:20:43 +0000 (00:20 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 21 Aug 2012 00:20:43 +0000 (00:20 +0000)
https://bugs.webkit.org/show_bug.cgi?id=94529

Patch by Genevieve Mak <gmak@rim.com> on 2012-08-20
Reviewed by George Staikos.

Send a notification to the client if the fat finger element does
not have a mouse move or touch move handler or if it is empty.
PR #177701

Reviewed Internally By Mike Lattanzio and Antonio Gomes.

* Api/WebPageClient.h:
* WebKitSupport/TouchEventHandler.cpp:
(BlackBerry::WebKit::TouchEventHandler::handleTouchPoint):

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

Source/WebKit/blackberry/Api/WebPageClient.h
Source/WebKit/blackberry/ChangeLog
Source/WebKit/blackberry/WebKitSupport/TouchEventHandler.cpp

index a3847d7..be574a6 100644 (file)
@@ -101,6 +101,7 @@ public:
     virtual void notifyRunLayoutTestsFinished() = 0;
 
     virtual void notifyInRegionScrollingStartingPointChanged(const std::vector<Platform::ScrollViewBase*>&) = 0;
+    virtual void notifyNoMouseMoveOrTouchMoveHandlers() = 0;
 
     virtual void notifyDocumentOnLoad() = 0;
 
index 0825722..67d6ddc 100644 (file)
@@ -1,3 +1,20 @@
+2012-08-20  Genevieve Mak  <gmak@rim.com>
+
+        [BLACKBERRY] Add notification if an element does not have touch move or mouse move handlers.
+        https://bugs.webkit.org/show_bug.cgi?id=94529
+
+        Reviewed by George Staikos.
+
+        Send a notification to the client if the fat finger element does
+        not have a mouse move or touch move handler or if it is empty.
+        PR #177701
+
+        Reviewed Internally By Mike Lattanzio and Antonio Gomes.
+
+        * Api/WebPageClient.h:
+        * WebKitSupport/TouchEventHandler.cpp:
+        (BlackBerry::WebKit::TouchEventHandler::handleTouchPoint):
+
 2012-08-17  Antonio Gomes  <agomes@rim.com>
 
         [BlackBerry] Allow in-region accelerated scrolling take advantage of defer_blits
index 76b88ef..ee3a75f 100644 (file)
@@ -196,6 +196,9 @@ bool TouchEventHandler::handleTouchPoint(Platform::TouchPoint& point, bool useFa
             Element* possibleTargetNodeForMouseMoveEvents = static_cast<Element*>(m_lastFatFingersResult.positionWasAdjusted() ? elementUnderFatFinger : m_lastFatFingersResult.node());
             m_convertTouchToMouse = pureWithMouseConversion ? true : shouldConvertTouchToMouse(possibleTargetNodeForMouseMoveEvents);
 
+            if (!possibleTargetNodeForMouseMoveEvents || (!possibleTargetNodeForMouseMoveEvents->hasEventListeners(eventNames().touchmoveEvent) && !m_convertTouchToMouse))
+                m_webPage->client()->notifyNoMouseMoveOrTouchMoveHandlers();
+
             if (elementUnderFatFinger)
                 drawTapHighlight();