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

Side by Side Diff: content/renderer/render_widget_fullscreen.cc

Issue 2447143003: Use kRenderClientId command line flag when starting a render process (Closed)
Patch Set: Removed unnecessary additions Created 4 years, 1 month 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #include "content/renderer/render_widget_fullscreen.h" 5 #include "content/renderer/render_widget_fullscreen.h"
6 6
7 #include "content/common/view_messages.h" 7 #include "content/common/view_messages.h"
8 #include "third_party/WebKit/public/web/WebWidget.h" 8 #include "third_party/WebKit/public/web/WebWidget.h"
9 9
10 using blink::WebWidget; 10 using blink::WebWidget;
11 11
12 namespace content { 12 namespace content {
13 13
14 void RenderWidgetFullscreen::show(blink::WebNavigationPolicy) { 14 void RenderWidgetFullscreen::show(blink::WebNavigationPolicy) {
15 DCHECK(!did_show_) << "received extraneous Show call"; 15 DCHECK(!did_show_) << "received extraneous Show call";
16 DCHECK_NE(MSG_ROUTING_NONE, routing_id()); 16 DCHECK_NE(MSG_ROUTING_NONE, routing_id());
17 DCHECK_NE(MSG_ROUTING_NONE, opener_id_); 17 DCHECK_NE(MSG_ROUTING_NONE, opener_id_);
18 18
19 if (!did_show_) { 19 if (!did_show_) {
20 did_show_ = true; 20 did_show_ = true;
21 Send(new ViewHostMsg_ShowFullscreenWidget(opener_id_, routing_id())); 21 Send(new ViewHostMsg_ShowFullscreenWidget(opener_id_, routing_id()));
22 SetPendingWindowRect(initial_rect_); 22 SetPendingWindowRect(initial_rect_);
23 } 23 }
24 } 24 }
25 25
26 RenderWidgetFullscreen::RenderWidgetFullscreen( 26 RenderWidgetFullscreen::RenderWidgetFullscreen(
27 CompositorDependencies* compositor_deps, 27 CompositorDependencies* compositor_deps,
28 const ScreenInfo& screen_info) 28 const ScreenInfo& screen_info,
29 const cc::FrameSinkId& frame_sink_id)
29 : RenderWidget(compositor_deps, 30 : RenderWidget(compositor_deps,
30 blink::WebPopupTypeNone, 31 blink::WebPopupTypeNone,
31 screen_info, 32 screen_info,
33 frame_sink_id,
32 false, 34 false,
33 false, 35 false,
34 false) {} 36 false) {}
35 37
36 RenderWidgetFullscreen::~RenderWidgetFullscreen() {} 38 RenderWidgetFullscreen::~RenderWidgetFullscreen() {}
37 39
38 WebWidget* RenderWidgetFullscreen::CreateWebWidget() { 40 WebWidget* RenderWidgetFullscreen::CreateWebWidget() {
39 // TODO(boliu): Handle full screen render widgets here. 41 // TODO(boliu): Handle full screen render widgets here.
40 return RenderWidget::CreateWebWidget(this); 42 return RenderWidget::CreateWebWidget(this);
41 } 43 }
42 44
43 bool RenderWidgetFullscreen::Init(int32_t opener_id) { 45 bool RenderWidgetFullscreen::Init(int32_t opener_id) {
44 DCHECK(!GetWebWidget()); 46 DCHECK(!GetWebWidget());
45 47
46 bool success = RenderWidget::DoInit( 48 bool success = RenderWidget::DoInit(
47 opener_id, CreateWebWidget(), 49 opener_id, CreateWebWidget(),
48 new ViewHostMsg_CreateFullscreenWidget(opener_id, &routing_id_)); 50 new ViewHostMsg_CreateFullscreenWidget(opener_id, &routing_id_));
49 if (success) { 51 if (success) {
50 // TODO(fsamuel): This is a bit ugly. The |create_widget_message| should 52 // TODO(fsamuel): This is a bit ugly. The |create_widget_message| should
51 // probably be factored out of RenderWidget::DoInit. 53 // probably be factored out of RenderWidget::DoInit.
52 SetRoutingID(routing_id_); 54 SetRoutingID(routing_id_);
53 return true; 55 return true;
54 } 56 }
55 return false; 57 return false;
56 } 58 }
57 59
58 } // namespace content 60 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698