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

Unified Diff: sky/engine/core/frame/EventHandlerRegistry.cpp

Issue 713073003: remove updateTouchEventTargetRectsIfNeeded (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: moar Created 6 years, 1 month 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: sky/engine/core/frame/EventHandlerRegistry.cpp
diff --git a/sky/engine/core/frame/EventHandlerRegistry.cpp b/sky/engine/core/frame/EventHandlerRegistry.cpp
deleted file mode 100644
index 9857b17bd262fa26b0365e6b3208f06978884208..0000000000000000000000000000000000000000
--- a/sky/engine/core/frame/EventHandlerRegistry.cpp
+++ /dev/null
@@ -1,240 +0,0 @@
-// Copyright 2014 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "config.h"
-#include "core/frame/EventHandlerRegistry.h"
-
-#include "core/events/ThreadLocalEventNames.h"
-#include "core/frame/LocalDOMWindow.h"
-#include "core/frame/LocalFrame.h"
-#include "core/page/Chrome.h"
-#include "core/page/ChromeClient.h"
-#include "core/page/Page.h"
-#include "core/page/scrolling/ScrollingCoordinator.h"
-
-namespace blink {
-
-EventHandlerRegistry::EventHandlerRegistry(FrameHost& frameHost)
- : m_frameHost(frameHost)
-{
-}
-
-EventHandlerRegistry::~EventHandlerRegistry()
-{
- checkConsistency();
-}
-
-bool EventHandlerRegistry::eventTypeToClass(const AtomicString& eventType, EventHandlerClass* result)
-{
- if (eventType == EventTypeNames::scroll) {
- *result = ScrollEvent;
- } else if (eventType == EventTypeNames::wheel || eventType == EventTypeNames::mousewheel) {
- *result = WheelEvent;
- } else if (isTouchEventType(eventType)) {
- *result = TouchEvent;
-#if ENABLE(ASSERT)
- } else if (eventType == EventTypeNames::load || eventType == EventTypeNames::mousemove || eventType == EventTypeNames::touchstart) {
- *result = EventsForTesting;
-#endif
- } else {
- return false;
- }
- return true;
-}
-
-const EventTargetSet* EventHandlerRegistry::eventHandlerTargets(EventHandlerClass handlerClass) const
-{
- checkConsistency();
- return &m_targets[handlerClass];
-}
-
-bool EventHandlerRegistry::hasEventHandlers(EventHandlerClass handlerClass) const
-{
- checkConsistency();
- return m_targets[handlerClass].size();
-}
-
-bool EventHandlerRegistry::updateEventHandlerTargets(ChangeOperation op, EventHandlerClass handlerClass, EventTarget* target)
-{
- EventTargetSet* targets = &m_targets[handlerClass];
- if (op == Add) {
- if (!targets->add(target).isNewEntry) {
- // Just incremented refcount, no real change.
- return false;
- }
- } else {
- ASSERT(op == Remove || op == RemoveAll);
- ASSERT(op == RemoveAll || targets->contains(target));
-
- if (op == RemoveAll) {
- if (!targets->contains(target))
- return false;
- targets->removeAll(target);
- } else {
- if (!targets->remove(target)) {
- // Just decremented refcount, no real update.
- return false;
- }
- }
- }
- return true;
-}
-
-void EventHandlerRegistry::updateEventHandlerInternal(ChangeOperation op, EventHandlerClass handlerClass, EventTarget* target)
-{
- bool hadHandlers = m_targets[handlerClass].size();
- bool targetSetChanged = updateEventHandlerTargets(op, handlerClass, target);
- bool hasHandlers = m_targets[handlerClass].size();
-
- if (hadHandlers != hasHandlers)
- notifyHasHandlersChanged(handlerClass, hasHandlers);
-
- if (targetSetChanged)
- notifyDidAddOrRemoveEventHandlerTarget(handlerClass);
-}
-
-void EventHandlerRegistry::updateEventHandlerOfType(ChangeOperation op, const AtomicString& eventType, EventTarget* target)
-{
- EventHandlerClass handlerClass;
- if (!eventTypeToClass(eventType, &handlerClass))
- return;
- updateEventHandlerInternal(op, handlerClass, target);
-}
-
-void EventHandlerRegistry::didAddEventHandler(EventTarget& target, const AtomicString& eventType)
-{
- updateEventHandlerOfType(Add, eventType, &target);
-}
-
-void EventHandlerRegistry::didRemoveEventHandler(EventTarget& target, const AtomicString& eventType)
-{
- updateEventHandlerOfType(Remove, eventType, &target);
-}
-
-void EventHandlerRegistry::didAddEventHandler(EventTarget& target, EventHandlerClass handlerClass)
-{
- updateEventHandlerInternal(Add, handlerClass, &target);
-}
-
-void EventHandlerRegistry::didRemoveEventHandler(EventTarget& target, EventHandlerClass handlerClass)
-{
- updateEventHandlerInternal(Remove, handlerClass, &target);
-}
-
-void EventHandlerRegistry::didMoveIntoFrameHost(EventTarget& target)
-{
- if (!target.hasEventListeners())
- return;
-
- Vector<AtomicString> eventTypes = target.eventTypes();
- for (size_t i = 0; i < eventTypes.size(); ++i) {
- EventHandlerClass handlerClass;
- if (!eventTypeToClass(eventTypes[i], &handlerClass))
- continue;
- for (unsigned count = target.getEventListeners(eventTypes[i]).size(); count > 0; --count)
- didAddEventHandler(target, handlerClass);
- }
-}
-
-void EventHandlerRegistry::didMoveOutOfFrameHost(EventTarget& target)
-{
- didRemoveAllEventHandlers(target);
-}
-
-void EventHandlerRegistry::didMoveBetweenFrameHosts(EventTarget& target, FrameHost* oldFrameHost, FrameHost* newFrameHost)
-{
- ASSERT(newFrameHost != oldFrameHost);
- for (size_t i = 0; i < EventHandlerClassCount; ++i) {
- EventHandlerClass handlerClass = static_cast<EventHandlerClass>(i);
- const EventTargetSet* targets = &oldFrameHost->eventHandlerRegistry().m_targets[handlerClass];
- for (unsigned count = targets->count(&target); count > 0; --count)
- newFrameHost->eventHandlerRegistry().didAddEventHandler(target, handlerClass);
- oldFrameHost->eventHandlerRegistry().didRemoveAllEventHandlers(target);
- }
-}
-
-void EventHandlerRegistry::didRemoveAllEventHandlers(EventTarget& target)
-{
- for (size_t i = 0; i < EventHandlerClassCount; ++i) {
- EventHandlerClass handlerClass = static_cast<EventHandlerClass>(i);
- updateEventHandlerInternal(RemoveAll, handlerClass, &target);
- }
-}
-
-void EventHandlerRegistry::notifyHasHandlersChanged(EventHandlerClass handlerClass, bool hasActiveHandlers)
-{
- switch (handlerClass) {
- // FIXME(sky): Remove these enums from the EventHandlerClass entirely.
- case ScrollEvent:
- case WheelEvent:
- break;
- case TouchEvent:
- m_frameHost.chrome().client().needTouchEvents(hasActiveHandlers);
- break;
-#if ENABLE(ASSERT)
- case EventsForTesting:
- break;
-#endif
- default:
- ASSERT_NOT_REACHED();
- break;
- }
-}
-
-void EventHandlerRegistry::notifyDidAddOrRemoveEventHandlerTarget(EventHandlerClass handlerClass)
-{
- ScrollingCoordinator* scrollingCoordinator = m_frameHost.page().scrollingCoordinator();
- if (scrollingCoordinator && handlerClass == TouchEvent)
- scrollingCoordinator->touchEventTargetRectsDidChange();
-}
-
-void EventHandlerRegistry::documentDetached(Document& document)
-{
- // Remove all event targets under the detached document.
- for (size_t handlerClassIndex = 0; handlerClassIndex < EventHandlerClassCount; ++handlerClassIndex) {
- EventHandlerClass handlerClass = static_cast<EventHandlerClass>(handlerClassIndex);
- Vector<EventTarget*> targetsToRemove;
- const EventTargetSet* targets = &m_targets[handlerClass];
- for (EventTargetSet::const_iterator iter = targets->begin(); iter != targets->end(); ++iter) {
- if (Node* node = iter->key->toNode()) {
- if (node->document() == &document) {
- targetsToRemove.append(iter->key);
- break;
- }
- } else if (iter->key->toDOMWindow()) {
- // DOMWindows may outlive their documents, so we shouldn't remove their handlers
- // here.
- } else {
- ASSERT_NOT_REACHED();
- }
- }
- for (size_t i = 0; i < targetsToRemove.size(); ++i)
- updateEventHandlerInternal(RemoveAll, handlerClass, targetsToRemove[i]);
- }
-}
-
-void EventHandlerRegistry::checkConsistency() const
-{
-#if ENABLE(ASSERT)
- for (size_t i = 0; i < EventHandlerClassCount; ++i) {
- EventHandlerClass handlerClass = static_cast<EventHandlerClass>(i);
- const EventTargetSet* targets = &m_targets[handlerClass];
- for (EventTargetSet::const_iterator iter = targets->begin(); iter != targets->end(); ++iter) {
- if (Node* node = iter->key->toNode()) {
- // See the comment for |documentDetached| if either of these assertions fails.
- ASSERT(node->document().frameHost());
- ASSERT(node->document().frameHost() == &m_frameHost);
- } else if (LocalDOMWindow* window = iter->key->toDOMWindow()) {
- // If any of these assertions fail, LocalDOMWindow failed to unregister its handlers
- // properly.
- ASSERT(window->frame());
- ASSERT(window->frame()->host());
- ASSERT(window->frame()->host() == &m_frameHost);
- }
- }
- }
-#endif // ENABLE(ASSERT)
-}
-
-} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698