[Prerender] Add a "SimpleLoad" mode to be used as a comparison reference

[Prerender] Add a "SimpleLoad" mode to be used as a comparison reference

This CL introduces the SIMPLE_LOAD_EXPERIMENT prerender mode.
In this mode, no prerendering or prefetching happens, but the URLs that
would have been prefetched are recorded, and UMA metrics are reported.

This will be used as a reference point to evaluate the speed
improvements of prerendering and prefetching.

For now both the simple load mode and the prefetch mode record
the FCP histograms.
Prerender will have to be updated to record these histograms as
well in another CL.

BUG=668997
Committed: https://crrev.com/88ffbf898da32332e9e592954d45ddccffbe89a1
Cr-Commit-Position: refs/heads/master@{#434633}

[droger, 2016-11-25 15:00:10]

Patchset #1 (id:1) has been deleted

[droger, 2016-11-25 16:25:02]

Patchset #1 (id:20001) has been deleted

[droger, 2016-11-25 16:26:09]

Description was changed from

==========
[Prerender] Add a "SimpleLoad" mode to be used as a comparison reference

This CL introduces the SIMPLE_LOAD_EXPERIMENT prerender mode.
In this mode, no prerendering or prefetching happens, but the URLs that
would have been prefetched are recorded, and UMA metrics are reported.

This will be used as a reference point to evaluate the speed
improvements of prerendering and prefetching.
==========

to

==========
[Prerender] Add a "SimpleLoad" mode to be used as a comparison reference

This CL introduces the SIMPLE_LOAD_EXPERIMENT prerender mode.
In this mode, no prerendering or prefetching happens, but the URLs that
would have been prefetched are recorded, and UMA metrics are reported.

This will be used as a reference point to evaluate the speed
improvements of prerendering and prefetching.

For now both the simple load mode and the prefetch mode record
the FCP histograms.
Prerender will have to be updated to record these histograms as
well in another CL.
==========

[droger, 2016-11-25 16:26:37]

droger@chromium.org changed reviewers:
+ mattcary@chromium.org, pasko@chromium.org

[mattcary, 2016-11-28 09:28:23]

This still won't quite do what we want, because the nostate observer isn't
attached to the webcontents if we're prerendering?

[droger, 2016-11-28 09:47:40]

On 2016/11/28 09:28:23, mattcary wrote:
> This still won't quite do what we want, because the nostate observer isn't
> attached to the webcontents if we're prerendering?

Yes, this does not handle the prerender case.
Only prefetch and control.

To handle the prerender, we need your CL or something similar to land, so that
we can track perceived page load times for prerenders.

[mattcary, 2016-11-28 09:48:19]

lgtm

Agreed.

[pasko, 2016-11-28 10:26:49]

lgtm

the reference to 'nostate observer' was a little cryptic. I have read your guys'
discussion and now I think my theories about it were right, but it will become
cryptic again after a month or two.

So I guess it might be useful some time later to mention now that the words
'your CL' are pointing at this change:
https://codereview.chromium.org/2423383002/

Let me know if I misunderstand it all. There is a chance of it happening.

[mattcary, 2016-11-28 10:28:17]

On 2016/11/28 10:26:49, pasko wrote:
> lgtm
> 
> the reference to 'nostate observer' was a little cryptic. I have read your
guys'
> discussion and now I think my theories about it were right, but it will become
> cryptic again after a month or two.
> 
> So I guess it might be useful some time later to mention now that the words
> 'your CL' are pointing at this change:
> https://codereview.chromium.org/2423383002/
> 
> Let me know if I misunderstand it all. There is a chance of it happening.

That's correct, thanks for the clarification.

[droger, 2016-11-28 10:31:34]

Description was changed from

==========
[Prerender] Add a "SimpleLoad" mode to be used as a comparison reference

This CL introduces the SIMPLE_LOAD_EXPERIMENT prerender mode.
In this mode, no prerendering or prefetching happens, but the URLs that
would have been prefetched are recorded, and UMA metrics are reported.

This will be used as a reference point to evaluate the speed
improvements of prerendering and prefetching.

For now both the simple load mode and the prefetch mode record
the FCP histograms.
Prerender will have to be updated to record these histograms as
well in another CL.
==========

to

==========
[Prerender] Add a "SimpleLoad" mode to be used as a comparison reference

This CL introduces the SIMPLE_LOAD_EXPERIMENT prerender mode.
In this mode, no prerendering or prefetching happens, but the URLs that
would have been prefetched are recorded, and UMA metrics are reported.

This will be used as a reference point to evaluate the speed
improvements of prerendering and prefetching.

For now both the simple load mode and the prefetch mode record
the FCP histograms.
Prerender will have to be updated to record these histograms as
well in another CL.

BUG=668997
==========

[droger, 2016-11-28 10:37:31]

Thanks for the reviews, and sorry for the sloppy comments.

[droger, 2016-11-28 10:37:51]

The CQ bit was checked by droger@chromium.org

[commit-bot: I haz the power, 2016-11-28 10:38:10]

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, 2016-11-28 11:19:33]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1480329471906990,
"parent_rev": "1b47f0574e2b460123f05cf32d4b6dda12f44716", "commit_rev":
"89efc7467dd9e0dc7e1f3b4292e1a466af9ee12c"}

