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

Unified Diff: blimp/client/core/contents/blimp_contents_impl.cc

Issue 2201433002: Migrate TabControlFeature from 0.5 to 0.6 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: some more compile error fix Created 4 years, 4 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: blimp/client/core/contents/blimp_contents_impl.cc
diff --git a/blimp/client/core/contents/blimp_contents_impl.cc b/blimp/client/core/contents/blimp_contents_impl.cc
index bc4047e3de5111b6e2945861cef87012e3239f81..bfbbca356113bdd5f4e70743840be803cef4daf0 100644
--- a/blimp/client/core/contents/blimp_contents_impl.cc
+++ b/blimp/client/core/contents/blimp_contents_impl.cc
@@ -22,10 +22,12 @@ const char kBlimpContentsImplAndroidKey[] = "blimp_contents_impl_android";
#endif // OS_ANDROID
}
-BlimpContentsImpl::BlimpContentsImpl()
- : navigation_controller_(this, nullptr) {}
+BlimpContentsImpl::BlimpContentsImpl(int id)
+ : navigation_controller_(this, nullptr), id_(id) {}
-BlimpContentsImpl::~BlimpContentsImpl() {}
+BlimpContentsImpl::~BlimpContentsImpl() {
+ FOR_EACH_OBSERVER(BlimpContentsObserver, observers_, BlimpContentsDying());
+}
#if defined(OS_ANDROID)
@@ -59,6 +61,10 @@ void BlimpContentsImpl::RemoveObserver(BlimpContentsObserver* observer) {
observers_.RemoveObserver(observer);
}
+bool BlimpContentsImpl::HasObserver(BlimpContentsObserver* observer) {
+ return observers_.HasObserver(observer);
+}
+
void BlimpContentsImpl::OnNavigationStateChanged() {
FOR_EACH_OBSERVER(BlimpContentsObserver, observers_,
OnNavigationStateChanged());
« no previous file with comments | « blimp/client/core/contents/blimp_contents_impl.h ('k') | blimp/client/core/contents/blimp_contents_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698