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

Unified Diff: content/renderer/render_thread_impl.cc

Issue 2717213004: Move SharedBitmapManager implementation out of content/ (Closed)
Patch Set: rebase Created 3 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 | « content/renderer/render_thread_impl.h ('k') | content/renderer/render_view_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_thread_impl.cc
diff --git a/content/renderer/render_thread_impl.cc b/content/renderer/render_thread_impl.cc
index 07a2abffd91d9318ee6c610c38b9a5e626c62549..5de936137000dad03534c67eb33701a5b564b744 100644
--- a/content/renderer/render_thread_impl.cc
+++ b/content/renderer/render_thread_impl.cc
@@ -56,7 +56,6 @@
#include "content/child/blob_storage/blob_message_filter.h"
#include "content/child/child_histogram_message_filter.h"
#include "content/child/child_resource_message_filter.h"
-#include "content/child/child_shared_bitmap_manager.h"
#include "content/child/content_child_helpers.h"
#include "content/child/db_message_filter.h"
#include "content/child/indexed_db/indexed_db_dispatcher.h"
@@ -145,6 +144,7 @@
#include "services/service_manager/public/cpp/connector.h"
#include "services/service_manager/public/cpp/interface_provider.h"
#include "services/service_manager/public/cpp/interface_registry.h"
+#include "services/ui/public/cpp/bitmap/child_shared_bitmap_manager.h"
#include "services/ui/public/cpp/gpu/context_provider_command_buffer.h"
#include "services/ui/public/interfaces/constants.mojom.h"
#include "skia/ext/event_tracer_impl.h"
@@ -555,13 +555,6 @@ mojom::RenderMessageFilter* RenderThreadImpl::current_render_message_filter() {
}
// static
-const scoped_refptr<mojom::ThreadSafeRenderMessageFilterAssociatedPtr>&
-RenderThreadImpl::current_thread_safe_render_message_filter() {
- DCHECK(current());
- return current()->thread_safe_render_message_filter();
-}
-
-// static
void RenderThreadImpl::SetRenderMessageFilterForTesting(
mojom::RenderMessageFilter* render_message_filter) {
g_render_message_filter_for_testing = render_message_filter;
@@ -635,10 +628,13 @@ void RenderThreadImpl::Init(
IsRunningInMash() ? ui::mojom::kServiceName : mojom::kBrowserServiceName,
GetIOTaskRunner());
- channel()->GetThreadSafeRemoteAssociatedInterface(
- &thread_safe_render_message_filter_);
- shared_bitmap_manager_.reset(
- new ChildSharedBitmapManager(thread_safe_render_message_filter_));
+ cc::mojom::SharedBitmapManagerAssociatedPtr shared_bitmap_manager_ptr;
+ render_message_filter()->GetSharedBitmapManager(
+ mojo::MakeRequest(&shared_bitmap_manager_ptr));
+ shared_bitmap_manager_.reset(new ui::ChildSharedBitmapManager(
+ cc::mojom::ThreadSafeSharedBitmapManagerAssociatedPtr::Create(
+ shared_bitmap_manager_ptr.PassInterface(),
+ GetChannel()->ipc_task_runner_refptr())));
InitializeWebKit(resource_task_queue);
@@ -2075,11 +2071,6 @@ mojom::RenderMessageFilter* RenderThreadImpl::render_message_filter() {
return render_message_filter_.get();
}
-const scoped_refptr<mojom::ThreadSafeRenderMessageFilterAssociatedPtr>&
-RenderThreadImpl::thread_safe_render_message_filter() {
- return thread_safe_render_message_filter_;
-}
-
gpu::GpuChannelHost* RenderThreadImpl::GetGpuChannel() {
if (!gpu_channel_)
return nullptr;
« no previous file with comments | « content/renderer/render_thread_impl.h ('k') | content/renderer/render_view_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698