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

Unified Diff: content/renderer/android/synchronous_compositor_external_begin_frame_source.cc

Issue 1969263004: sync compositor: Remove begin frame source (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase onto https://codereview.chromium.org/1974133002/ Created 4 years, 6 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/renderer/android/synchronous_compositor_external_begin_frame_source.cc
diff --git a/content/renderer/android/synchronous_compositor_external_begin_frame_source.cc b/content/renderer/android/synchronous_compositor_external_begin_frame_source.cc
deleted file mode 100644
index 7bb1dc8f72766b7cd80aa24b8c595003a93934e8..0000000000000000000000000000000000000000
--- a/content/renderer/android/synchronous_compositor_external_begin_frame_source.cc
+++ /dev/null
@@ -1,79 +0,0 @@
-// Copyright 2014 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/renderer/android/synchronous_compositor_external_begin_frame_source.h"
-
-#include "cc/output/begin_frame_args.h"
-#include "content/renderer/android/synchronous_compositor_registry.h"
-
-namespace content {
-
-SynchronousCompositorExternalBeginFrameSource::
- SynchronousCompositorExternalBeginFrameSource(
- int routing_id,
- SynchronousCompositorRegistry* registry)
- : routing_id_(routing_id),
- registry_(registry),
- registered_(false),
- client_(nullptr) {
- thread_checker_.DetachFromThread();
-}
-
-SynchronousCompositorExternalBeginFrameSource::
- ~SynchronousCompositorExternalBeginFrameSource() {
- DCHECK(CalledOnValidThread());
-
- if (registered_) {
- registry_->UnregisterBeginFrameSource(routing_id_, this);
- }
- DCHECK(!client_);
-}
-
-void SynchronousCompositorExternalBeginFrameSource::BeginFrame(
- const cc::BeginFrameArgs& args) {
- DCHECK(CalledOnValidThread());
- CallOnBeginFrame(args);
-}
-
-void SynchronousCompositorExternalBeginFrameSource::SetClient(
- SynchronousCompositorExternalBeginFrameSourceClient* client) {
- DCHECK(CalledOnValidThread());
- if (client_ == client)
- return;
-
- if (client_)
- client_->OnNeedsBeginFramesChange(false);
-
- client_ = client;
-
- if (client_)
- client_->OnNeedsBeginFramesChange(needs_begin_frames());
-
- // State without client is paused, and default client state is not paused.
- SetBeginFrameSourcePaused(!client_);
-}
-
-void SynchronousCompositorExternalBeginFrameSource::OnNeedsBeginFramesChanged(
- bool needs_begin_frames) {
- DCHECK(CalledOnValidThread());
- if (client_)
- client_->OnNeedsBeginFramesChange(needs_begin_frames);
-}
-
-void SynchronousCompositorExternalBeginFrameSource::AddObserver(
- cc::BeginFrameObserver* obs) {
- DCHECK(CalledOnValidThread());
- BeginFrameSourceBase::AddObserver(obs);
- if (registered_)
- return;
- registry_->RegisterBeginFrameSource(routing_id_, this);
- registered_ = true;
-}
-
-bool
-SynchronousCompositorExternalBeginFrameSource::CalledOnValidThread() const {
- return thread_checker_.CalledOnValidThread();
-}
-
-} // namespace content

Powered by Google App Engine
This is Rietveld 408576698