Separate TransferableResource into own header

Separate TransferableResource into own header

This is so that we can write the picklers without having to include webcore/wtf.
This also changes the resource transfer functions to avoid copies.

BUG=146080
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=165847

[piman, 2012-11-03 00:54:48]

Dana: please review
James: please take a look at DEPS. This header will be included by content/ so
the #include <> trick doesn't work.

[danakj, 2012-11-03 01:00:51]

https://codereview.chromium.org/11358080/diff/1/cc/resource_provider.h
File cc/resource_provider.h (right):

https://codereview.chromium.org/11358080/diff/1/cc/resource_provider.h#newcode52
cc/resource_provider.h:52: GraphicsContext* context() const { return m_context;
}
Why'd you add this one? I don't see any users.

[piman, 2012-11-03 01:02:32]

On 2012/11/03 01:00:51, danakj wrote:
> https://codereview.chromium.org/11358080/diff/1/cc/resource_provider.h
> File cc/resource_provider.h (right):
> 
>
https://codereview.chromium.org/11358080/diff/1/cc/resource_provider.h#newcode52
> cc/resource_provider.h:52: GraphicsContext* context() const { return
m_context;
> }
> Why'd you add this one? I don't see any users.

Oh, left out when I separated this bit. I'll remove from this patch until we
need it later.

[danakj, 2012-11-03 01:03:40]

k, lgtm without that :)

[piman, 2012-11-03 01:05:47]

https://codereview.chromium.org/11358080/diff/1/cc/resource_provider.h
File cc/resource_provider.h (right):

https://codereview.chromium.org/11358080/diff/1/cc/resource_provider.h#newcode52
cc/resource_provider.h:52: GraphicsContext* context() const { return m_context;
}
On 2012/11/03 01:00:51, danakj wrote:
> Why'd you add this one? I don't see any users.

Done.

[jamesr, 2012-11-03 01:23:16]

lgtm

https://codereview.chromium.org/11358080/diff/4002/cc/DEPS
File cc/DEPS (right):

https://codereview.chromium.org/11358080/diff/4002/cc/DEPS#newcode34
cc/DEPS:34:
"+third_party/WebKit/Source/Platform/chromium/public/WebCompositorTransferableResourceList.h"
lgtm

[commit-bot: I haz the power, 2012-11-03 01:30:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/piman@chromium.org/11358080/4002

[commit-bot: I haz the power, 2012-11-03 07:10:23]

Change committed as 165847