Migrate TabControlFeature from 0.5 to 0.6

blimp/client/core/contents/blimp_contents_manager.h

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef BLIMP_CLIENT_CORE_CONTENTS_BLIMP_CONTENTS_MANAGER_H_
+#define BLIMP_CLIENT_CORE_CONTENTS_BLIMP_CONTENTS_MANAGER_H_
+
+#include "blimp/client/core/contents/blimp_contents_impl.h"
+
+namespace blimp {
+namespace client {
+
+class BlimpContentsManager {

[nyquist, 2016/08/05 23:03:23]

It would be great with a high level description of this class.

[Menglin, 2016/08/06 00:57:00]

Done

+ public:
+  BlimpContentsManager();
+  ~BlimpContentsManager();
+
+  std::unique_ptr<BlimpContentsImpl> CreateBlimpContents();

[nyquist, 2016/08/05 23:03:23]

It's probably helpful to add a comment for each of these public methods.

[Menglin, 2016/08/06 00:57:00]

Given I added high level description for the class. I think it's obvious what
CreateBlimpContents() does and so save the comment for it

+  BlimpContentsImpl* GetBlimpContents(int id);
+
+ private:
+  class BlimpContentsDeletionObserver;
+  friend class BlimpContentsDeletionObserver;
+
+  int CreateBlimpContentsId();

[nyquist, 2016/08/05 23:03:23]

I think it's helpful to in general document what these methods do, even though
they're private.

[Menglin, 2016/08/06 00:57:00]

Done.

+  void EraseObserverFromMap(int id);
+  base::WeakPtr<BlimpContentsManager> GetWeakPtr();
+
+  std::map<int, std::unique_ptr<BlimpContentsDeletionObserver>> observer_map_;

[nyquist, 2016/08/05 23:03:23]

Could you explain what this does?

[Menglin, 2016/08/06 00:57:00]

Done.

+
+  base::WeakPtrFactory<BlimpContentsManager> weak_ptr_factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(BlimpContentsManager);
+};
+
+}  // namespace client
+}  // namespace blimp
+
+#endif  // BLIMP_CLIENT_CORE_CONTENTS_BLIMP_CONTENTS_MANAGER_H_