[commit-bot: I haz the power, 2016-11-28 11:19:56]

Description was changed from

==========
[Prerender] Add a "SimpleLoad" mode to be used as a comparison reference

This CL introduces the SIMPLE_LOAD_EXPERIMENT prerender mode.
In this mode, no prerendering or prefetching happens, but the URLs that
would have been prefetched are recorded, and UMA metrics are reported.

This will be used as a reference point to evaluate the speed
improvements of prerendering and prefetching.

For now both the simple load mode and the prefetch mode record
the FCP histograms.
Prerender will have to be updated to record these histograms as
well in another CL.

BUG=668997
==========

to

==========
[Prerender] Add a "SimpleLoad" mode to be used as a comparison reference

This CL introduces the SIMPLE_LOAD_EXPERIMENT prerender mode.
In this mode, no prerendering or prefetching happens, but the URLs that
would have been prefetched are recorded, and UMA metrics are reported.

This will be used as a reference point to evaluate the speed
improvements of prerendering and prefetching.

For now both the simple load mode and the prefetch mode record
the FCP histograms.
Prerender will have to be updated to record these histograms as
well in another CL.

BUG=668997
==========

[commit-bot: I haz the power, 2016-11-28 11:19:57]

Committed patchset #1 (id:40001)

[commit-bot: I haz the power, 2016-11-28 11:21:46]

Description was changed from

==========
[Prerender] Add a "SimpleLoad" mode to be used as a comparison reference

This CL introduces the SIMPLE_LOAD_EXPERIMENT prerender mode.
In this mode, no prerendering or prefetching happens, but the URLs that
would have been prefetched are recorded, and UMA metrics are reported.

This will be used as a reference point to evaluate the speed
improvements of prerendering and prefetching.

For now both the simple load mode and the prefetch mode record
the FCP histograms.
Prerender will have to be updated to record these histograms as
well in another CL.

BUG=668997
==========

to

==========
[Prerender] Add a "SimpleLoad" mode to be used as a comparison reference

This CL introduces the SIMPLE_LOAD_EXPERIMENT prerender mode.
In this mode, no prerendering or prefetching happens, but the URLs that
would have been prefetched are recorded, and UMA metrics are reported.

This will be used as a reference point to evaluate the speed
improvements of prerendering and prefetching.

For now both the simple load mode and the prefetch mode record
the FCP histograms.
Prerender will have to be updated to record these histograms as
well in another CL.

BUG=668997

Committed: https://crrev.com/88ffbf898da32332e9e592954d45ddccffbe89a1
Cr-Commit-Position: refs/heads/master@{#434633}
==========

[commit-bot: I haz the power, 2016-11-28 11:21:47]

Patchset 1 (id:??) landed as
https://crrev.com/88ffbf898da32332e9e592954d45ddccffbe89a1
Cr-Commit-Position: refs/heads/master@{#434633}