Make cc::SoftwareImageDecodeController, cc::ResourcePool, and cc::StagingBufferPoo…

cc/tiles/gpu_image_decode_controller.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/tiles/gpu_image_decode_controller.h"
 
 #include <inttypes.h>
 
 #include "base/memory/discardable_memory_allocator.h"
 #include "base/memory/ptr_util.h"
@@ skipping 199 matching lines @@
   ~ImageUploadTaskImpl() override {}
 
  private:
   GpuImageDecodeController* controller_;
   DrawImage image_;
   const ImageDecodeController::TracingInfo tracing_info_;
 
   DISALLOW_COPY_AND_ASSIGN(ImageUploadTaskImpl);
 };
 
 GpuImageDecodeController::DecodedImageData::DecodedImageData() = default;

[tasak, 2016/08/26 10:08:05]

Need to register this from ChildMemoryCoordinator here.

 GpuImageDecodeController::DecodedImageData::~DecodedImageData() {
   ResetData();

[tasak, 2016/08/26 10:08:05]

Need to unregister this from ChildMemoryCoordinator here.

 }
 
 bool GpuImageDecodeController::DecodedImageData::Lock() {
   DCHECK(!is_locked_);
   is_locked_ = data_->Lock();
   if (is_locked_)
     ++usage_stats_.lock_count;
   return is_locked_;
 }
 
@@ skipping 894 matching lines @@
 
 bool GpuImageDecodeController::DiscardableIsLockedForTesting(
     const DrawImage& image) {
   base::AutoLock lock(lock_);
   auto found = persistent_cache_.Peek(image.image()->uniqueID());
   DCHECK(found != persistent_cache_.end());
   ImageData* image_data = found->second.get();
   return image_data->decode.is_locked();
 }
 
+void GpuImageDecodeController::OnMemoryStateChange(
+    memory_coordinator::mojom::MemoryState state) {
+  if (state == memory_coordinator::mojom::MemoryState::SUSPENDED)
+    SetShouldAggressivelyFreeResources(true);

[ericrk, 2016/08/31 17:50:32]

Setting this is stateful, so we need to be careful about setting this from more
than one place (we need to make sure this is consistently re-set to "false" once
we are no longer in a suspended memory state.)

Can we ever be "SUSPENDED" while visible? GpuImageDecodeController should
already set AggressivelyFreeResources(true) any time we are not visible, so this
is likely not necessary. You could DCHECK that we are aggressively freeing in
this case.

+}
+
 }  // namespace cc