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

Unified Diff: blimp/client/core/blimp_client_context_impl.h

Issue 2300493002: Move glue code of blob channel to blimp_client_context_impl. (Closed)
Patch Set: More polish on comments. 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/blimp_client_context_impl.h
diff --git a/blimp/client/core/blimp_client_context_impl.h b/blimp/client/core/blimp_client_context_impl.h
index 355dddeb45fca4b5264b7aa2ddeb4f1d454cc0ee..f5ec68c2e8bd6a99755e594777eedf816bd000ba 100644
--- a/blimp/client/core/blimp_client_context_impl.h
+++ b/blimp/client/core/blimp_client_context_impl.h
@@ -12,6 +12,7 @@
#include "base/memory/weak_ptr.h"
#include "base/single_thread_task_runner.h"
#include "base/threading/thread.h"
+#include "blimp/client/core/compositor/blob_image_serialization_processor.h"
#include "blimp/client/core/session/client_network_components.h"
#include "blimp/client/core/session/identity_source.h"
#include "blimp/client/core/session/network_event_observer.h"
@@ -26,6 +27,7 @@ namespace client {
class BlimpCompositorDependencies;
class BlimpContentsManager;
+class BlobManager;
class CompositorDependencies;
class ImeFeature;
class NavigationFeature;
@@ -35,8 +37,10 @@ class TabControlFeature;
// BlimpClientContextImpl is the implementation of the main context-class for
// the blimp client.
-class BlimpClientContextImpl : public BlimpClientContext,
- public NetworkEventObserver {
+class BlimpClientContextImpl
+ : public BlimpClientContext,
+ public BlobImageSerializationProcessor::ErrorDelegate,
+ public NetworkEventObserver {
public:
// The |io_thread_task_runner| must be the task runner to use for IO
// operations.
@@ -83,6 +87,9 @@ class BlimpClientContextImpl : public BlimpClientContext,
// service.
void CreateIdentitySource();
+ // BlobImageSerializationProcessor::ErrorDelegate implementation.
+ void OnImageDecodeError() override;
+
// Provides functionality from the embedder.
BlimpClientContextDelegate* delegate_ = nullptr;
@@ -100,6 +107,7 @@ class BlimpClientContextImpl : public BlimpClientContext,
std::unique_ptr<BlimpCompositorDependencies> blimp_compositor_dependencies_;
// Features to handle all incoming and outgoing protobuf messages.
+ std::unique_ptr<BlobManager> blob_manager_;
std::unique_ptr<ImeFeature> ime_feature_;
std::unique_ptr<NavigationFeature> navigation_feature_;
std::unique_ptr<RenderWidgetFeature> render_widget_feature_;

Powered by Google App Engine
This is Rietveld 408576698