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

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

Issue 206603002: Add EventHandlerRegistry (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Tests now seem to pass. Created 6 years, 9 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 a9c7d4dbf82096893986cf498a685d28b312a72e..ed7ba2890845be2af3402b6f292031c484b782e2 100644
--- a/Source/core/frame/DOMWindow.cpp
+++ b/Source/core/frame/DOMWindow.cpp
@@ -405,13 +405,6 @@ PassRefPtr<Document> DOMWindow::installNewDocument(const String& mimeType, const
}
m_frame->selection().updateSecureKeyboardEntryIfActive();
-
- if (m_frame->isMainFrame()) {
- m_frame->notifyChromeClientWheelEventHandlerCountChanged();
- if (m_document->hasTouchEventHandlers())
- m_frame->host()->chrome().client().needTouchEvents(true);
- }
-
return m_document;
}
@@ -1517,9 +1510,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);
}
@@ -1549,11 +1540,6 @@ bool DOMWindow::removeEventListener(const AtomicString& eventType, EventListener
if (!EventTarget::removeEventListener(eventType, listener, useCapture))
return false;
- if (Document* document = this->document()) {
- if (isTouchEventType(eventType))
- document->didRemoveTouchEventHandler(document);
- }
-
lifecycleNotifier().notifyRemoveEventListener(this, eventType);
if (eventType == EventTypeNames::unload) {
@@ -1615,9 +1601,6 @@ void DOMWindow::removeAllEventListeners()
lifecycleNotifier().notifyRemoveAllEventListeners(this);
- if (Document* document = this->document())
- document->didClearTouchEventHandlers(document);
-
removeAllUnloadEventListeners(this);
removeAllBeforeUnloadEventListeners(this);
}

Powered by Google App Engine
This is Rietveld 408576698