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

Side by Side Diff: components/html_viewer/html_widget.h

Issue 1674903003: Extract shell methods from ApplicationImpl into a base class, and pass this to Initialize() instead. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@mojom
Patch Set: . Created 4 years, 10 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
« no previous file with comments | « components/html_viewer/html_viewer.cc ('k') | components/html_viewer/html_widget.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 COMPONENTS_HTML_VIEWER_HTML_WIDGET_H_ 5 #ifndef COMPONENTS_HTML_VIEWER_HTML_WIDGET_H_
6 #define COMPONENTS_HTML_VIEWER_HTML_WIDGET_H_ 6 #define COMPONENTS_HTML_VIEWER_HTML_WIDGET_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "third_party/WebKit/public/web/WebViewClient.h" 9 #include "third_party/WebKit/public/web/WebViewClient.h"
10 #include "third_party/WebKit/public/web/WebWidgetClient.h" 10 #include "third_party/WebKit/public/web/WebWidgetClient.h"
11 11
12 namespace blink { 12 namespace blink {
13 class WebFrameWidget; 13 class WebFrameWidget;
14 } 14 }
15 15
16 namespace mojo { 16 namespace mojo {
17 class ApplicationImpl; 17 class Shell;
18 } 18 }
19 19
20 namespace mus { 20 namespace mus {
21 class Window; 21 class Window;
22 } 22 }
23 23
24 namespace html_viewer { 24 namespace html_viewer {
25 25
26 class GlobalState; 26 class GlobalState;
27 class ImeController; 27 class ImeController;
(...skipping 28 matching lines...) Expand all
56 blink::WebView* web_view_; 56 blink::WebView* web_view_;
57 57
58 DISALLOW_COPY_AND_ASSIGN(HTMLWidgetRootRemote); 58 DISALLOW_COPY_AND_ASSIGN(HTMLWidgetRootRemote);
59 }; 59 };
60 60
61 // Used for the root frame when the frame is local. If there is only one 61 // Used for the root frame when the frame is local. If there is only one
62 // frame in the document, this is the HTMLWidget type created. 62 // frame in the document, this is the HTMLWidget type created.
63 class HTMLWidgetRootLocal : public HTMLWidget, public blink::WebViewClient { 63 class HTMLWidgetRootLocal : public HTMLWidget, public blink::WebViewClient {
64 public: 64 public:
65 struct CreateParams { 65 struct CreateParams {
66 CreateParams(mojo::ApplicationImpl* app, 66 CreateParams(mojo::Shell* shell,
67 GlobalState* global_state, 67 GlobalState* global_state,
68 mus::Window* window); 68 mus::Window* window);
69 ~CreateParams(); 69 ~CreateParams();
70 70
71 mojo::ApplicationImpl* app; 71 mojo::Shell* shell;
72 GlobalState* global_state; 72 GlobalState* global_state;
73 mus::Window* window; 73 mus::Window* window;
74 }; 74 };
75 75
76 HTMLWidgetRootLocal(CreateParams* create_params); 76 HTMLWidgetRootLocal(CreateParams* create_params);
77 ~HTMLWidgetRootLocal() override; 77 ~HTMLWidgetRootLocal() override;
78 78
79 blink::WebView* web_view() { return web_view_; } 79 blink::WebView* web_view() { return web_view_; }
80 80
81 protected: 81 protected:
82 // WebViewClient methods: 82 // WebViewClient methods:
83 blink::WebStorageNamespace* createSessionStorageNamespace() override; 83 blink::WebStorageNamespace* createSessionStorageNamespace() override;
84 void initializeLayerTreeView() override; 84 void initializeLayerTreeView() override;
85 blink::WebLayerTreeView* layerTreeView() override; 85 blink::WebLayerTreeView* layerTreeView() override;
86 void didMeaningfulLayout(blink::WebMeaningfulLayout layout_type) override; 86 void didMeaningfulLayout(blink::WebMeaningfulLayout layout_type) override;
87 void resetInputMethod() override; 87 void resetInputMethod() override;
88 void didHandleGestureEvent(const blink::WebGestureEvent& event, 88 void didHandleGestureEvent(const blink::WebGestureEvent& event,
89 bool event_cancelled) override; 89 bool event_cancelled) override;
90 void didUpdateTextOfFocusedElementByNonUserInput() override; 90 void didUpdateTextOfFocusedElementByNonUserInput() override;
91 void showImeIfNeeded() override; 91 void showImeIfNeeded() override;
92 92
93 private: 93 private:
94 // HTMLWidget: 94 // HTMLWidget:
95 blink::WebWidget* GetWidget() override; 95 blink::WebWidget* GetWidget() override;
96 void OnWindowBoundsChanged(mus::Window* window) override; 96 void OnWindowBoundsChanged(mus::Window* window) override;
97 97
98 mojo::ApplicationImpl* app_; 98 mojo::Shell* shell_;
99 GlobalState* global_state_; 99 GlobalState* global_state_;
100 mus::Window* window_; 100 mus::Window* window_;
101 blink::WebView* web_view_; 101 blink::WebView* web_view_;
102 scoped_ptr<WebLayerTreeViewImpl> web_layer_tree_view_impl_; 102 scoped_ptr<WebLayerTreeViewImpl> web_layer_tree_view_impl_;
103 scoped_ptr<ImeController> ime_controller_; 103 scoped_ptr<ImeController> ime_controller_;
104 104
105 DISALLOW_COPY_AND_ASSIGN(HTMLWidgetRootLocal); 105 DISALLOW_COPY_AND_ASSIGN(HTMLWidgetRootLocal);
106 }; 106 };
107 107
108 // Used for frames other than the root that are local. 108 // Used for frames other than the root that are local.
109 class HTMLWidgetLocalRoot : public HTMLWidget, public blink::WebWidgetClient { 109 class HTMLWidgetLocalRoot : public HTMLWidget, public blink::WebWidgetClient {
110 public: 110 public:
111 HTMLWidgetLocalRoot(mojo::ApplicationImpl* app, 111 HTMLWidgetLocalRoot(mojo::Shell* shell,
112 GlobalState* global_state, 112 GlobalState* global_state,
113 mus::Window* window, 113 mus::Window* window,
114 blink::WebLocalFrame* web_local_frame); 114 blink::WebLocalFrame* web_local_frame);
115 ~HTMLWidgetLocalRoot() override; 115 ~HTMLWidgetLocalRoot() override;
116 116
117 private: 117 private:
118 // HTMLWidget: 118 // HTMLWidget:
119 blink::WebWidget* GetWidget() override; 119 blink::WebWidget* GetWidget() override;
120 void OnWindowBoundsChanged(mus::Window* window) override; 120 void OnWindowBoundsChanged(mus::Window* window) override;
121 121
122 // WebWidgetClient: 122 // WebWidgetClient:
123 void initializeLayerTreeView() override; 123 void initializeLayerTreeView() override;
124 blink::WebLayerTreeView* layerTreeView() override; 124 blink::WebLayerTreeView* layerTreeView() override;
125 void resetInputMethod() override; 125 void resetInputMethod() override;
126 void didHandleGestureEvent(const blink::WebGestureEvent& event, 126 void didHandleGestureEvent(const blink::WebGestureEvent& event,
127 bool event_cancelled) override; 127 bool event_cancelled) override;
128 void didUpdateTextOfFocusedElementByNonUserInput() override; 128 void didUpdateTextOfFocusedElementByNonUserInput() override;
129 void showImeIfNeeded() override; 129 void showImeIfNeeded() override;
130 130
131 mojo::ApplicationImpl* app_; 131 mojo::Shell* shell_;
132 GlobalState* global_state_; 132 GlobalState* global_state_;
133 blink::WebFrameWidget* web_frame_widget_; 133 blink::WebFrameWidget* web_frame_widget_;
134 scoped_ptr<WebLayerTreeViewImpl> web_layer_tree_view_impl_; 134 scoped_ptr<WebLayerTreeViewImpl> web_layer_tree_view_impl_;
135 scoped_ptr<ImeController> ime_controller_; 135 scoped_ptr<ImeController> ime_controller_;
136 136
137 DISALLOW_COPY_AND_ASSIGN(HTMLWidgetLocalRoot); 137 DISALLOW_COPY_AND_ASSIGN(HTMLWidgetLocalRoot);
138 }; 138 };
139 139
140 } // namespace html_viewer 140 } // namespace html_viewer
141 141
142 #endif // COMPONENTS_HTML_VIEWER_HTML_WIDGET_H_ 142 #endif // COMPONENTS_HTML_VIEWER_HTML_WIDGET_H_
OLDNEW
« no previous file with comments | « components/html_viewer/html_viewer.cc ('k') | components/html_viewer/html_widget.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698