precache: Add geometric resource weight function.

precache: Add geometric resource weight function.

Add an alternative function for combining resource weight with host
visit frequency. This one is a measure of the probability that there
will be a cache hit for the resource, given that resource_weight is the
probability of a cache hit for a single trial and host_visits is the
number of independent trials.

BUG=

Review-Url: https://codereview.chromium.org/2623553003
Cr-Commit-Position: refs/heads/master@{#443464}
Committed: https://chromium.googlesource.com/chromium/src/+/591b347c5ffa6c9841a0ae9e08e28b3820ed9641

[twifkak, 2017-01-10 02:12:47]

twifkak@chromium.org changed reviewers:
+ jamartin@chromium.org

[twifkak, 2017-01-11 22:25:08]

twifkak@chromium.org changed reviewers:
+ bengr@chromium.org
- jamartin@chromium.org

[twifkak, 2017-01-11 22:25:09]

bengr: PTAL. jamartin reviewed over IM but due to login issues was unable to
leave comments here.

[bengr, 2017-01-12 18:48:02]

lgtm modulo requests for comments.

https://codereview.chromium.org/2623553003/diff/20001/components/precache/cor...
File components/precache/core/precache_fetcher.cc (right):

https://codereview.chromium.org/2623553003/diff/20001/components/precache/cor...
components/precache/core/precache_fetcher.cc:231: // Models the expected number
of requests for the resource.
Please describe the parameters here and at lines 236 and 244.

[twifkak, 2017-01-12 20:53:39]

Thanks! Submitting.

https://codereview.chromium.org/2623553003/diff/20001/components/precache/cor...
File components/precache/core/precache_fetcher.cc (right):

https://codereview.chromium.org/2623553003/diff/20001/components/precache/cor...
components/precache/core/precache_fetcher.cc:231: // Models the expected number
of requests for the resource.
On 2017/01/12 18:48:01, bengr wrote:
> Please describe the parameters here and at lines 236 and 244.

Done.

[twifkak, 2017-01-12 20:53:45]

The CQ bit was checked by twifkak@chromium.org

[twifkak, 2017-01-12 20:53:45]

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

[commit-bot: I haz the power, 2017-01-12 20:54:31]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-12 20:57:25]

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

[commit-bot: I haz the power, 2017-01-12 20:57:26]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[twifkak, 2017-01-12 20:59:30]

The CQ bit was checked by twifkak@chromium.org

[twifkak, 2017-01-12 20:59:30]

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

[commit-bot: I haz the power, 2017-01-12 21:00:12]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-12 21:29:53]

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

[commit-bot: I haz the power, 2017-01-12 21:29:54]

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

[twifkak, 2017-01-12 23:12:10]

The CQ bit was checked by twifkak@chromium.org

[twifkak, 2017-01-12 23:12:11]

The patchset sent to the CQ was uploaded after l-g-t-m from bengr@chromium.org
Link to the patchset: https://codereview.chromium.org/2623553003/#ps80001
(title: "Add default case, needed by some compilers.")

[commit-bot: I haz the power, 2017-01-12 23:12:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-13 02:14:09]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1484262730781390,
"parent_rev": "744bfc85a783e50ac5ae830fcdd5a9204df60d66", "commit_rev":
"591b347c5ffa6c9841a0ae9e08e28b3820ed9641"}

[commit-bot: I haz the power, 2017-01-13 02:14:38]

Description was changed from

==========
precache: Add geometric resource weight function.

Add an alternative function for combining resource weight with host
visit frequency. This one is a measure of the probability that there
will be a cache hit for the resource, given that resource_weight is the
probability of a cache hit for a single trial and host_visits is the
number of independent trials.

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

to

==========
precache: Add geometric resource weight function.

Add an alternative function for combining resource weight with host
visit frequency. This one is a measure of the probability that there
will be a cache hit for the resource, given that resource_weight is the
probability of a cache hit for a single trial and host_visits is the
number of independent trials.

BUG=

Review-Url: https://codereview.chromium.org/2623553003
Cr-Commit-Position: refs/heads/master@{#443464}
Committed:
https://chromium.googlesource.com/chromium/src/+/591b347c5ffa6c9841a0ae9e08e2...
==========

[commit-bot: I haz the power, 2017-01-13 02:14:39]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/591b347c5ffa6c9841a0ae9e08e2...