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

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

Issue 2058263002: Tied up BlimpNavigationController to NavigationFeature (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@blimp_core
Patch Set: Removed BlimpContentsTest.java 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 37d25a9ad5d172435c2ef24eba5a15cf65596921..bc4047e3de5111b6e2945861cef87012e3239f81 100644
--- a/blimp/client/core/contents/blimp_contents_impl.cc
+++ b/blimp/client/core/contents/blimp_contents_impl.cc
@@ -22,7 +22,8 @@ const char kBlimpContentsImplAndroidKey[] = "blimp_contents_impl_android";
#endif // OS_ANDROID
}
-BlimpContentsImpl::BlimpContentsImpl() : navigation_controller_(this) {}
+BlimpContentsImpl::BlimpContentsImpl()
+ : navigation_controller_(this, nullptr) {}
BlimpContentsImpl::~BlimpContentsImpl() {}
@@ -58,8 +59,9 @@ void BlimpContentsImpl::RemoveObserver(BlimpContentsObserver* observer) {
observers_.RemoveObserver(observer);
}
-void BlimpContentsImpl::NotifyURLLoaded(const GURL& url) {
- FOR_EACH_OBSERVER(BlimpContentsObserver, observers_, OnURLUpdated(url));
+void BlimpContentsImpl::OnNavigationStateChanged() {
+ FOR_EACH_OBSERVER(BlimpContentsObserver, observers_,
+ OnNavigationStateChanged());
}
} // namespace client
« 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