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

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

Issue 206603002: Add EventHandlerRegistry (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: More comments. 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 e173e7fb3dbd6bd8ef2c4b11ca1e34a2bb3343f4..f1a4399d3bb139bdd7bbaf9b5cb54247abe01174 100644
--- a/Source/core/dom/Document.h
+++ b/Source/core/dom/Document.h
@@ -194,8 +194,6 @@ enum NodeListInvalidationType {
};
const int numNodeListInvalidationTypes = InvalidateOnAnyAttrChange + 1;
-typedef HashCountedSet<Node*> TouchEventTargetSet;
-
enum DocumentClass {
DefaultDocumentClass = 0,
HTMLDocumentClass = 1,
@@ -953,20 +951,8 @@ public:
void initDNSPrefetch();
- 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(); }
+ double lastHandledUserGestureTimestamp() const { return m_lastHandledUserGestureTimestamp; }
Rick Byers 2014/04/04 20:12:27 Looks like you added this back (after it was remov
Sami 2014/04/07 14:56:49 Oh, thanks for pointing this out. I was working re
+ void resetLastHandledUserGestureTimestamp();
bool isInDocumentWrite() { return m_writeRecursionDepth > 0; }
@@ -1154,8 +1140,6 @@ private:
void processHttpEquivXFrameOptions(const AtomicString& content);
void processHttpEquivContentSecurityPolicy(const AtomicString& equiv, const AtomicString& content);
- void didRemoveTouchEventHandler(Node*, bool clearAll);
-
bool needsRenderTreeUpdate() const;
bool shouldScheduleRenderTreeUpdate() const;
@@ -1329,7 +1313,7 @@ private:
bool m_writeRecursionIsTooDeep;
unsigned m_writeRecursionDepth;
- OwnPtr<TouchEventTargetSet> m_touchEventTargets;
+ double m_lastHandledUserGestureTimestamp;
Rick Byers 2014/04/04 20:12:27 ditto
Sami 2014/04/07 14:56:49 Done.
RefPtr<ScriptedAnimationController> m_scriptedAnimationController;
OwnPtr<MainThreadTaskRunner> m_taskRunner;

Powered by Google App Engine
This is Rietveld 408576698