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

Unified Diff: Source/core/dom/ContextLifecycleObserver.cpp

Issue 968633002: Simplify lifecycle notifiers and observers. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 10 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/ContextLifecycleObserver.cpp
diff --git a/Source/core/dom/ContextLifecycleObserver.cpp b/Source/core/dom/ContextLifecycleObserver.cpp
index 8f158361e75697254cb1e5646ecff63f64ce6c5d..922fc9215476dbabac05f977ccf76348352c2d24 100644
--- a/Source/core/dom/ContextLifecycleObserver.cpp
+++ b/Source/core/dom/ContextLifecycleObserver.cpp
@@ -32,19 +32,11 @@
namespace blink {
-template<> void observeContext(ExecutionContext* context, LifecycleObserver<ExecutionContext>* observer)
-{
- context->addObserver(observer);
-}
-
-template<> void unobserveContext(ExecutionContext* context, LifecycleObserver<ExecutionContext>* observer)
-{
- context->removeObserver(observer);
-}
-
ContextLifecycleObserver::ContextLifecycleObserver(ExecutionContext* executionContext, Type type)
- : LifecycleObserver<ExecutionContext>(executionContext, type)
+ : LifecycleObserver<ExecutionContext, ContextLifecycleObserver, ContextLifecycleNotifier>(executionContext)
+ , m_observerType(type)
{
+ setContext(executionContext);
}
ContextLifecycleObserver::~ContextLifecycleObserver()

Powered by Google App Engine
This is Rietveld 408576698