Use uint64_t instead of size_t inside the web cache manager code.

Use uint64_t instead of size_t inside the web cache manager code.

size_t is not sufficient when the browser is 32 bit and the renderers are 64 bits.

BUG=581409, 585382
Committed: https://crrev.com/31963689d7f4b2242d4630b34b93852ed03e1dc3
Cr-Commit-Position: refs/heads/master@{#374760}

[jam, 2016-02-10 15:43:56]

Description was changed from

==========
tbd

BUG=
==========

to

==========
Use uint64_t instead of size_t inside the web cache manager code.

size_t is not sufficient when the browser is 32 bit and the renderers are 64
bits.

BUG=581409,585382
==========

[jam, 2016-02-10 15:43:56]

jam@chromium.org changed reviewers:
+ sky@chromium.org, tsepez@chromium.org

[jam, 2016-02-10 15:44:18]

Tom: render_messages.*
Scott: rest

[sky, 2016-02-10 17:05:57]

LGTM

https://codereview.chromium.org/1685883004/diff/20001/components/web_cache/br...
File components/web_cache/browser/web_cache_manager_unittest.cc (right):

https://codereview.chromium.org/1685883004/diff/20001/components/web_cache/br...
components/web_cache/browser/web_cache_manager_unittest.cc:91: const
WebCacheManager::RendererInfo& rhs) {
nit: run git cl format

https://codereview.chromium.org/1685883004/diff/20001/components/web_cache/re...
File components/web_cache/renderer/web_cache_render_process_observer.cc (right):

https://codereview.chromium.org/1685883004/diff/20001/components/web_cache/re...
components/web_cache/renderer/web_cache_render_process_observer.cc:64: uint64_t
min_dead_capacity64,
Did you intend to name these with 64 in the name? If so, update the header to
match.

[jam, 2016-02-10 17:52:10]

https://codereview.chromium.org/1685883004/diff/20001/components/web_cache/br...
File components/web_cache/browser/web_cache_manager_unittest.cc (right):

https://codereview.chromium.org/1685883004/diff/20001/components/web_cache/br...
components/web_cache/browser/web_cache_manager_unittest.cc:91: const
WebCacheManager::RendererInfo& rhs) {
On 2016/02/10 17:05:57, sky wrote:
> nit: run git cl format

Done.

https://codereview.chromium.org/1685883004/diff/20001/components/web_cache/re...
File components/web_cache/renderer/web_cache_render_process_observer.cc (right):

https://codereview.chromium.org/1685883004/diff/20001/components/web_cache/re...
components/web_cache/renderer/web_cache_render_process_observer.cc:64: uint64_t
min_dead_capacity64,
On 2016/02/10 17:05:57, sky wrote:
> Did you intend to name these with 64 in the name? If so, update the header to
> match.

Done.

[Tom Sepez, 2016-02-10 17:54:56]

lgtm

[jam, 2016-02-10 17:59:22]

The CQ bit was checked by jam@chromium.org

[jam, 2016-02-10 17:59:23]

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

[commit-bot: I haz the power, 2016-02-10 18:00:09]

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

[commit-bot: I haz the power, 2016-02-10 20:42:54]

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

[commit-bot: I haz the power, 2016-02-10 20:42:55]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[jam, 2016-02-10 21:34:09]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2016-02-10 21:38:55]

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

[commit-bot: I haz the power, 2016-02-10 22:55:10]

Description was changed from

==========
Use uint64_t instead of size_t inside the web cache manager code.

size_t is not sufficient when the browser is 32 bit and the renderers are 64
bits.

BUG=581409,585382
==========

to

==========
Use uint64_t instead of size_t inside the web cache manager code.

size_t is not sufficient when the browser is 32 bit and the renderers are 64
bits.

BUG=581409,585382
==========

[commit-bot: I haz the power, 2016-02-10 22:55:11]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-02-16 22:31:59]

Description was changed from

==========
Use uint64_t instead of size_t inside the web cache manager code.

size_t is not sufficient when the browser is 32 bit and the renderers are 64
bits.

BUG=581409,585382
==========

to

==========
Use uint64_t instead of size_t inside the web cache manager code.

size_t is not sufficient when the browser is 32 bit and the renderers are 64
bits.

BUG=581409,585382

Committed: https://crrev.com/31963689d7f4b2242d4630b34b93852ed03e1dc3
Cr-Commit-Position: refs/heads/master@{#374760}
==========

[commit-bot: I haz the power, 2016-02-16 22:32:00]

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