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

Unified Diff: blimp/client/public/contents/blimp_contents_observer.h

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/public/contents/blimp_contents_observer.h
diff --git a/blimp/client/public/contents/blimp_contents_observer.h b/blimp/client/public/contents/blimp_contents_observer.h
index 39b600dbeef55de1a9d5283ba71aba2372949ee1..dc3e1ec892ab03b6ee1fe4453337a05608e487b9 100644
--- a/blimp/client/public/contents/blimp_contents_observer.h
+++ b/blimp/client/public/contents/blimp_contents_observer.h
@@ -17,8 +17,8 @@ class BlimpContentsObserver {
public:
virtual ~BlimpContentsObserver() = default;
- // Invoked when the URL of the BlimpContents is changes.
- virtual void OnURLUpdated(const GURL& url) {}
+ // Invoked when the navigation state of the BlimpContents has changed.
+ virtual void OnNavigationStateChanged() {}
protected:
BlimpContentsObserver() {}
« no previous file with comments | « blimp/client/public/contents/blimp_contents.h ('k') | blimp/client/public/contents/blimp_navigation_controller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698