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

Unified Diff: content/renderer/gpu/render_widget_compositor.cc

Issue 2629243002: content: Remove blimp compositing dependencies. (Closed)
Patch Set: fix gn file Created 3 years, 11 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/gpu/render_widget_compositor.cc
diff --git a/content/renderer/gpu/render_widget_compositor.cc b/content/renderer/gpu/render_widget_compositor.cc
index b9cdc15aa7d1c0f90bcf7d33b0253d186ddf6da8..78c7d983d1af3836f1e612c7431c0b8de90a1738 100644
--- a/content/renderer/gpu/render_widget_compositor.cc
+++ b/content/renderer/gpu/render_widget_compositor.cc
@@ -30,10 +30,6 @@
#include "cc/animation/animation_host.h"
#include "cc/animation/animation_timeline.h"
#include "cc/base/switches.h"
-#include "cc/blimp/engine_picture_cache.h"
-#include "cc/blimp/image_serialization_processor.h"
-#include "cc/blimp/layer_tree_host_remote.h"
-#include "cc/blimp/remote_compositor_bridge.h"
#include "cc/blink/web_layer_impl.h"
#include "cc/debug/layer_tree_debug_state.h"
#include "cc/debug/micro_benchmark.h"
@@ -44,7 +40,6 @@
#include "cc/output/copy_output_result.h"
#include "cc/output/latency_info_swap_promise.h"
#include "cc/output/swap_promise.h"
-#include "cc/proto/compositor_message.pb.h"
#include "cc/resources/single_release_callback.h"
#include "cc/scheduler/begin_frame_source.h"
#include "cc/trees/latency_info_swap_promise_monitor.h"
@@ -213,7 +208,6 @@ RenderWidgetCompositor::RenderWidgetCompositor(
threaded_(!!compositor_deps_->GetCompositorImplThreadTaskRunner()),
never_visible_(false),
layout_and_paint_async_callback_(nullptr),
- remote_proto_channel_receiver_(nullptr),
weak_factory_(this) {}
void RenderWidgetCompositor::Initialize(float device_scale_factor,
@@ -223,24 +217,6 @@ void RenderWidgetCompositor::Initialize(float device_scale_factor,
*cmd, compositor_deps_, device_scale_factor, screen_info);
animation_host_ = cc::AnimationHost::CreateMainInstance();
-
- if (cmd->HasSwitch(switches::kUseRemoteCompositing)) {
- DCHECK(!threaded_);
-
- cc::LayerTreeHostRemote::InitParams params;
- params.client = this;
- params.main_task_runner =
- compositor_deps_->GetCompositorMainThreadTaskRunner();
- params.mutator_host = animation_host_.get();
- params.remote_compositor_bridge =
- GetContentClient()->renderer()->CreateRemoteCompositorBridge(
- this, params.main_task_runner);
- params.engine_picture_cache =
- compositor_deps_->GetImageSerializationProcessor()
- ->CreateEnginePictureCache();
- params.settings = &settings;
- layer_tree_host_ = base::MakeUnique<cc::LayerTreeHostRemote>(&params);
- } else {
cc::LayerTreeHostInProcess::InitParams params;
Charlie Reis 2017/01/19 23:37:33 Style nit: Fix indent on this block. (Might be wo
Khushal 2017/01/19 23:47:17 Woops. Done.
params.client = this;
params.settings = &settings;
@@ -264,8 +240,6 @@ void RenderWidgetCompositor::Initialize(float device_scale_factor,
layer_tree_host_ = cc::LayerTreeHostInProcess::CreateThreaded(
compositor_deps_->GetCompositorImplThreadTaskRunner(), &params);
}
- }
-
DCHECK(layer_tree_host_);
}
@@ -1099,40 +1073,12 @@ void RenderWidgetCompositor::DidLoseCompositorFrameSink() {
NOTREACHED();
}
-void RenderWidgetCompositor::SetProtoReceiver(ProtoReceiver* receiver) {
- remote_proto_channel_receiver_ = receiver;
-}
-
-void RenderWidgetCompositor::SendCompositorProto(
- const cc::proto::CompositorMessage& proto) {
- int signed_size = proto.ByteSize();
- size_t unsigned_size = base::checked_cast<size_t>(signed_size);
- std::vector<uint8_t> serialized(unsigned_size);
- proto.SerializeToArray(serialized.data(), signed_size);
- delegate_->ForwardCompositorProto(serialized);
-}
-
void RenderWidgetCompositor::SetFrameSinkId(
const cc::FrameSinkId& frame_sink_id) {
frame_sink_id_ = frame_sink_id;
layer_tree_host_->SetFrameSinkId(frame_sink_id);
}
-void RenderWidgetCompositor::OnHandleCompositorProto(
- const std::vector<uint8_t>& proto) {
- DCHECK(remote_proto_channel_receiver_);
-
- std::unique_ptr<cc::proto::CompositorMessage> deserialized(
- new cc::proto::CompositorMessage);
- int signed_size = base::checked_cast<int>(proto.size());
- if (!deserialized->ParseFromArray(proto.data(), signed_size)) {
- LOG(ERROR) << "Unable to parse compositor proto.";
- return;
- }
-
- remote_proto_channel_receiver_->OnProtoReceived(std::move(deserialized));
-}
-
void RenderWidgetCompositor::SetPaintedDeviceScaleFactor(
float device_scale) {
layer_tree_host_->GetLayerTree()->SetPaintedDeviceScaleFactor(device_scale);

Powered by Google App Engine
This is Rietveld 408576698