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

Unified Diff: content/browser/renderer_host/begin_frame_observer_proxy.cc

Issue 1841083007: Remove SendBeginFramesToChildren plumbing (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@scheduler_output_surface_client_set_beginframesource
Patch Set: Rebase Created 4 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/begin_frame_observer_proxy.cc
diff --git a/content/browser/renderer_host/begin_frame_observer_proxy.cc b/content/browser/renderer_host/begin_frame_observer_proxy.cc
deleted file mode 100644
index 06baa18bd2a128390dd6436cf581077043c0f5fc..0000000000000000000000000000000000000000
--- a/content/browser/renderer_host/begin_frame_observer_proxy.cc
+++ /dev/null
@@ -1,78 +0,0 @@
-// Copyright 2015 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "content/browser/renderer_host/begin_frame_observer_proxy.h"
-
-namespace content {
-
-BeginFrameObserverProxy::BeginFrameObserverProxy(
- BeginFrameObserverProxyClient* client)
- : needs_begin_frames_(false),
- client_(client),
- compositor_(nullptr) {
-}
-
-BeginFrameObserverProxy::~BeginFrameObserverProxy() {
- DCHECK(!compositor_);
-}
-
-void BeginFrameObserverProxy::SetNeedsBeginFrames(bool needs_begin_frames) {
- if (needs_begin_frames_ == needs_begin_frames)
- return;
-
- needs_begin_frames_ = needs_begin_frames;
-
- // In some cases, BeginFrame message is requested before |client_|'s window is
- // added in the root window hierarchy.
- if (!compositor_)
- return;
-
- if (needs_begin_frames)
- StartObservingBeginFrames();
- else
- StopObservingBeginFrames();
-}
-
-void BeginFrameObserverProxy::SetCompositor(ui::Compositor* compositor) {
- DCHECK(!compositor_);
- DCHECK(compositor);
-
- compositor_ = compositor;
- compositor_->AddObserver(this);
- if (needs_begin_frames_)
- StartObservingBeginFrames();
-}
-
-void BeginFrameObserverProxy::ResetCompositor() {
- if (!compositor_)
- return;
- compositor_->RemoveObserver(this);
-
- if (needs_begin_frames_)
- StopObservingBeginFrames();
- compositor_ = nullptr;
-}
-
-void BeginFrameObserverProxy::OnSendBeginFrame(const cc::BeginFrameArgs& args) {
- if (last_sent_begin_frame_args_.frame_time != args.frame_time)
- client_->SendBeginFrame(args);
- last_sent_begin_frame_args_ = args;
-}
-
-void BeginFrameObserverProxy::OnCompositingShuttingDown(
- ui::Compositor* compositor) {
- ResetCompositor();
-}
-
-void BeginFrameObserverProxy::StartObservingBeginFrames() {
- DCHECK(compositor_);
- compositor_->AddBeginFrameObserver(this);
-}
-
-void BeginFrameObserverProxy::StopObservingBeginFrames() {
- DCHECK(compositor_);
- compositor_->RemoveBeginFrameObserver(this);
-}
-
-} // namespace content

Powered by Google App Engine
This is Rietveld 408576698