Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(23)

Side by Side Diff: blimp/client/core/contents/blimp_contents_manager.h

Issue 2241623002: blimp: Move compositing, input and render widget feature to client/core. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressed comments from #7 Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef BLIMP_CLIENT_CORE_CONTENTS_BLIMP_CONTENTS_MANAGER_H_ 5 #ifndef BLIMP_CLIENT_CORE_CONTENTS_BLIMP_CONTENTS_MANAGER_H_
6 #define BLIMP_CLIENT_CORE_CONTENTS_BLIMP_CONTENTS_MANAGER_H_ 6 #define BLIMP_CLIENT_CORE_CONTENTS_BLIMP_CONTENTS_MANAGER_H_
7 7
8 #include "blimp/client/core/compositor/compositor_deps_provider.h"
8 #include "blimp/client/core/contents/blimp_contents_impl.h" 9 #include "blimp/client/core/contents/blimp_contents_impl.h"
9 10
10 namespace blimp { 11 namespace blimp {
11 namespace client { 12 namespace client {
13 class RenderWidgetFeature;
12 14
13 // BlimpContentsManager does the real work of creating BlimpContentsImpl, and 15 // BlimpContentsManager does the real work of creating BlimpContentsImpl, and
14 // then passes the ownership to the caller. It also owns the observers to 16 // then passes the ownership to the caller. It also owns the observers to
15 // monitor the life time of the contents it creates. 17 // monitor the life time of the contents it creates.
16 class BlimpContentsManager { 18 class BlimpContentsManager {
17 public: 19 public:
18 BlimpContentsManager(); 20 explicit BlimpContentsManager(RenderWidgetFeature* render_widget_feature);
19 ~BlimpContentsManager(); 21 ~BlimpContentsManager();
20 22
21 std::unique_ptr<BlimpContentsImpl> CreateBlimpContents(); 23 std::unique_ptr<BlimpContentsImpl> CreateBlimpContents();
22 24
23 // The caller can query the contents through its id. 25 // The caller can query the contents through its id.
24 BlimpContentsImpl* GetBlimpContents(int id); 26 BlimpContentsImpl* GetBlimpContents(int id);
25 27
26 private: 28 private:
27 class BlimpContentsDeletionObserver; 29 class BlimpContentsDeletionObserver;
28 friend class BlimpContentsDeletionObserver; 30 friend class BlimpContentsDeletionObserver;
29 31
30 // When creating the BlimpContentsImpl, an id is created for the content. 32 // When creating the BlimpContentsImpl, an id is created for the content.
31 int CreateBlimpContentsId(); 33 int CreateBlimpContentsId();
32 34
33 // When a BlimpContentsImpl is destroyed, its observer is able to retrieve the 35 // When a BlimpContentsImpl is destroyed, its observer is able to retrieve the
34 // id, and use this to destroy the observer entry from the observer_map_. 36 // id, and use this to destroy the observer entry from the observer_map_.
35 void EraseObserverFromMap(int id); 37 void EraseObserverFromMap(int id);
36 base::WeakPtr<BlimpContentsManager> GetWeakPtr(); 38 base::WeakPtr<BlimpContentsManager> GetWeakPtr();
37 39
38 // BlimpContentsManager owns the BlimpContentsDeletionObserver for the 40 // BlimpContentsManager owns the BlimpContentsDeletionObserver for the
39 // contents it creates, with the content id being the key to help manage the 41 // contents it creates, with the content id being the key to help manage the
40 // lifetime of the observers. 42 // lifetime of the observers.
41 std::map<int, std::unique_ptr<BlimpContentsDeletionObserver>> observer_map_; 43 std::map<int, std::unique_ptr<BlimpContentsDeletionObserver>> observer_map_;
42 44
45 // The network bridge for a BlimpContents to the WebContents on the engine.
46 RenderWidgetFeature* render_widget_feature_;
47
43 base::WeakPtrFactory<BlimpContentsManager> weak_ptr_factory_; 48 base::WeakPtrFactory<BlimpContentsManager> weak_ptr_factory_;
44 49
45 DISALLOW_COPY_AND_ASSIGN(BlimpContentsManager); 50 DISALLOW_COPY_AND_ASSIGN(BlimpContentsManager);
46 }; 51 };
47 52
48 } // namespace client 53 } // namespace client
49 } // namespace blimp 54 } // namespace blimp
50 55
51 #endif // BLIMP_CLIENT_CORE_CONTENTS_BLIMP_CONTENTS_MANAGER_H_ 56 #endif // BLIMP_CLIENT_CORE_CONTENTS_BLIMP_CONTENTS_MANAGER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698