Rename cc::ResourceProvider::ResourceId to cc::ResourceId and move it to its own file.

Rename cc::ResourceProvider::ResourceId to cc::ResourceId and move it to its own file.

This allows files like surface_resource_holder.h and transferable_resource.h to depend on the typedef without depending on resource_provider.h

Committed: https://crrev.com/bbd425e075be8416cdc1d83a0b3907c17194a6ae
Cr-Commit-Position: refs/heads/master@{#330459}

[jbauman, 2015-05-15 02:07:44]

jbauman@chromium.org changed reviewers:
+ danakj@chromium.org, jamesr@chromium.org

[danakj, 2015-05-15 17:19:35]

LGTM

https://codereview.chromium.org/1144523003/diff/60001/cc/resources/resource_id.h
File cc/resources/resource_id.h (right):

https://codereview.chromium.org/1144523003/diff/60001/cc/resources/resource_i...
cc/resources/resource_id.h:10: using ResourceId = unsigned;
nit: can you uint32_t instead?

[jbauman, 2015-05-15 22:27:12]

On 2015/05/15 17:19:35, danakj wrote:
> LGTM
> 
>
https://codereview.chromium.org/1144523003/diff/60001/cc/resources/resource_id.h
> File cc/resources/resource_id.h (right):
> 
>
https://codereview.chromium.org/1144523003/diff/60001/cc/resources/resource_i...
> cc/resources/resource_id.h:10: using ResourceId = unsigned;
> nit: can you uint32_t instead?

done.

[jbauman, 2015-05-18 22:45:42]

The CQ bit was checked by jbauman@chromium.org

[jbauman, 2015-05-18 22:45:43]

The patchset sent to the CQ was uploaded after l-g-t-m from danakj@chromium.org
Link to the patchset: https://codereview.chromium.org/1144523003/#ps120001
(title: " ")

[commit-bot: I haz the power, 2015-05-18 22:46:04]

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

[commit-bot: I haz the power, 2015-05-18 22:57:46]

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

[commit-bot: I haz the power, 2015-05-18 22:57:48]

Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[jbauman, 2015-05-19 00:19:32]

The CQ bit was checked by jbauman@chromium.org

[jbauman, 2015-05-19 00:19:33]

The patchset sent to the CQ was uploaded after l-g-t-m from danakj@chromium.org
Link to the patchset: https://codereview.chromium.org/1144523003/#ps140001
(title: " ")

[commit-bot: I haz the power, 2015-05-19 00:20:28]

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

[commit-bot: I haz the power, 2015-05-19 00:33:44]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2015-05-19 00:34:42]

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