cc: Remove RasterWorkerPool::Task.

cc: Remove RasterWorkerPool::Task.

This removes some unnecessary layers of abstraction between
internal::Task and the tile manager. Prerequisite to getting
rid of our large raster task objects and anonymous functions.

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

[reveman, 2014-02-15 01:00:34]

Here's the RasterWorkerPool::Task removal part of my original
DirectRasterWorkerPool patch. Please take a look.

[reveman, 2014-02-18 20:28:47]

Renamed RasterTaskQueue::Task to RasterTaskQueue::Item as discussed. PTAL.

[vmpstr, 2014-02-18 21:10:22]

lgtm. Thanks for explaining this.

https://codereview.chromium.org/168083002/diff/50001/cc/resources/raster_work...
File cc/resources/raster_worker_pool.h (right):

https://codereview.chromium.org/168083002/diff/50001/cc/resources/raster_work...
cc/resources/raster_worker_pool.h:125: void Swap(RasterTaskQueue* other);
This might be awkward, but wdyt about adding Begin/End/PushBack to the queue
directly as forwarding to items? That was you can have queue->Begin(),
queue->End() instead of queue->items.begin(), etc.

I think it might make this more of an opaque container.

[reveman, 2014-02-18 23:48:48]

Removed RasterTaskQueueIterator as discussed.

[reveman, 2014-02-18 23:48:58]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2014-02-18 23:50:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/reveman@chromium.org/168083002/110001

[commit-bot: I haz the power, 2014-02-19 00:03:08]

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

[commit-bot: I haz the power, 2014-02-19 00:03:08]

Retried try job too often on win_rel for step(s) app_list_unittests,
ash_unittests, aura_unittests, cacheinvalidation_unittests, cc_unittests,
check_deps, check_deps2git, chrome_elf_unittests, chromedriver_unittests,
components_unittests, compositor_unittests, content_browsertests,
content_unittests, crypto_unittests, events_unittests, google_apis_unittests,
gpu_unittests, installer_util_unittests, ipc_tests, jingle_unittests,
media_unittests, mini_installer_test, nacl_integration, ppapi_unittests,
printing_unittests, remoting_unittests, sql_unittests, sync_integration_tests,
sync_unit_tests, telemetry_perf_unittests, telemetry_unittests, views_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[reveman, 2014-02-19 00:04:17]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2014-02-19 00:06:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/reveman@chromium.org/168083002/110001

[commit-bot: I haz the power, 2014-02-19 02:42:06]

Change committed as 251938