Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(930)

Unified Diff: Source/core/frame/DOMWindow.cpp

Issue 225903009: Migrate touch events to EventHandlerRegistry (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebased. Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/frame/DOMWindow.cpp
diff --git a/Source/core/frame/DOMWindow.cpp b/Source/core/frame/DOMWindow.cpp
index 87085bb74e1a3061a2fde3063a3a7dd785fa71c0..ae3144d4a63595d7321affa0f52c5115891a2dba 100644
--- a/Source/core/frame/DOMWindow.cpp
+++ b/Source/core/frame/DOMWindow.cpp
@@ -414,12 +414,6 @@ PassRefPtrWillBeRawPtr<Document> DOMWindow::installNewDocument(const String& mim
}
m_frame->selection().updateSecureKeyboardEntryIfActive();
-
- if (m_frame->isMainFrame()) {
- if (m_document->hasTouchEventHandlers())
- m_frame->host()->chrome().client().needTouchEvents(true);
- }
-
// FIXME: Oilpan: Remove .get() when m_document becomes Member<>.
return m_document.get();
}
@@ -1543,9 +1537,7 @@ bool DOMWindow::addEventListener(const AtomicString& eventType, PassRefPtr<Event
if (Document* document = this->document()) {
document->addListenerTypeIfNeeded(eventType);
- if (isTouchEventType(eventType))
- document->didAddTouchEventHandler(document);
- else if (eventType == EventTypeNames::storage)
+ if (eventType == EventTypeNames::storage)
didAddStorageEventListener(this);
}
@@ -1578,11 +1570,6 @@ bool DOMWindow::removeEventListener(const AtomicString& eventType, EventListener
if (m_frame && m_frame->host())
m_frame->host()->eventHandlerRegistry().didRemoveEventHandler(*this, eventType);
- if (Document* document = this->document()) {
- if (isTouchEventType(eventType))
- document->didRemoveTouchEventHandler(document);
- }
-
lifecycleNotifier().notifyRemoveEventListener(this, eventType);
if (eventType == EventTypeNames::unload) {
@@ -1651,9 +1638,6 @@ void DOMWindow::removeAllEventListenersInternal(BroadcastListenerRemoval mode)
if (mode == DoBroadcastListenerRemoval) {
if (m_frame && m_frame->host())
m_frame->host()->eventHandlerRegistry().didRemoveAllEventHandlers(*this);
-
- if (Document* document = this->document())
- document->didClearTouchEventHandlers(document);
}
removeAllUnloadEventListeners(this);

Powered by Google App Engine
This is Rietveld 408576698