Remove dead DocumentState metrics tracking code

Remove dead DocumentState metrics tracking code

In https://codereview.chromium.org/2437863005, page_load_histograms,
which was the only consumer of this code, was removed. Now that the
page_load_histograms code has been removed, this code can be removed as
well.

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

Committed: https://crrev.com/49550fef9418c915f8d73ac8ea768d729c0b31c8
Cr-Commit-Position: refs/heads/master@{#429646}

[Bryan McQuade, 2016-10-24 17:44:49]

Description was changed from

==========
Remove dead document_state metrics tracking code

In https://codereview.chromium.org/2437863005, page_load_histograms,
which was the only consumer of this code, was removed. Now that the
page_load_histograms code has been removed, this code can be removed as
well.

BUG=657993
==========

to

==========
Remove dead document_state metrics tracking code

In https://codereview.chromium.org/2437863005, page_load_histograms,
which was the only consumer of this code, was removed. Now that the
page_load_histograms code has been removed, this code can be removed as
well.

Note that the remaining page load metrics in document_state continue
to be referenced from the GetCSI logic in loadtimes_extension_bindings.cc.
Removal of that code is tracked in bug 621512.

BUG=657993
==========

[Bryan McQuade, 2016-10-24 17:45:06]

Description was changed from

==========
Remove dead document_state metrics tracking code

In https://codereview.chromium.org/2437863005, page_load_histograms,
which was the only consumer of this code, was removed. Now that the
page_load_histograms code has been removed, this code can be removed as
well.

Note that the remaining page load metrics in document_state continue
to be referenced from the GetCSI logic in loadtimes_extension_bindings.cc.
Removal of that code is tracked in bug 621512.

BUG=657993
==========

to

==========
Remove dead document_state metrics tracking code

In https://codereview.chromium.org/2437863005, page_load_histograms,
which was the only consumer of this code, was removed. Now that the
page_load_histograms code has been removed, this code can be removed as
well.

Note that the remaining page load metrics in document_state continue
to be referenced from the GetCSI logic in loadtimes_extension_bindings.cc.
Removal of that code is tracked in crbug.com/621512.

BUG=657993
==========

[Bryan McQuade, 2016-10-28 19:31:19]

Description was changed from

==========
Remove dead document_state metrics tracking code

In https://codereview.chromium.org/2437863005, page_load_histograms,
which was the only consumer of this code, was removed. Now that the
page_load_histograms code has been removed, this code can be removed as
well.

Note that the remaining page load metrics in document_state continue
to be referenced from the GetCSI logic in loadtimes_extension_bindings.cc.
Removal of that code is tracked in crbug.com/621512.

BUG=657993
==========

to

==========
Remove dead document_state metrics tracking code

In https://codereview.chromium.org/2437863005, page_load_histograms,
which was the only consumer of this code, was removed. Now that the
page_load_histograms code has been removed, this code can be removed as
well.

Note that the remaining page load metrics in document_state continue
to be referenced from the GetCSI logic in loadtimes_extension_bindings.cc.
Removal of that code is tracked in crbug.com/621512.

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

[Bryan McQuade, 2016-10-28 19:31:24]

The CQ bit was checked by [email protected] to run a CQ dry run

[commit-bot: I haz the power, 2016-10-28 19:32:32]

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-10-28 19:46:17]

The CQ bit was unchecked by [email protected]

[commit-bot: I haz the power, 2016-10-28 19:46:18]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Bryan McQuade, 2016-10-28 19:46:53]

The CQ bit was checked by [email protected] to run a CQ dry run

[commit-bot: I haz the power, 2016-10-28 19:47:42]

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-10-28 20:00:04]

The CQ bit was unchecked by [email protected]

[commit-bot: I haz the power, 2016-10-28 20:00:05]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Bryan McQuade, 2016-10-31 14:51:26]

Description was changed from

==========
Remove dead document_state metrics tracking code

In https://codereview.chromium.org/2437863005, page_load_histograms,
which was the only consumer of this code, was removed. Now that the
page_load_histograms code has been removed, this code can be removed as
well.

