Restrict number of BlimpContents to [0, 1]

blimp/client/core/contents/blimp_contents_manager.h

Index: blimp/client/core/contents/blimp_contents_manager.h
diff --git a/blimp/client/core/contents/blimp_contents_manager.h b/blimp/client/core/contents/blimp_contents_manager.h
index 9b58e76186bb9cd19145a2b6161ca04b2639c05f..2efe8013f559fe796575bcdc4f7c4fbc159eacf5 100644
--- a/blimp/client/core/contents/blimp_contents_manager.h
+++ b/blimp/client/core/contents/blimp_contents_manager.h
@@ -23,6 +23,8 @@ class BlimpContentsManager {
   ~BlimpContentsManager();
 
   // Builds a BlimpContentsImpl and notifies the engine.
+  // Currently we want to have a single BlimpContents. If there is an existing
+  // contents, return nullptr.
   std::unique_ptr<BlimpContentsImpl> CreateBlimpContents();
 
   // The caller can query the contents through its id.
@@ -55,6 +57,10 @@ class BlimpContentsManager {
   NavigationFeature* navigation_feature_;
   TabControlFeature* tab_control_feature_;
 
+  // TODO(mlliu): Currently we want to have a single BlimpContents. Remove this
+  // when it supports multiple tabs.
+  bool tab_exists_ = false;

[David Trainor- moved to gerrit, 2016/08/30 23:31:12]

Can we just check the map to see if any entries exist?

[Menglin, 2016/08/31 00:13:02]

There can be such case:
https://codesearch.chromium.org/chromium/src/blimp/client/core/contents/blimp...
, i.e., the contents is destroyed but the observer map is not empty yet. So I
figured it's more straightforward to just use a bool.

+
   base::WeakPtrFactory<BlimpContentsManager> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BlimpContentsManager);