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

Side by Side Diff: third_party/WebKit/Source/web/WebViewFrameWidget.cpp

Issue 2850113002: Move WebViewFrameWidget to use WebViewBase rather than WebViewImpl. (Closed)
Patch Set: Fix code review comments. Created 3 years, 7 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 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 found 2 // Use of this source code is governed by a BSD-style license that can be found
3 // in the LICENSE file. 3 // in the LICENSE file.
4 4
5 #include "web/WebViewFrameWidget.h" 5 #include "web/WebViewFrameWidget.h"
6 6
7 #include "core/exported/WebViewBase.h"
8 #include "core/layout/HitTestResult.h"
7 #include "web/WebInputMethodControllerImpl.h" 9 #include "web/WebInputMethodControllerImpl.h"
8 #include "web/WebLocalFrameImpl.h" 10 #include "web/WebLocalFrameImpl.h"
9 #include "web/WebViewImpl.h"
10 11
11 namespace blink { 12 namespace blink {
12 13
13 WebViewFrameWidget::WebViewFrameWidget(WebWidgetClient& client, 14 WebViewFrameWidget::WebViewFrameWidget(WebWidgetClient& client,
14 WebViewImpl& web_view, 15 WebViewBase& web_view,
15 WebLocalFrameImpl& main_frame) 16 WebLocalFrameImpl& main_frame)
16 : client_(&client), web_view_(&web_view), main_frame_(&main_frame) { 17 : client_(&client), web_view_(&web_view), main_frame_(&main_frame) {
17 main_frame_->SetFrameWidget(this); 18 main_frame_->SetFrameWidget(this);
18 web_view_->SetCompositorVisibility(true); 19 web_view_->SetCompositorVisibility(true);
19 } 20 }
20 21
21 WebViewFrameWidget::~WebViewFrameWidget() {} 22 WebViewFrameWidget::~WebViewFrameWidget() {}
22 23
23 void WebViewFrameWidget::Close() { 24 void WebViewFrameWidget::Close() {
24 // Note: it's important to use the captured main frame pointer here. During 25 // Note: it's important to use the captured main frame pointer here. During
(...skipping 218 matching lines...) Expand 10 before | Expand all | Expand 10 after
243 244
244 CompositorAnimationHost* WebViewFrameWidget::AnimationHost() const { 245 CompositorAnimationHost* WebViewFrameWidget::AnimationHost() const {
245 return web_view_->AnimationHost(); 246 return web_view_->AnimationHost();
246 } 247 }
247 248
248 HitTestResult WebViewFrameWidget::CoreHitTestResultAt(const WebPoint& point) { 249 HitTestResult WebViewFrameWidget::CoreHitTestResultAt(const WebPoint& point) {
249 return web_view_->CoreHitTestResultAt(point); 250 return web_view_->CoreHitTestResultAt(point);
250 } 251 }
251 252
252 } // namespace blink 253 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/web/WebViewFrameWidget.h ('k') | third_party/WebKit/Source/web/WebViewImpl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698