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

Unified Diff: content/public/test/mock_render_process_host.cc

Issue 247263005: Remove TransportDIB methods from RenderProcess(Host) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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/public/test/mock_render_process_host.h ('k') | content/renderer/render_process.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/public/test/mock_render_process_host.cc
diff --git a/content/public/test/mock_render_process_host.cc b/content/public/test/mock_render_process_host.cc
index b07f5e50e14e076723cd1dbe86ab8dd8f3876c3d..50c19ee849d81fe6876e80482f353b4ed4ce8328 100644
--- a/content/public/test/mock_render_process_host.cc
+++ b/content/public/test/mock_render_process_host.cc
@@ -19,8 +19,7 @@
namespace content {
MockRenderProcessHost::MockRenderProcessHost(BrowserContext* browser_context)
- : transport_dib_(NULL),
- bad_msg_count_(0),
+ : bad_msg_count_(0),
factory_(NULL),
id_(ChildProcessHostImpl::GenerateChildProcessUniqueId()),
browser_context_(browser_context),
@@ -37,7 +36,6 @@ MockRenderProcessHost::MockRenderProcessHost(BrowserContext* browser_context)
MockRenderProcessHost::~MockRenderProcessHost() {
ChildProcessSecurityPolicyImpl::GetInstance()->Remove(GetID());
- delete transport_dib_;
if (factory_)
factory_->Remove(this);
@@ -141,34 +139,6 @@ bool MockRenderProcessHost::Send(IPC::Message* msg) {
return true;
}
-TransportDIB* MockRenderProcessHost::MapTransportDIB(TransportDIB::Id dib_id) {
-#if defined(OS_WIN)
- // NULL should be used here instead of INVALID_HANDLE_VALUE (or pseudo-handle)
- // except for when dealing with the small number of Win16-derived APIs
- // that require INVALID_HANDLE_VALUE (e.g. CreateFile and friends).
- HANDLE duped = NULL;
- if (!DuplicateHandle(GetCurrentProcess(), dib_id.handle, GetCurrentProcess(),
- &duped, 0, TRUE, DUPLICATE_SAME_ACCESS))
- duped = NULL;
- return TransportDIB::Map(duped);
-#elif defined(OS_ANDROID)
- // On Android, Handles and Ids are the same underlying type.
- return TransportDIB::Map(dib_id);
-#else
- // On POSIX, TransportDIBs are always created in the browser, so we cannot map
- // one from a dib_id.
- return TransportDIB::Create(100 * 100 * 4, 0);
-#endif
-}
-
-TransportDIB* MockRenderProcessHost::GetTransportDIB(TransportDIB::Id dib_id) {
- if (transport_dib_)
- return transport_dib_;
-
- transport_dib_ = MapTransportDIB(dib_id);
- return transport_dib_;
-}
-
int MockRenderProcessHost::GetID() const {
return id_;
}
« no previous file with comments | « content/public/test/mock_render_process_host.h ('k') | content/renderer/render_process.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698