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

Unified Diff: blimp/client/core/compositor/delegated_output_surface.cc

Issue 2308143002: Fix DelegatedOutputSurface in Blimp (Closed)
Patch Set: Created 4 years, 3 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: blimp/client/core/compositor/delegated_output_surface.cc
diff --git a/blimp/client/core/compositor/delegated_output_surface.cc b/blimp/client/core/compositor/delegated_output_surface.cc
index 6772d6b9cad5f9d3dad95e6d32931d861b7d145e..a28d8a23b953ea11b9c4fe856012b4870e14f670 100644
--- a/blimp/client/core/compositor/delegated_output_surface.cc
+++ b/blimp/client/core/compositor/delegated_output_surface.cc
@@ -9,7 +9,11 @@
#include "base/memory/ptr_util.h"
#include "base/single_thread_task_runner.h"
#include "base/threading/thread_checker.h"
+#include "base/threading/thread_task_runner_handle.h"
#include "cc/output/compositor_frame.h"
+#include "cc/output/output_surface_client.h"
+#include "cc/scheduler/begin_frame_source.h"
+#include "cc/scheduler/delay_based_time_source.h"
namespace blimp {
namespace client {
@@ -46,6 +50,11 @@ uint32_t DelegatedOutputSurface::GetFramebufferCopyTextureFormat() {
bool DelegatedOutputSurface::BindToClient(cc::OutputSurfaceClient* client) {
bool success = cc::OutputSurface::BindToClient(client);
if (success) {
+ begin_frame_source_ = base::MakeUnique<cc::DelayBasedBeginFrameSource>(
+ base::MakeUnique<cc::DelayBasedTimeSource>(
+ base::ThreadTaskRunnerHandle::Get().get()));
+ client->SetBeginFrameSource(begin_frame_source_.get());
Khushal 2016/09/03 16:04:31 Woops. Also destroy the |begin_frame_source_| and
+
main_task_runner_->PostTask(
FROM_HERE, base::Bind(&BlimpOutputSurfaceClient::BindToOutputSurface,
client_, weak_factory_.GetWeakPtr()));
« no previous file with comments | « blimp/client/core/compositor/delegated_output_surface.h ('k') | blimp/client/support/compositor/blimp_layer_tree_settings.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698