Remove the pools from the ResourceProvider.

Remove the pools from the ResourceProvider.

Pools were used in the past and were planned to be used to
call deleteOwnedResources. We have moved the responsibility
of deleting resources to the structure that created them
(for example, the PrioritizedResourceManager now deletes
contents textures, instead of calling deleteOwnedResources).

BUG=164947
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=173212

[ccameron, 2012-12-14 00:06:25]

This removes the concept of ResourceProvider pools.

I'll introduce something that the PrioritizedResourceManager uses to identify
that "things that I allocate should be counted as evictable by the GPU memory
manager" in the original CL.

Because the vast majority of createResource calls are unmanaged (not evictable),
I may just add a createManagedResource call function.

[piman, 2012-12-14 01:48:45]

LGTM, thanks!

[ccameron, 2012-12-14 01:54:32]

Thanks!!

[commit-bot: I haz the power, 2012-12-14 01:58:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ccameron@chromium.org/11578019/1

[commit-bot: I haz the power, 2012-12-14 07:46:17]

Failed to apply patch for cc/direct_renderer.cc:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file cc/direct_renderer.cc
  Hunk #1 FAILED at 267.
  1 out of 1 hunk FAILED -- saving rejects to file cc/direct_renderer.cc.rej

Patch:       cc/direct_renderer.cc
Index: cc/direct_renderer.cc
diff --git a/cc/direct_renderer.cc b/cc/direct_renderer.cc
index
0b2d74b82076e5108b3d1388d18e71a06b43677d..3ca987544d0a4d353e63c0fcaff5a1382e3cafb5
100644
--- a/cc/direct_renderer.cc
+++ b/cc/direct_renderer.cc
@@ -267,7 +267,7 @@ bool DirectRenderer::useRenderPass(DrawingFrame& frame,
const RenderPass* render
 
     CachedResource* texture = m_renderPassTextures.get(renderPass->id);
     DCHECK(texture);
-    if (!texture->id() && !texture->Allocate(Renderer::ImplPool,
renderPassTextureSize(renderPass), renderPassTextureFormat(renderPass),
ResourceProvider::TextureUsageFramebuffer))
+    if (!texture->id() && !texture->Allocate(renderPassTextureSize(renderPass),
renderPassTextureFormat(renderPass), ResourceProvider::TextureUsageFramebuffer))
         return false;
 
     return bindFramebufferToTexture(frame, texture, renderPass->output_rect);

[commit-bot: I haz the power, 2012-12-14 08:53:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ccameron@chromium.org/11578019/25

[commit-bot: I haz the power, 2012-12-14 13:00:51]

Retried try job too often on win_rel for step(s) browser_tests

[commit-bot: I haz the power, 2012-12-14 17:42:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ccameron@chromium.org/11578019/25