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

Unified Diff: blimp/engine/renderer/engine_image_serialization_processor.cc

Issue 1859753002: Blimp: create Mojo component for renderer/browser communication. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: yzshen and nyquist feedback 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
« no previous file with comments | « blimp/engine/renderer/engine_image_serialization_processor.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: blimp/engine/renderer/engine_image_serialization_processor.cc
diff --git a/blimp/common/compositor/blimp_image_serialization_processor.cc b/blimp/engine/renderer/engine_image_serialization_processor.cc
similarity index 87%
copy from blimp/common/compositor/blimp_image_serialization_processor.cc
copy to blimp/engine/renderer/engine_image_serialization_processor.cc
index 24564e4b144f51e860d2a32384ef6095093d0d4e..8d1f2bb5f05310f481cd3fab66ac312ee8da584a 100644
--- a/blimp/common/compositor/blimp_image_serialization_processor.cc
+++ b/blimp/engine/renderer/engine_image_serialization_processor.cc
@@ -2,13 +2,14 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "blimp/common/compositor/blimp_image_serialization_processor.h"
+#include "blimp/engine/renderer/engine_image_serialization_processor.h"
#include <stddef.h>
#include <vector>
#include "base/logging.h"
#include "blimp/common/compositor/webp_decoder.h"
+#include "content/public/renderer/render_frame.h"
#include "third_party/libwebp/webp/encode.h"
#include "third_party/skia/include/core/SkData.h"
#include "third_party/skia/include/core/SkPicture.h"
@@ -139,31 +140,30 @@ class WebPImageEncoder : public SkPixelSerializer {
} // namespace
namespace blimp {
+namespace engine {
-BlimpImageSerializationProcessor::BlimpImageSerializationProcessor(Mode mode) {
- switch (mode) {
- case Mode::SERIALIZATION:
- pixel_serializer_.reset(new WebPImageEncoder);
- pixel_deserializer_ = nullptr;
- break;
- case Mode::DESERIALIZATION:
- pixel_serializer_ = nullptr;
- pixel_deserializer_ = &WebPDecoder;
- break;
- default:
- NOTREACHED() << "Unknown serialization mode";
- }
+EngineImageSerializationProcessor::EngineImageSerializationProcessor(
+ mojom::BlobChannelPtr blob_channel)
+ : blob_channel_(std::move(blob_channel)) {
+ DCHECK(blob_channel_);
+
+ pixel_serializer_.reset(new WebPImageEncoder);
+
+ // Dummy BlobChannel command.
+ // TODO(nyquist): Remove this after integrating BlobChannel.
+ blob_channel_->Push("foo");
}
-BlimpImageSerializationProcessor::~BlimpImageSerializationProcessor() {}
+EngineImageSerializationProcessor::~EngineImageSerializationProcessor() {}
-SkPixelSerializer* BlimpImageSerializationProcessor::GetPixelSerializer() {
+SkPixelSerializer* EngineImageSerializationProcessor::GetPixelSerializer() {
return pixel_serializer_.get();
}
SkPicture::InstallPixelRefProc
-BlimpImageSerializationProcessor::GetPixelDeserializer() {
- return pixel_deserializer_;
+EngineImageSerializationProcessor::GetPixelDeserializer() {
+ return nullptr;
}
+} // namespace engine
} // namespace blimp
« no previous file with comments | « blimp/engine/renderer/engine_image_serialization_processor.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698