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

Unified Diff: cc/tiles/gpu_image_decode_controller.cc

Issue 2349553002: Keep allocation size available in minidumps. (Closed)
Patch Set: 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/tiles/gpu_image_decode_controller.cc
diff --git a/cc/tiles/gpu_image_decode_controller.cc b/cc/tiles/gpu_image_decode_controller.cc
index 405879e9f65abf76460441306366a9df3a84bd41..61f3f1d8ec9ffa6b266a47e61f414bb5b3fca430 100644
--- a/cc/tiles/gpu_image_decode_controller.cc
+++ b/cc/tiles/gpu_image_decode_controller.cc
@@ -945,9 +945,10 @@ void GpuImageDecodeController::DecodeImageIfNecessary(
{
base::AutoUnlock unlock(lock_);
+ auto image_data_size = image_data->size;
+ base::debug::Alias(image_data_size);
jbauman 2016/09/15 23:08:34 I think you need &image_data_size
cblume 2016/09/15 23:26:31 Done. Sorry, my mind was elsewhere.
backing_memory = base::DiscardableMemoryAllocator::GetInstance()
->AllocateLockedDiscardableMemory(image_data->size);
- CHECK(backing_memory);
switch (image_data->mode) {
case DecodedDataMode::CPU: {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698