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

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

Issue 616133002: Make RenderFrame(Host) own a RenderWidget(Host). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comments addressed Created 6 years, 2 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 (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.h" 5 #include "content/renderer/render_widget.h"
6 6
7 #include "base/auto_reset.h" 7 #include "base/auto_reset.h"
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/debug/trace_event.h" 10 #include "base/debug/trace_event.h"
(...skipping 431 matching lines...) Expand 10 before | Expand all | Expand 10 after
442 DCHECK(opener_id != MSG_ROUTING_NONE); 442 DCHECK(opener_id != MSG_ROUTING_NONE);
443 scoped_refptr<RenderWidget> widget( 443 scoped_refptr<RenderWidget> widget(
444 new RenderWidget(popup_type, screen_info, false, false, false)); 444 new RenderWidget(popup_type, screen_info, false, false, false));
445 if (widget->Init(opener_id)) { // adds reference on success. 445 if (widget->Init(opener_id)) { // adds reference on success.
446 return widget.get(); 446 return widget.get();
447 } 447 }
448 return NULL; 448 return NULL;
449 } 449 }
450 450
451 // static 451 // static
452 RenderWidget* RenderWidget::CreateForFrame(
453 int routing_id,
454 bool hidden,
455 const blink::WebScreenInfo& screen_info) {
456 DCHECK(routing_id != MSG_ROUTING_NONE);
457 scoped_refptr<RenderWidget> widget(
458 new RenderWidget(blink::WebPopupTypeNone,
459 screen_info,
460 false,
461 hidden,
462 false));
463 widget->routing_id_ = routing_id;
464 //TODO(kenrb): Change CreateWebWidget to CreateWebFrameWidget before landing!
465 if (widget->DoInit(MSG_ROUTING_NONE,
466 RenderWidget::CreateWebWidget(widget.get()),
467 nullptr)) {
468 return widget.get();
469 }
470 return NULL;
471 }
472
473 // static
452 WebWidget* RenderWidget::CreateWebWidget(RenderWidget* render_widget) { 474 WebWidget* RenderWidget::CreateWebWidget(RenderWidget* render_widget) {
453 switch (render_widget->popup_type_) { 475 switch (render_widget->popup_type_) {
454 case blink::WebPopupTypeNone: // Nothing to create. 476 case blink::WebPopupTypeNone: // Nothing to create.
455 break; 477 break;
456 case blink::WebPopupTypeSelect: 478 case blink::WebPopupTypeSelect:
457 case blink::WebPopupTypeSuggestion: 479 case blink::WebPopupTypeSuggestion:
458 return WebPopupMenu::create(render_widget); 480 return WebPopupMenu::create(render_widget);
459 case blink::WebPopupTypePage: 481 case blink::WebPopupTypePage:
460 return WebPagePopup::create(render_widget); 482 return WebPagePopup::create(render_widget);
461 default: 483 default:
(...skipping 12 matching lines...) Expand all
474 bool RenderWidget::DoInit(int32 opener_id, 496 bool RenderWidget::DoInit(int32 opener_id,
475 WebWidget* web_widget, 497 WebWidget* web_widget,
476 IPC::SyncMessage* create_widget_message) { 498 IPC::SyncMessage* create_widget_message) {
477 DCHECK(!webwidget_); 499 DCHECK(!webwidget_);
478 500
479 if (opener_id != MSG_ROUTING_NONE) 501 if (opener_id != MSG_ROUTING_NONE)
480 opener_id_ = opener_id; 502 opener_id_ = opener_id;
481 503
482 webwidget_ = web_widget; 504 webwidget_ = web_widget;
483 505
484 bool result = RenderThread::Get()->Send(create_widget_message); 506 bool result = true;
507 if (create_widget_message)
508 result = RenderThread::Get()->Send(create_widget_message);
509
485 if (result) { 510 if (result) {
486 RenderThread::Get()->AddRoute(routing_id_, this); 511 RenderThread::Get()->AddRoute(routing_id_, this);
487 // Take a reference on behalf of the RenderThread. This will be balanced 512 // Take a reference on behalf of the RenderThread. This will be balanced
488 // when we receive ViewMsg_Close. 513 // when we receive ViewMsg_Close.
489 AddRef(); 514 AddRef();
490 if (RenderThreadImpl::current()) { 515 if (RenderThreadImpl::current()) {
491 RenderThreadImpl::current()->WidgetCreated(); 516 RenderThreadImpl::current()->WidgetCreated();
492 if (is_hidden_) 517 if (is_hidden_)
493 RenderThreadImpl::current()->WidgetHidden(); 518 RenderThreadImpl::current()->WidgetHidden();
494 } 519 }
(...skipping 1700 matching lines...) Expand 10 before | Expand all | Expand 10 after
2195 void RenderWidget::RegisterVideoHoleFrame(RenderFrameImpl* frame) { 2220 void RenderWidget::RegisterVideoHoleFrame(RenderFrameImpl* frame) {
2196 video_hole_frames_.AddObserver(frame); 2221 video_hole_frames_.AddObserver(frame);
2197 } 2222 }
2198 2223
2199 void RenderWidget::UnregisterVideoHoleFrame(RenderFrameImpl* frame) { 2224 void RenderWidget::UnregisterVideoHoleFrame(RenderFrameImpl* frame) {
2200 video_hole_frames_.RemoveObserver(frame); 2225 video_hole_frames_.RemoveObserver(frame);
2201 } 2226 }
2202 #endif // defined(VIDEO_HOLE) 2227 #endif // defined(VIDEO_HOLE)
2203 2228
2204 } // namespace content 2229 } // namespace content
OLDNEW
« content/renderer/render_widget.h ('K') | « content/renderer/render_widget.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698