content: Rename RasterWorkerPool -> CategorizedWorkerPool.

content: Rename RasterWorkerPool -> CategorizedWorkerPool.

When this class was implemented, it was used for only raster work. But
we had started using it for other tasks, along with raster tasks, being
scheduled with different interfaces provided in the class.

BUG=599863
Committed: https://crrev.com/fad657e7142a08ebe2f632ae70149f5719c26a58
Cr-Commit-Position: refs/heads/master@{#397079}

[prashant.n, 2016-05-31 15:11:15]

Description was changed from

==========
content: Rename RasterWorkerPool -> CategorizedWorkerPool.

When this class was implemented, it was used for only raster work. But
we had started using it for other tasks, along with raster tasks, being
scheduled with different interfaces provided in the class.

BUG=599863
==========

to

==========
content: Rename RasterWorkerPool -> CategorizedWorkerPool.

When this class was implemented, it was used for only raster work. But
we had started using it for other tasks, along with raster tasks, being
scheduled with different interfaces provided in the class.

BUG=599863
==========

[prashant.n, 2016-05-31 15:11:15]

prashant.n@samsung.com changed reviewers:
+ reveman@chromium.org

[prashant.n, 2016-05-31 15:24:10]

ptal. I'll submit the kNumRasterThreads change in another CL.

[prashant.n, 2016-05-31 15:46:30]

The CQ bit was checked by prashant.n@samsung.com to run a CQ dry run

[commit-bot: I haz the power, 2016-05-31 15:46:47]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2021323002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2021323002/1

[reveman, 2016-05-31 16:05:46]

reveman@chromium.org changed reviewers:
+ avi@chromium.org

[reveman, 2016-05-31 16:05:47]

lgtm, just a small request.

+avi

https://codereview.chromium.org/2021323002/diff/1/content/renderer/categorize...
File content/renderer/categorized_worker_pool.h (right):

https://codereview.chromium.org/2021323002/diff/1/content/renderer/categorize...
content/renderer/categorized_worker_pool.h:30: // 3. It's also possible to get
the underlying TaskGraphRunner to schedule a
As TaskGraphRunner is now an abstract interface in the same way as TaskRunner,
can you update this comment to be similar to 1). E.g. "It also implements
TaskGraphRunner interface that allows a graph of tasks to be scheduled..."

[Avi (use Gerrit), 2016-05-31 16:10:43]

It's as good a name as any.

LGTM

[prashant.n, 2016-05-31 16:46:08]

https://codereview.chromium.org/2021323002/diff/1/content/renderer/categorize...
File content/renderer/categorized_worker_pool.h (right):

https://codereview.chromium.org/2021323002/diff/1/content/renderer/categorize...
content/renderer/categorized_worker_pool.h:30: // 3. It's also possible to get
the underlying TaskGraphRunner to schedule a
May be earlier we had embedded object. I'll modify it.

[commit-bot: I haz the power, 2016-05-31 16:47:48]

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

[commit-bot: I haz the power, 2016-05-31 16:47:48]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[prashant.n, 2016-05-31 16:56:08]

prashant.n@samsung.com changed reviewers:
+ danakj@chromium.org, jam@chromium.org

[prashant.n, 2016-05-31 16:56:09]

+danakj | jam

[prashant.n, 2016-05-31 16:56:27]

The CQ bit was checked by prashant.n@samsung.com

[prashant.n, 2016-05-31 16:56:28]

The patchset sent to the CQ was uploaded after l-g-t-m from avi@chromium.org,
reveman@chromium.org
Link to the patchset: https://codereview.chromium.org/2021323002/#ps20001
(title: "feedback")

[commit-bot: I haz the power, 2016-05-31 16:57:04]

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

[commit-bot: I haz the power, 2016-05-31 16:57:06]

This CL has an open dependency (Issue 2021023002 Patch 1). Please resolve the
dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[danakj, 2016-05-31 18:28:19]

base LGTM

[prashant.n, 2016-06-01 05:29:15]

The CQ bit was checked by prashant.n@samsung.com

[prashant.n, 2016-06-01 05:29:15]

The patchset sent to the CQ was uploaded after l-g-t-m from danakj@chromium.org,
reveman@chromium.org, avi@chromium.org
Link to the patchset: https://codereview.chromium.org/2021323002/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2016-06-01 05:29:32]

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

[commit-bot: I haz the power, 2016-06-01 07:52:30]

Description was changed from

==========
content: Rename RasterWorkerPool -> CategorizedWorkerPool.

When this class was implemented, it was used for only raster work. But
we had started using it for other tasks, along with raster tasks, being
scheduled with different interfaces provided in the class.

BUG=599863
==========

to

==========
content: Rename RasterWorkerPool -> CategorizedWorkerPool.

When this class was implemented, it was used for only raster work. But
we had started using it for other tasks, along with raster tasks, being
scheduled with different interfaces provided in the class.

BUG=599863
==========

[commit-bot: I haz the power, 2016-06-01 07:52:31]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-01 07:54:01]

Description was changed from

==========
content: Rename RasterWorkerPool -> CategorizedWorkerPool.

When this class was implemented, it was used for only raster work. But
we had started using it for other tasks, along with raster tasks, being
scheduled with different interfaces provided in the class.

BUG=599863
==========

to

==========
content: Rename RasterWorkerPool -> CategorizedWorkerPool.

When this class was implemented, it was used for only raster work. But
we had started using it for other tasks, along with raster tasks, being
scheduled with different interfaces provided in the class.

BUG=599863

Committed: https://crrev.com/fad657e7142a08ebe2f632ae70149f5719c26a58
Cr-Commit-Position: refs/heads/master@{#397079}
==========

[commit-bot: I haz the power, 2016-06-01 07:54:02]

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