Add page load metrics for no-store forward back navigations.

Add page load metricsfor no-store forward back navigations.

This patch adds support in NavigationHandle to retrieve http response
headers. This data is used by the page_load_metrics system to separate
out histograms for no-store back/forward navigations.

BUG=631309
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

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

[Charlie Harrison, 2016-07-26 00:33:25]

Description was changed from

==========
Add page load metricsfor no-store forward back navigations.

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

to

==========
Add page load metricsfor no-store forward back navigations.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[Charlie Harrison, 2016-07-26 00:33:29]

The CQ bit was checked by csharrison@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-26 00:34:04]

Dry run: 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-07-26 00:36:36]

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

[commit-bot: I haz the power, 2016-07-26 00:36:37]

Dry run: 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-gn on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-gn/bui...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-gn on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-gn/...)

[Charlie Harrison, 2016-07-26 01:22:07]

The CQ bit was checked by csharrison@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-26 01:22:30]

Dry run: 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-07-26 01:57:22]

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

[commit-bot: I haz the power, 2016-07-26 01:57:23]

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

[Charlie Harrison, 2016-07-26 03:04:45]

The CQ bit was checked by csharrison@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-26 03:05:05]

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

[Charlie Harrison, 2016-07-26 03:51:46]

Description was changed from

==========
Add page load metricsfor no-store forward back navigations.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Add page load metricsfor no-store forward back navigations.

This patch adds support in NavigationHandle to retrieve http response
headers. This data is used by the page_load_metrics system to separate
out histograms for no-store back/forward navigations.

BUG=631309
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[commit-bot: I haz the power, 2016-07-26 03:57:04]

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

[commit-bot: I haz the power, 2016-07-26 03:57:05]

Dry run: This issue passed the CQ dry run.

[Charlie Harrison, 2016-08-02 17:57:40]

The CQ bit was checked by csharrison@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-02 17:58:19]

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

[Charlie Harrison, 2016-08-02 18:02:34]

csharrison@chromium.org changed reviewers:
+ nasko@chromium.org

[Charlie Harrison, 2016-08-02 18:02:35]

cc creis

Here's the review plumbing const http headers into the content public api for
NavigationHandle, as we talked about over email.

Does it look okay to you?

[commit-bot: I haz the power, 2016-08-02 19:23:36]

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

[commit-bot: I haz the power, 2016-08-02 19:23:37]

Dry run: This issue passed the CQ dry run.

[nasko, 2016-08-03 16:51:26]

content/ LGTM

[Charlie Harrison, 2016-08-03 17:00:32]

csharrison@chromium.org changed reviewers:
+ holte@chromium.org, shivanisha@chromium.org

[Charlie Harrison, 2016-08-03 17:00:33]

Thanks nasko!

+shivanisha for PLM and holte@ for histograms.xml

[Steven Holte, 2016-08-03 19:07:41]

lgtm

[shivanisha, 2016-08-04 19:10:30]

On 2016/08/03 at 19:07:41, holte wrote:
> lgtm

lgtm

[Charlie Harrison, 2016-08-04 19:11:21]

Thanks, all.

[Charlie Harrison, 2016-08-04 19:12:44]

The CQ bit was checked by csharrison@chromium.org

[commit-bot: I haz the power, 2016-08-04 19:13:42]

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-08-04 21:45:22]

Description was changed from

==========
Add page load metricsfor no-store forward back navigations.

This patch adds support in NavigationHandle to retrieve http response
headers. This data is used by the page_load_metrics system to separate
out histograms for no-store back/forward navigations.

BUG=631309
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Add page load metricsfor no-store forward back navigations.

This patch adds support in NavigationHandle to retrieve http response
headers. This data is used by the page_load_metrics system to separate
out histograms for no-store back/forward navigations.

BUG=631309
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[commit-bot: I haz the power, 2016-08-04 21:45:23]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-08-04 21:46:59]

Description was changed from

==========
Add page load metricsfor no-store forward back navigations.

This patch adds support in NavigationHandle to retrieve http response
headers. This data is used by the page_load_metrics system to separate
out histograms for no-store back/forward navigations.

BUG=631309
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Add page load metricsfor no-store forward back navigations.

This patch adds support in NavigationHandle to retrieve http response
headers. This data is used by the page_load_metrics system to separate
out histograms for no-store back/forward navigations.

BUG=631309
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/be78b7813aa8753f94871792fb0e80090a535c89
Cr-Commit-Position: refs/heads/master@{#409907}
==========

[commit-bot: I haz the power, 2016-08-04 21:47:00]

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