diff options
author | Ben Murdoch <benm@google.com> | 2010-03-11 09:39:27 -0800 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2010-03-11 09:39:27 -0800 |
commit | b3521bb4bae36e8cde56e41c42b0ccb8d1eef604 (patch) | |
tree | 93a559f83bf51dcc2bf174565c7e646bc6dac1d0 | |
parent | aa699fff907b14728bd8e515e3ae498620ed9afe (diff) | |
parent | 0361ff17d43cdfdc6c9d3ae65e8716a4cbeed862 (diff) | |
download | external_webkit-b3521bb4bae36e8cde56e41c42b0ccb8d1eef604.zip external_webkit-b3521bb4bae36e8cde56e41c42b0ccb8d1eef604.tar.gz external_webkit-b3521bb4bae36e8cde56e41c42b0ccb8d1eef604.tar.bz2 |
Merge "Prepare touch key modifiers for upstreaming."
-rw-r--r-- | WebCore/page/EventHandler.cpp | 7 | ||||
-rw-r--r-- | WebCore/platform/PlatformTouchEvent.h | 1 | ||||
-rw-r--r-- | WebCore/platform/android/PlatformTouchEventAndroid.cpp | 3 |
3 files changed, 2 insertions, 9 deletions
diff --git a/WebCore/page/EventHandler.cpp b/WebCore/page/EventHandler.cpp index 233fd05..6082cfe 100644 --- a/WebCore/page/EventHandler.cpp +++ b/WebCore/page/EventHandler.cpp @@ -2554,7 +2554,7 @@ static PassRefPtr<TouchList> assembleTargetTouches(Touch* touchTarget, TouchList { RefPtr<TouchList> targetTouches = TouchList::create(); - for (int i = 0; i < touches->length(); ++i) { + for (unsigned i = 0; i < touches->length(); ++i) { if (touches->item(i)->target()->toNode()->isSameNode(touchTarget->target()->toNode())) targetTouches->append(touches->item(i)); } @@ -2573,7 +2573,7 @@ bool EventHandler::handleTouchEvent(const PlatformTouchEvent& event) const Vector<PlatformTouchPoint>& points = event.touchPoints(); AtomicString* eventName = 0; - for (int i = 0; i < points.size(); ++i) { + for (unsigned i = 0; i < points.size(); ++i) { const PlatformTouchPoint& point = points[i]; IntPoint pagePoint = documentPointForWindowPoint(m_frame, point.pos()); HitTestResult result = hitTestResultAtPoint(pagePoint, /*allowShadowContent*/ false); @@ -2653,7 +2653,6 @@ bool EventHandler::handleTouchEvent(const PlatformTouchEvent& event) *eventName, touchEventTarget->toNode()->document()->defaultView(), 0, 0, 0, 0, event.ctrlKey(), event.altKey(), event.shiftKey(), event.metaKey()); - ExceptionCode ec = 0; touchEventTarget->dispatchEvent(cancelEv.get(), ec); defaultPrevented |= cancelEv->defaultPrevented(); @@ -2693,7 +2692,6 @@ bool EventHandler::handleTouchEvent(const PlatformTouchEvent& event) *eventName, touchEventTarget->toNode()->document()->defaultView(), 0, 0, 0, 0, event.ctrlKey(), event.altKey(), event.shiftKey(), event.metaKey()); - ExceptionCode ec = 0; touchEventTarget->dispatchEvent(longpressEv.get(), ec); defaultPrevented |= longpressEv->defaultPrevented(); @@ -2704,7 +2702,6 @@ bool EventHandler::handleTouchEvent(const PlatformTouchEvent& event) *eventName, touchEventTarget->toNode()->document()->defaultView(), 0, 0, 0, 0, event.ctrlKey(), event.altKey(), event.shiftKey(), event.metaKey()); - ExceptionCode ec = 0; touchEventTarget->dispatchEvent(doubleTapEv.get(), ec); defaultPrevented |= doubleTapEv->defaultPrevented(); diff --git a/WebCore/platform/PlatformTouchEvent.h b/WebCore/platform/PlatformTouchEvent.h index 7e2ac84..d300a82 100644 --- a/WebCore/platform/PlatformTouchEvent.h +++ b/WebCore/platform/PlatformTouchEvent.h @@ -60,7 +60,6 @@ public: #if PLATFORM(QT) PlatformTouchEvent(QTouchEvent*); #elif PLATFORM(ANDROID) - // TODO (benm): eventTime and metaState are new and need to be upstreamed. PlatformTouchEvent(const IntPoint& windowPos, TouchEventType, PlatformTouchPoint::State, int metaState); #endif diff --git a/WebCore/platform/android/PlatformTouchEventAndroid.cpp b/WebCore/platform/android/PlatformTouchEventAndroid.cpp index 0b3ac13..e33aec0 100644 --- a/WebCore/platform/android/PlatformTouchEventAndroid.cpp +++ b/WebCore/platform/android/PlatformTouchEventAndroid.cpp @@ -30,7 +30,6 @@ namespace WebCore { -// TODO(benm): This enum needs upstreaming. // These values should be kept in sync with those defined in the android.view.KeyEvent class from the Android SDK. enum AndroidMetaKeyState { META_SHIFT_ON = 0x01, @@ -38,14 +37,12 @@ enum AndroidMetaKeyState { META_SYM_ON = 0x04 }; -// TODO (benm): metaState are new and needs to be upstreamed. PlatformTouchEvent::PlatformTouchEvent(const IntPoint& windowPos, TouchEventType type, PlatformTouchPoint::State state, int metaState) : m_type(type) , m_metaKey(false) { m_touchPoints.append(PlatformTouchPoint(windowPos, state)); - // TODO(benm): metaState needs upstreaming. m_altKey = metaState & META_ALT_ON; m_shiftKey = metaState & META_SHIFT_ON; m_ctrlKey = metaState & META_SYM_ON; |