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

Side by Side Diff: Source/core/dom/FullscreenElementStack.h

Issue 354403002: Expand addDocumentToFullScreenChangeEventQueue at the call sites for clarity (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 5 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * (C) 2001 Dirk Mueller (mueller@kde.org) 4 * (C) 2001 Dirk Mueller (mueller@kde.org)
5 * (C) 2006 Alexey Proskuryakov (ap@webkit.org) 5 * (C) 2006 Alexey Proskuryakov (ap@webkit.org)
6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All r ights reserved. 6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All r ights reserved.
7 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.t orchmobile.com/) 7 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.t orchmobile.com/)
8 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies) 8 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
9 * Copyright (C) 2013 Google Inc. All rights reserved. 9 * Copyright (C) 2013 Google Inc. All rights reserved.
10 * 10 *
(...skipping 87 matching lines...) Expand 10 before | Expand all | Expand 10 after
98 private: 98 private:
99 static FullscreenElementStack* fromIfExistsSlow(Document&); 99 static FullscreenElementStack* fromIfExistsSlow(Document&);
100 100
101 explicit FullscreenElementStack(Document&); 101 explicit FullscreenElementStack(Document&);
102 102
103 Document* document(); 103 Document* document();
104 104
105 void clearFullscreenElementStack(); 105 void clearFullscreenElementStack();
106 void popFullscreenElementStack(); 106 void popFullscreenElementStack();
107 void pushFullscreenElementStack(Element&); 107 void pushFullscreenElementStack(Element&);
108 void addDocumentToFullScreenChangeEventQueue(Document&); 108
109 void enqueueChangeEvent(Node&);
109 void fullScreenChangeDelayTimerFired(Timer<FullscreenElementStack>*); 110 void fullScreenChangeDelayTimerFired(Timer<FullscreenElementStack>*);
110 111
111 void fullScreenElementRemoved(); 112 void fullScreenElementRemoved();
112 113
113 bool m_areKeysEnabledInFullScreen; 114 bool m_areKeysEnabledInFullScreen;
114 RefPtrWillBeMember<Element> m_fullScreenElement; 115 RefPtrWillBeMember<Element> m_fullScreenElement;
115 WillBeHeapVector<RefPtrWillBeMember<Element> > m_fullScreenElementStack; 116 WillBeHeapVector<RefPtrWillBeMember<Element> > m_fullScreenElementStack;
116 RenderFullScreen* m_fullScreenRenderer; 117 RenderFullScreen* m_fullScreenRenderer;
117 Timer<FullscreenElementStack> m_fullScreenChangeDelayTimer; 118 Timer<FullscreenElementStack> m_fullScreenChangeDelayTimer;
118 WillBeHeapDeque<RefPtrWillBeMember<Node> > m_fullScreenChangeEventTargetQueu e; 119 WillBeHeapDeque<RefPtrWillBeMember<Node> > m_fullScreenChangeEventTargetQueu e;
(...skipping 13 matching lines...) Expand all
132 inline FullscreenElementStack* FullscreenElementStack::fromIfExists(Document& do cument) 133 inline FullscreenElementStack* FullscreenElementStack::fromIfExists(Document& do cument)
133 { 134 {
134 if (!document.hasFullscreenElementStack()) 135 if (!document.hasFullscreenElementStack())
135 return 0; 136 return 0;
136 return fromIfExistsSlow(document); 137 return fromIfExistsSlow(document);
137 } 138 }
138 139
139 } // namespace WebCore 140 } // namespace WebCore
140 141
141 #endif // FullscreenElementStack_h 142 #endif // FullscreenElementStack_h
OLDNEW
« no previous file with comments | « no previous file | Source/core/dom/FullscreenElementStack.cpp » ('j') | Source/core/dom/FullscreenElementStack.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698