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

Side by Side Diff: content/browser/renderer_host/render_widget_host_view_android.cc

Issue 221833009: cc: Move scheduling logic out of OutputSurface (Closed) Base URL: http://git.chromium.org/chromium/src.git@swapAck2Sched11
Patch Set: rebase; add comment about race Created 6 years, 8 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/browser/renderer_host/render_widget_host_view_android.h" 5 #include "content/browser/renderer_host/render_widget_host_view_android.h"
6 6
7 #include <android/bitmap.h> 7 #include <android/bitmap.h>
8 8
9 #include "base/android/sys_utils.h" 9 #include "base/android/sys_utils.h"
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 512 matching lines...) Expand 10 before | Expand all | Expand 10 after
523 523
524 if (flush_input_requested_) { 524 if (flush_input_requested_) {
525 flush_input_requested_ = false; 525 flush_input_requested_ = false;
526 host_->FlushInput(); 526 host_->FlushInput();
527 content_view_core_->RemoveBeginFrameSubscriber(); 527 content_view_core_->RemoveBeginFrameSubscriber();
528 } 528 }
529 529
530 host_->Send(new ViewMsg_BeginFrame(host_->GetRoutingID(), args)); 530 host_->Send(new ViewMsg_BeginFrame(host_->GetRoutingID(), args));
531 } 531 }
532 532
533 void RenderWidgetHostViewAndroid::OnSetNeedsBeginFrame( 533 void RenderWidgetHostViewAndroid::OnSetNeedsBeginFrame(bool enabled) {
534 bool enabled) {
535 TRACE_EVENT1("cc", "RenderWidgetHostViewAndroid::OnSetNeedsBeginFrame", 534 TRACE_EVENT1("cc", "RenderWidgetHostViewAndroid::OnSetNeedsBeginFrame",
536 "enabled", enabled); 535 "enabled", enabled);
537 // ContentViewCoreImpl handles multiple subscribers to the BeginFrame, so 536 // ContentViewCoreImpl handles multiple subscribers to the BeginFrame, so
538 // we have to make sure calls to ContentViewCoreImpl's 537 // we have to make sure calls to ContentViewCoreImpl's
539 // {Add,Remove}BeginFrameSubscriber are balanced, even if 538 // {Add,Remove}BeginFrameSubscriber are balanced, even if
540 // RenderWidgetHostViewAndroid's may not be. 539 // RenderWidgetHostViewAndroid's may not be.
541 if (content_view_core_ && needs_begin_frame_ != enabled) { 540 if (content_view_core_ && needs_begin_frame_ != enabled) {
542 if (enabled) 541 if (enabled)
543 content_view_core_->AddBeginFrameSubscriber(); 542 content_view_core_->AddBeginFrameSubscriber();
544 else 543 else
(...skipping 957 matching lines...) Expand 10 before | Expand all | Expand 10 after
1502 // RenderWidgetHostView, public: 1501 // RenderWidgetHostView, public:
1503 1502
1504 // static 1503 // static
1505 RenderWidgetHostView* 1504 RenderWidgetHostView*
1506 RenderWidgetHostView::CreateViewForWidget(RenderWidgetHost* widget) { 1505 RenderWidgetHostView::CreateViewForWidget(RenderWidgetHost* widget) {
1507 RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(widget); 1506 RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(widget);
1508 return new RenderWidgetHostViewAndroid(rwhi, NULL); 1507 return new RenderWidgetHostViewAndroid(rwhi, NULL);
1509 } 1508 }
1510 1509
1511 } // namespace content 1510 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698