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

Unified Diff: content/browser/android/synchronous_compositor_host.cc

Issue 2160743002: sync compositor: Reduce begin frame sync IPC overhead (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: clang format Created 4 years, 5 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/android/synchronous_compositor_host.cc
diff --git a/content/browser/android/synchronous_compositor_host.cc b/content/browser/android/synchronous_compositor_host.cc
index b0e0b9dab5a7012e3af9754467aba79c169f46ee..45c98d6b2a3b56d0eea1ddc72e6a0aad2fab0677 100644
--- a/content/browser/android/synchronous_compositor_host.cc
+++ b/content/browser/android/synchronous_compositor_host.cc
@@ -11,6 +11,7 @@
#include "base/memory/ptr_util.h"
#include "base/memory/shared_memory.h"
#include "base/trace_event/trace_event_argument.h"
+#include "content/browser/android/synchronous_compositor_observer.h"
#include "content/browser/renderer_host/render_widget_host_view_android.h"
#include "content/browser/web_contents/web_contents_android.h"
#include "content/browser/web_contents/web_contents_impl.h"
@@ -68,6 +69,7 @@ SynchronousCompositorHost::SynchronousCompositorHost(
ui_task_runner_(BrowserThread::GetTaskRunnerForThread(BrowserThread::UI)),
process_id_(rwhva_->GetRenderWidgetHost()->GetProcess()->GetID()),
routing_id_(rwhva_->GetRenderWidgetHost()->GetRoutingID()),
+ rph_observer_(SynchronousCompositorObserver::GetOrCreateFor(process_id_)),
sender_(rwhva_->GetRenderWidgetHost()),
use_in_process_zero_copy_software_draw_(use_in_proc_software_draw),
bytes_limit_(0u),
@@ -337,13 +339,9 @@ void SynchronousCompositorHost::DidOverscroll(
over_scroll_params.current_fling_velocity);
}
-void SynchronousCompositorHost::DidSendBeginFrame() {
- SyncCompositorCommonRendererParams common_renderer_params;
- if (!sender_->Send(new SyncCompositorMsg_SynchronizeRendererState(
- routing_id_, &common_renderer_params))) {
- return;
- }
- ProcessCommonParams(common_renderer_params);
+void SynchronousCompositorHost::DidSendBeginFrame(
+ ui::WindowAndroid* window_android) {
+ rph_observer_->SyncStateAfterVSync(window_android, this);
}
void SynchronousCompositorHost::OutputSurfaceCreated() {
« no previous file with comments | « content/browser/android/synchronous_compositor_host.h ('k') | content/browser/android/synchronous_compositor_observer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698