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

Unified Diff: components/viz/client/client_shared_bitmap_manager.cc

Issue 2965103003: Test
Patch Set: c Created 3 years, 5 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: components/viz/client/client_shared_bitmap_manager.cc
diff --git a/components/viz/client/client_shared_bitmap_manager.cc b/components/viz/client/client_shared_bitmap_manager.cc
index 0651549b8c50395167dcb2c7e03c8708bdf7b353..b3469abad340226498f7ec28c8272f1f34a835b2 100644
--- a/components/viz/client/client_shared_bitmap_manager.cc
+++ b/components/viz/client/client_shared_bitmap_manager.cc
@@ -55,6 +55,9 @@ class ClientSharedBitmap : public cc::SharedBitmap {
return base::SharedMemoryHandle();
return shared_memory_holder_->handle();
}
+ base::SharedMemory* GetSharedMemory() const override {
+ return shared_memory_holder_.get();
+ }
private:
scoped_refptr<
@@ -135,7 +138,7 @@ ClientSharedBitmapManager::AllocateSharedBitmap(const gfx::Size& size) {
// Close the associated FD to save resources, the previously mapped memory
// remains available.
- memory->Close();
+ //memory->Close();
return base::MakeUnique<ClientSharedBitmap>(
shared_bitmap_allocation_notifier_, std::move(memory), id);
@@ -161,6 +164,7 @@ ClientSharedBitmapManager::GetBitmapForSharedMemory(base::SharedMemory* mem) {
void ClientSharedBitmapManager::NotifyAllocatedSharedBitmap(
base::SharedMemory* memory,
const cc::SharedBitmapId& id) {
+ return;
base::SharedMemoryHandle handle_to_send =
base::SharedMemory::DuplicateHandle(memory->handle());
if (!base::SharedMemory::IsHandleValid(handle_to_send)) {
« no previous file with comments | « cc/trees/layer_tree_host_impl.cc ('k') | components/viz/service/display_embedder/server_shared_bitmap_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698