add a couple of casts.

add a couple of casts.

BUG=614150
Committed: https://crrev.com/40c05d919958da304af825c140ee76a8e386c94b
Cr-Commit-Position: refs/heads/master@{#395443}

[bburns, 2016-05-23 20:33:50]

bburns@chromium.org changed reviewers:
+ fgorski@chromium.org, romax@chromium.org

[romax, 2016-05-23 20:39:43]

lgtm

[fgorski, 2016-05-23 20:45:42]

lgtm, but address the comment

https://codereview.chromium.org/2004993002/diff/20001/components/offline_page...
File components/offline_pages/offline_page_storage_manager.cc (right):

https://codereview.chromium.org/2004993002/diff/20001/components/offline_page...
components/offline_pages/offline_page_storage_manager.cc:115: int page_list_size
= page_list.size();
this is the place the compiler complained about. But good that you found the one
above as well.

[fgorski, 2016-05-23 20:50:06]

lgtm

[fgorski, 2016-05-23 20:51:04]

please update description to BUG=614150

[bburns, 2016-05-23 20:51:30]

Description was changed from

==========
add a couple of casts.

BUG=NONE
==========

to

==========
add a couple of casts.

BUG=614150
==========

[bburns, 2016-05-23 20:51:35]

The CQ bit was checked by bburns@chromium.org

[bburns, 2016-05-23 20:51:35]

The patchset sent to the CQ was uploaded after l-g-t-m from romax@chromium.org
Link to the patchset: https://codereview.chromium.org/2004993002/#ps40001
(title: "address comments.")

[commit-bot: I haz the power, 2016-05-23 20:51:58]

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

[commit-bot: I haz the power, 2016-05-23 20:56:57]

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

[commit-bot: I haz the power, 2016-05-23 20:56:58]

Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  chromeos_x86-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)

[bburns, 2016-05-23 22:42:13]

The CQ bit was checked by bburns@chromium.org

[commit-bot: I haz the power, 2016-05-23 22:42:49]

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

[commit-bot: I haz the power, 2016-05-23 22:47:54]

Description was changed from

==========
add a couple of casts.

BUG=614150
==========

to

==========
add a couple of casts.

BUG=614150
==========

[commit-bot: I haz the power, 2016-05-23 22:47:55]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-23 22:49:39]

Description was changed from

==========
add a couple of casts.

BUG=614150
==========

to

==========
add a couple of casts.

BUG=614150

Committed: https://crrev.com/40c05d919958da304af825c140ee76a8e386c94b
Cr-Commit-Position: refs/heads/master@{#395443}
==========

[commit-bot: I haz the power, 2016-05-23 22:49:40]

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