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

Unified Diff: third_party/WebKit/Source/core/dom/ContextLifecycleNotifier.cpp

Issue 2366253002: Remove ActiveDOMObject::stop() (Closed)
Patch Set: temp Created 4 years, 2 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: third_party/WebKit/Source/core/dom/ContextLifecycleNotifier.cpp
diff --git a/third_party/WebKit/Source/core/dom/ContextLifecycleNotifier.cpp b/third_party/WebKit/Source/core/dom/ContextLifecycleNotifier.cpp
index 18c26a91cdd35df9500c70a63b7edd811e10388f..fbd88b6faf52a3c38bd8ae8f583ff08bdab8ca06 100644
--- a/third_party/WebKit/Source/core/dom/ContextLifecycleNotifier.cpp
+++ b/third_party/WebKit/Source/core/dom/ContextLifecycleNotifier.cpp
@@ -62,25 +62,6 @@ void ContextLifecycleNotifier::notifySuspendingActiveDOMObjects() {
}
}
-void ContextLifecycleNotifier::notifyStoppingActiveDOMObjects() {
- // Observers may be removed, but handled after iteration has completed.
- AutoReset<IterationState> scope(&m_iterationState, AllowPendingRemoval);
- ObserverSet observers;
- m_observers.swap(observers);
- for (ContextLifecycleObserver* observer : observers) {
- if (observer->observerType() !=
- ContextLifecycleObserver::ActiveDOMObjectType)
- continue;
- ActiveDOMObject* activeDOMObject = static_cast<ActiveDOMObject*>(observer);
-#if DCHECK_IS_ON()
- DCHECK_EQ(activeDOMObject->getExecutionContext(), context());
- DCHECK(activeDOMObject->suspendIfNeededCalled());
-#endif
- activeDOMObject->stop();
- }
- removePending(observers);
-}
-
unsigned ContextLifecycleNotifier::activeDOMObjectCount() const {
DCHECK(!isIteratingOverObservers());
unsigned activeDOMObjects = 0;

Powered by Google App Engine
This is Rietveld 408576698