cc: Stop using TextureUploader for UIResources, HUD, and tests.

cc: Stop using TextureUploader for UIResources, HUD, and tests.

This adds a new simpler method to ResourceProvider called
CopyToResource and uses that instead. This removes TextureUploader
from the path of most impl-side painting code.
R=enne
BUG=454575
Committed: https://crrev.com/8d86c9e54e34d7c4b75b6c94689ef28b1fe65dfb
Cr-Commit-Position: refs/heads/master@{#316978}

[danakj, 2015-02-19 00:53:44]

After this, the remaining hold outs are the VideoResourceUpdater and async
uploads.

[enne (OOO), 2015-02-19 01:01:27]

https://codereview.chromium.org/938893002/diff/20001/cc/resources/resource_pr...
File cc/resources/resource_provider_unittest.cc (right):

https://codereview.chromium.org/938893002/diff/20001/cc/resources/resource_pr...
cc/resources/resource_provider_unittest.cc:610: TEST_P(ResourceProviderTest,
SimpleUpload) {
Would it make sense to test bitmap and ETC1 resources here too?

[danakj, 2015-02-19 01:03:56]

https://codereview.chromium.org/938893002/diff/20001/cc/resources/resource_pr...
File cc/resources/resource_provider_unittest.cc (right):

https://codereview.chromium.org/938893002/diff/20001/cc/resources/resource_pr...
cc/resources/resource_provider_unittest.cc:610: TEST_P(ResourceProviderTest,
SimpleUpload) {
On 2015/02/19 01:01:27, enne wrote:
> Would it make sense to test bitmap and ETC1 resources here too?

Hm!

Bitmap should be covered by TEST_P-ness. But ETC would not.

/tries

[danakj, 2015-02-19 01:38:41]

On 2015/02/19 01:03:56, danakj wrote:
>
https://codereview.chromium.org/938893002/diff/20001/cc/resources/resource_pr...
> File cc/resources/resource_provider_unittest.cc (right):
> 
>
https://codereview.chromium.org/938893002/diff/20001/cc/resources/resource_pr...
> cc/resources/resource_provider_unittest.cc:610: TEST_P(ResourceProviderTest,
> SimpleUpload) {
> On 2015/02/19 01:01:27, enne wrote:
> > Would it make sense to test bitmap and ETC1 resources here too?
> 
> Hm!
> 
> Bitmap should be covered by TEST_P-ness. But ETC would not.
> 
> /tries

Oh there is a test. I forgot to change some more tests in ResourceProviderTests.
Now we test ETC1, and I removed it from TextureUploader.

[danakj, 2015-02-19 01:39:34]

PTAL

[enne (OOO), 2015-02-19 01:40:46]

The CQ bit was checked by enne@chromium.org

[enne (OOO), 2015-02-19 01:40:47]

lgtm

[commit-bot: I haz the power, 2015-02-19 01:41:42]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/938893002/40001

[commit-bot: I haz the power, 2015-02-19 02:41:17]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-19 02:41:44]

Patchset 3 (id:??) landed as
https://crrev.com/8d86c9e54e34d7c4b75b6c94689ef28b1fe65dfb
Cr-Commit-Position: refs/heads/master@{#316978}