cc: Remove TextureUploader.

cc: Remove TextureUploader.

Drop the TextureUploader and the methods on ResourceProvider for using
it. This is no longer used except in a few places that should be using
ResourceProvider::CopyToResource instead. Switch those places over.

R=enne, vmpstr
BUG=413479
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/84b61428f5da71db985a929c57a6dc49cf07b465
Cr-Commit-Position: refs/heads/master@{#335341}

[danakj, 2015-06-19 01:16:11]

Depends on https://codereview.chromium.org/1194743002/

[danakj, 2015-06-19 18:25:39]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2015-06-19 18:27:49]

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

[commit-bot: I haz the power, 2015-06-19 18:27:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-19 18:27:54]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[vmpstr, 2015-06-19 18:33:46]

https://codereview.chromium.org/1191173002/diff/1/cc/resources/prioritized_re...
File cc/resources/prioritized_resource.cc (left):

https://codereview.chromium.org/1191173002/diff/1/cc/resources/prioritized_re...
cc/resources/prioritized_resource.cc:91: resource_provider->SetPixels(
This doesn't need CopyToResource?

[danakj, 2015-06-19 18:34:41]

https://codereview.chromium.org/1191173002/diff/1/cc/resources/prioritized_re...
File cc/resources/prioritized_resource.cc (left):

https://codereview.chromium.org/1191173002/diff/1/cc/resources/prioritized_re...
cc/resources/prioritized_resource.cc:91: resource_provider->SetPixels(
On 2015/06/19 18:33:46, vmpstr wrote:
> This doesn't need CopyToResource?

Dead code, didn't bother. Unittests pass.

[vmpstr, 2015-06-19 18:40:05]

lgtm

[danakj, 2015-06-19 18:40:23]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2015-06-19 18:41:12]

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

[commit-bot: I haz the power, 2015-06-19 20:54:40]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-06-19 20:56:42]

Patchset 1 (id:??) landed as
https://crrev.com/84b61428f5da71db985a929c57a6dc49cf07b465
Cr-Commit-Position: refs/heads/master@{#335341}

[brianderson, 2015-06-19 21:57:33]

brianderson@chromium.org changed reviewers:
+ brianderson@chromium.org

[brianderson, 2015-06-19 21:57:33]

Now that this is gone, I think we can remove DidAnticipatedDrawTimeChange and
the final remaining polling logic from cc::Scheduler.

Death to SetupPollingMechanisms!

[danakj, 2015-06-19 22:05:04]

On Fri, Jun 19, 2015 at 2:57 PM, <brianderson@chromium.org> wrote:

> Now that this is gone, I think we can remove DidAnticipatedDrawTimeChange
> and
> the final remaining polling logic from cc::Scheduler.
>

AnticipatedDrawTime is already gone. Not sure if there's more. Please do
check. :)


>
> Death to SetupPollingMechanisms!
>
> https://codereview.chromium.org/1191173002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[mithro-old, 2015-06-22 12:18:15]

On 2015/06/19 22:05:04, danakj wrote:
> On Fri, Jun 19, 2015 at 2:57 PM, <mailto:brianderson@chromium.org> wrote:

<snip>

> >
> > Death to SetupPollingMechanisms!
> >
> > https://codereview.chromium.org/1191173002/
> >

\o/

Yay, thanks for doing this!