Note that the remaining page load metrics in document_state continue
to be referenced from the GetCSI logic in loadtimes_extension_bindings.cc.
Removal of that code is tracked in crbug.com/621512.

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

to

==========
Remove dead document_state metrics tracking code

In https://codereview.chromium.org/2437863005, page_load_histograms,
which was the only consumer of this code, was removed. Now that the
page_load_histograms code has been removed, this code can be removed as
well.

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

[Bryan McQuade, 2016-10-31 18:43:59]

Description was changed from

==========
Remove dead document_state metrics tracking code

In https://codereview.chromium.org/2437863005, page_load_histograms,
which was the only consumer of this code, was removed. Now that the
page_load_histograms code has been removed, this code can be removed as
well.

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

to

==========
Remove dead DocumentState metrics tracking code

In https://codereview.chromium.org/2437863005, page_load_histograms,
which was the only consumer of this code, was removed. Now that the
page_load_histograms code has been removed, this code can be removed as
well.

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

[Bryan McQuade, 2016-11-02 18:08:10]

The CQ bit was checked by [email protected] to run a CQ dry run

[commit-bot: I haz the power, 2016-11-02 18:08:40]

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-11-02 18:56:10]

The CQ bit was unchecked by [email protected]

[commit-bot: I haz the power, 2016-11-02 18:56:11]

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

[Bryan McQuade, 2016-11-02 20:40:54]

The CQ bit was checked by [email protected] to run a CQ dry run

[commit-bot: I haz the power, 2016-11-02 20:41:43]

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

[Bryan McQuade, 2016-11-02 21:57:47]

[email protected] changed reviewers:
+ [email protected], [email protected]

[Bryan McQuade, 2016-11-02 21:58:17]

jochen, PTAL for content/ and third_party/WebKit
rbyers, PTAL for test_runner and third_party/WebKit

Thanks!

[Rick Byers, 2016-11-02 22:10:27]

On 2016/11/02 21:58:17, Bryan McQuade wrote:
> jochen, PTAL for content/ and third_party/WebKit
> rbyers, PTAL for test_runner and third_party/WebKit
> 
> Thanks!

test_runner and WebKit LGTM

[commit-bot: I haz the power, 2016-11-02 22:58:38]

The CQ bit was unchecked by [email protected]

[commit-bot: I haz the power, 2016-11-02 22:58:39]

Dry run: This issue passed the CQ dry run.

[Bryan McQuade, 2016-11-02 23:16:34]

[email protected] changed reviewers:
+ [email protected]

[Bryan McQuade, 2016-11-02 23:16:48]

tsepez, PTAL for 
content/common/frame_messages.h change. Thanks!

[jochen (gone - plz use gerrit), 2016-11-03 11:53:23]

lgtm

[Tom Sepez, 2016-11-03 17:55:04]

messages LGTM.

[Bryan McQuade, 2016-11-03 17:57:06]

The CQ bit was checked by [email protected]

[commit-bot: I haz the power, 2016-11-03 17:57: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-11-03 18:05:17]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2016-11-03 18:14:32]

Description was changed from

==========
Remove dead DocumentState metrics tracking code

In https://codereview.chromium.org/2437863005, page_load_histograms,
which was the only consumer of this code, was removed. Now that the
page_load_histograms code has been removed, this code can be removed as
well.

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

to

==========
Remove dead DocumentState metrics tracking code

In https://codereview.chromium.org/2437863005, page_load_histograms,
which was the only consumer of this code, was removed. Now that the
page_load_histograms code has been removed, this code can be removed as
well.

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

Committed: https://crrev.com/49550fef9418c915f8d73ac8ea768d729c0b31c8
Cr-Commit-Position: refs/heads/master@{#429646}
==========

[commit-bot: I haz the power, 2016-11-03 18:14:34]

Patchset 8 (id:??) landed as
https://crrev.com/49550fef9418c915f8d73ac8ea768d729c0b31c8
Cr-Commit-Position: refs/heads/master@{#429646}