Add EventHandlerRegistry

Source/core/dom/ScrollEventHandlerController.cpp

Index: Source/core/dom/ScrollEventHandlerController.cpp
diff --git a/Source/core/dom/WheelController.cpp b/Source/core/dom/ScrollEventHandlerController.cpp
similarity index 54%
copy from Source/core/dom/WheelController.cpp
copy to Source/core/dom/ScrollEventHandlerController.cpp
index b241ad52cff223e50a93f45813933878a1b3099e..8df7a05621171c86baee2e958a4a4a2a9eac1b83 100644
--- a/Source/core/dom/WheelController.cpp
+++ b/Source/core/dom/ScrollEventHandlerController.cpp
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2013 Google, Inc. All rights reserved.
+ * Copyright (C) 2014 Google, Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -24,43 +24,42 @@
  */
 
 #include "config.h"
-#include "core/dom/WheelController.h"
+#include "core/dom/ScrollEventHandlerController.h"
 
 #include "core/dom/Document.h"
 #include "core/events/ThreadLocalEventNames.h"
-#include "core/events/WheelEvent.h"
-#include "core/frame/LocalFrame.h"
+#include "core/frame/DOMWindow.h"
 #include "core/page/Page.h"
 #include "core/page/scrolling/ScrollingCoordinator.h"
 
 namespace WebCore {
 
-WheelController::WheelController(Document& document)
+ScrollEventHandlerController::ScrollEventHandlerController(Document& document)
     : DOMWindowLifecycleObserver(document.domWindow())
-    , m_wheelEventHandlerCount(0)
+    , m_scrollEventHandlerCount(0)
 {
 }
 
-WheelController::~WheelController()
+ScrollEventHandlerController::~ScrollEventHandlerController()
 {
 }
 
-const char* WheelController::supplementName()
+const char* ScrollEventHandlerController::supplementName()
 {
-    return "WheelController";
+    return "ScrollEventHandlerController";
 }
 
-WheelController* WheelController::from(Document& document)
+ScrollEventHandlerController* ScrollEventHandlerController::from(Document& document)
 {
-    WheelController* controller = static_cast<WheelController*>(DocumentSupplement::from(document, supplementName()));
+    ScrollEventHandlerController* controller = static_cast<ScrollEventHandlerController*>(DocumentSupplement::from(document, supplementName()));
     if (!controller) {
-        controller = new WheelController(document);
+        controller = new ScrollEventHandlerController(document);
         DocumentSupplement::provideTo(document, supplementName(), adoptPtr(controller));
     }
     return controller;
 }
 
-static void wheelEventHandlerCountChanged(Document& document)
+static void scrollEventHandlerCountChanged(Document& document)
 {
     Page* page = document.page();
     if (!page)
@@ -74,50 +73,40 @@ static void wheelEventHandlerCountChanged(Document& document)
     if (!frameView)
         return;
 
-    scrollingCoordinator->frameViewWheelEventHandlerCountChanged(frameView);
+    scrollingCoordinator->frameViewScrollEventHandlerCountChanged(frameView);
 }

[abarth-chromium, 2014/03/20 23:42:06]

So much copy/pasted code.  Can you extract a base class to share some of this
code?

 
-void WheelController::didAddWheelEventHandler(Document& document)
+void ScrollEventHandlerController::didAddScrollEventHandler(Document& document)
 {
-    ++m_wheelEventHandlerCount;
-    Page* page = document.page();
-    LocalFrame* mainFrame = page ? page->mainFrame() : 0;
-    if (mainFrame)
-        mainFrame->notifyChromeClientWheelEventHandlerCountChanged();
-
-    wheelEventHandlerCountChanged(document);
+    ++m_scrollEventHandlerCount;
+    scrollEventHandlerCountChanged(document);
 }
 
-void WheelController::didRemoveWheelEventHandler(Document& document)
+void ScrollEventHandlerController::didRemoveScrollEventHandler(Document& document)
 {
-    ASSERT(m_wheelEventHandlerCount > 0);
-    --m_wheelEventHandlerCount;
-    Page* page = document.page();
-    LocalFrame* mainFrame = page ? page->mainFrame() : 0;
-    if (mainFrame)
-        mainFrame->notifyChromeClientWheelEventHandlerCountChanged();
-
-    wheelEventHandlerCountChanged(document);
+    ASSERT(m_scrollEventHandlerCount > 0);
+    --m_scrollEventHandlerCount;
+    scrollEventHandlerCountChanged(document);
 }
 
-void WheelController::didAddEventListener(DOMWindow* window, const AtomicString& eventType)
+void ScrollEventHandlerController::didAddEventListener(DOMWindow* window, const AtomicString& eventType)
 {
-    if (eventType != EventTypeNames::wheel && eventType != EventTypeNames::mousewheel)
+    if (eventType != EventTypeNames::scroll)
         return;
 
     Document* document = window->document();
     ASSERT(document);
-    didAddWheelEventHandler(*document);
+    didAddScrollEventHandler(*document);
 }
 
-void WheelController::didRemoveEventListener(DOMWindow* window, const AtomicString& eventType)
+void ScrollEventHandlerController::didRemoveEventListener(DOMWindow* window, const AtomicString& eventType)
 {
-    if (eventType != EventTypeNames::wheel && eventType != EventTypeNames::mousewheel)
+    if (eventType != EventTypeNames::scroll)
         return;
 
     Document* document = window->document();
     ASSERT(document);
-    didRemoveWheelEventHandler(*document);
+    didRemoveScrollEventHandler(*document);
 }
 
 } // namespace WebCore