Adding BlimpNavigationController to Tab

blimp/client/core/contents/blimp_navigation_controller_impl.cc

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 would have failed earlier.

[shaktisahu, 2016/08/18 19:36:06]

Actually this is also set in LoadURL() due to which it didn't fail. Updated the
test.

   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();
 }