Add TabControlFeature to BlimpClientContextImpl

blimp/client/core/contents/blimp_contents_manager.cc

Index: blimp/client/core/contents/blimp_contents_manager.cc
diff --git a/blimp/client/core/contents/blimp_contents_manager.cc b/blimp/client/core/contents/blimp_contents_manager.cc
index f0a1f6bf560da4167512eb7a99ba49eced4195f3..edaf4937f84a0169505b7a10c5df0158b8c5b2a0 100644
--- a/blimp/client/core/contents/blimp_contents_manager.cc
+++ b/blimp/client/core/contents/blimp_contents_manager.cc
@@ -7,6 +7,7 @@
 #include "base/bind.h"
 #include "base/memory/ptr_util.h"
 #include "base/threading/thread_task_runner_handle.h"
+#include "blimp/client/core/contents/tab_control_feature.h"
 #include "blimp/client/public/contents/blimp_contents_observer.h"
 
 namespace {
@@ -40,26 +41,32 @@ BlimpContentsManager::BlimpContentsDeletionObserver::
 void BlimpContentsManager::BlimpContentsDeletionObserver::
     OnContentsDestroyed() {
   BlimpContents* contents = blimp_contents();
-  int id = static_cast<BlimpContentsImpl*>(contents)->id();
-  DCHECK(base::ThreadTaskRunnerHandle::Get());
-  base::ThreadTaskRunnerHandle::Get()->PostTask(
-      FROM_HERE, base::Bind(&BlimpContentsManager::EraseObserverFromMap,
-                            blimp_contents_manager_->GetWeakPtr(), id));
+  blimp_contents_manager_->OnContentsDestroyed(contents);
 }
 
-BlimpContentsManager::BlimpContentsManager() : weak_ptr_factory_(this) {}
+BlimpContentsManager::BlimpContentsManager(
+    TabControlFeature* tab_control_feature)
+    : tab_control_feature_(tab_control_feature),
+      weak_ptr_factory_(this) {}
 
 BlimpContentsManager::~BlimpContentsManager() {}
 
 std::unique_ptr<BlimpContentsImpl> BlimpContentsManager::CreateBlimpContents() {
   int id = CreateBlimpContentsId();
+
   std::unique_ptr<BlimpContentsImpl> new_contents =
-      base::MakeUnique<BlimpContentsImpl>(id);
+      base::MakeUnique<BlimpContentsImpl>(id, tab_control_feature_);
+
+  // Create an observer entry for the contents

[David Trainor- moved to gerrit, 2016/08/19 20:55:05]

. at end

[Menglin, 2016/08/19 21:33:02]

Done.

   std::unique_ptr<BlimpContentsDeletionObserver> observer =
       base::MakeUnique<BlimpContentsDeletionObserver>(this, new_contents.get());
   observer_map_.insert(
       std::pair<int, std::unique_ptr<BlimpContentsDeletionObserver>>(
           id, std::move(observer)));
+
+  // Creates a tab for the contents.

[David Trainor- moved to gerrit, 2016/08/19 20:55:05]

Maybe: Notifies the engine that we've created a new BlimpContents

[Menglin, 2016/08/19 21:33:02]

Done.

+  tab_control_feature_->CreateTab(id);
+
   return new_contents;
 }
 
@@ -87,6 +94,19 @@ void BlimpContentsManager::EraseObserverFromMap(int id) {
   observer_map_.erase(id);
 }
 
+void BlimpContentsManager::OnContentsDestroyed(BlimpContents* contents) {
+  int id = static_cast<BlimpContentsImpl*>(contents)->id();
+
+  // Close the contents' tab

[David Trainor- moved to gerrit, 2016/08/19 20:55:05]

Maybe: Notify the engine that we've destroyed the BlimpContents.

[Menglin, 2016/08/19 21:33:02]

Done.

+  tab_control_feature_->CloseTab(id);
+
+  // Destroy the observer entry from the observer_map_.
+  DCHECK(base::ThreadTaskRunnerHandle::Get());
+  base::ThreadTaskRunnerHandle::Get()->PostTask(
+      FROM_HERE, base::Bind(&BlimpContentsManager::EraseObserverFromMap,
+                            this->GetWeakPtr(), id));
+}
+
 base::WeakPtr<BlimpContentsManager> BlimpContentsManager::GetWeakPtr() {
   return weak_ptr_factory_.GetWeakPtr();
 }