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

Unified Diff: Source/core/dom/Document.h

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/dom/Document.h
diff --git a/Source/core/dom/Document.h b/Source/core/dom/Document.h
index 7ddecbc8dbb8e7e0893ef953079806d81ca7f370..742a27494523ced02a9f382e494126b6701abdba 100644
--- a/Source/core/dom/Document.h
+++ b/Source/core/dom/Document.h
@@ -193,8 +193,6 @@ enum NodeListInvalidationType {
};
const int numNodeListInvalidationTypes = InvalidateOnAnyAttrChange + 1;
-typedef HashCountedSet<Node*> TouchEventTargetSet;
-
enum DocumentClass {
DefaultDocumentClass = 0,
HTMLDocumentClass = 1,
@@ -945,21 +943,6 @@ public:
double lastHandledUserGestureTimestamp() const { return m_lastHandledUserGestureTimestamp; }
void resetLastHandledUserGestureTimestamp();
- bool hasTouchEventHandlers() const { return (m_touchEventTargets.get()) ? m_touchEventTargets->size() : false; }
-
- // Called when a single touch event handler has been added or removed for a node.
- // The Node should always be in this Document, except for child Documents which report
- // themselves to their parent exactly once if they have any touch handlers.
- // Handlers added/removed from the DOMWindow are reported as the Document.
- void didAddTouchEventHandler(Node*);
- void didRemoveTouchEventHandler(Node* handler) { didRemoveTouchEventHandler(handler, false); }
-
- // Called whenever all touch event handlers have been removed for a node (such as when the
- // node itself is being removed from the document).
- void didClearTouchEventHandlers(Node* handler) { didRemoveTouchEventHandler(handler, true); }
-
- const TouchEventTargetSet* touchEventTargets() const { return m_touchEventTargets.get(); }
-
bool isInDocumentWrite() { return m_writeRecursionDepth > 0; }
IntSize initialViewportSize() const;
@@ -1143,8 +1126,6 @@ private:
void processHttpEquivXFrameOptions(const AtomicString& content);
void processHttpEquivContentSecurityPolicy(const AtomicString& equiv, const AtomicString& content);
- void didRemoveTouchEventHandler(Node*, bool clearAll);
-
// Returns true if Document::recalcStyle() needs to be run.
bool shouldCallRecalcStyleForDocument();
bool shouldScheduleStyleRecalc();
@@ -1315,8 +1296,6 @@ private:
bool m_writeRecursionIsTooDeep;
unsigned m_writeRecursionDepth;
- OwnPtr<TouchEventTargetSet> m_touchEventTargets;
-
double m_lastHandledUserGestureTimestamp;
RefPtr<ScriptedAnimationController> m_scriptedAnimationController;

Powered by Google App Engine
This is Rietveld 408576698