Index: blimp/client/core/contents/blimp_navigation_controller_impl.cc |
diff --git a/blimp/client/core/contents/blimp_navigation_controller_impl.cc b/blimp/client/core/contents/blimp_navigation_controller_impl.cc |
index c8209610b823ef926fba44bdc27325f39766f2e0..65492ee68b90674a540ca8eae3c0cdcfb5852503 100644 |
--- a/blimp/client/core/contents/blimp_navigation_controller_impl.cc |
+++ b/blimp/client/core/contents/blimp_navigation_controller_impl.cc |
@@ -42,12 +42,12 @@ void BlimpNavigationControllerImpl::Reload() { |
bool BlimpNavigationControllerImpl::CanGoBack() const { |
NOTIMPLEMENTED(); |
- return false; |
+ return true; |
} |
bool BlimpNavigationControllerImpl::CanGoForward() const { |
NOTIMPLEMENTED(); |
- return false; |
+ return true; |
} |
void BlimpNavigationControllerImpl::GoBack() { |
@@ -62,7 +62,12 @@ const GURL& BlimpNavigationControllerImpl::GetURL() { |
return current_url_; |
} |
+const std::string& BlimpNavigationControllerImpl::GetTitle() { |
+ return current_title_; |
+} |
+ |
void BlimpNavigationControllerImpl::OnUrlChanged(int tab_id, const GURL& url) { |
+ current_url_ = url; |
David Trainor- moved to gerrit
2016/08/18 17:06:30
Do we have unit tests for this? It looks like it
shaktisahu
2016/08/18 19:36:06
Actually this is also set in LoadURL() due to whic
|
delegate_->OnNavigationStateChanged(); |
} |
@@ -73,6 +78,7 @@ void BlimpNavigationControllerImpl::OnFaviconChanged(int tab_id, |
void BlimpNavigationControllerImpl::OnTitleChanged(int tab_id, |
const std::string& title) { |
+ current_title_ = title; |
delegate_->OnNavigationStateChanged(); |
} |