[Page Load Metrics] Structure PageLoadTiming.

[Page Load Metrics] Structure PageLoadTiming.

Add another 3 categories to PageLoadTiming to group different types of timing,
which are DocumentTiming, ParseTiming and PaintTiming.

DocumentTiming includes:
* dom_content_loaded_event_start
* load_event_start
* first_layout

ParseTiming includes:
* parse_start
* parse_stop
* parse_blocked_on_script_load_duration
* parse_blocked_on_script_load_from_document_write_duration
* parse_blocked_on_script_execution_duration
* parse_blocked_on_script_execution_from_document_write_duration

PaintTiming includes:
* first_paint
* first_text_paint
* first_image_paint
* first_contentful_paint
* first_meaningful_paint

BUG=711351
Review-Url: https://codereview.chromium.org/2806863003
Cr-Commit-Position: refs/heads/master@{#464693}
Committed: https://chromium.googlesource.com/chromium/src/+/e95b6b3993dedab3ee364fa591ad032ab243fa6a

[lpy, 2017-04-07 23:40:46]

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

[commit-bot: I haz the power, 2017-04-07 23:41: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, 2017-04-08 00:07:16]

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

[commit-bot: I haz the power, 2017-04-08 00:07:17]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[lpy, 2017-04-10 19:37:38]

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

[commit-bot: I haz the power, 2017-04-10 19:38: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, 2017-04-10 20:05:19]

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

[commit-bot: I haz the power, 2017-04-10 20:05:20]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[lpy, 2017-04-10 20:13:48]

Patchset #2 (id:20001) has been deleted

[lpy, 2017-04-10 20:15:10]

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

[commit-bot: I haz the power, 2017-04-10 20:15:59]

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, 2017-04-10 20:44:17]

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

[commit-bot: I haz the power, 2017-04-10 20:44:18]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[lpy, 2017-04-10 20:49:42]

Patchset #2 (id:40001) has been deleted

[lpy, 2017-04-10 20:51:27]

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

[commit-bot: I haz the power, 2017-04-10 20:52:19]

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, 2017-04-10 21:39:12]

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

[commit-bot: I haz the power, 2017-04-10 21:39:13]

Dry run: This issue passed the CQ dry run.

[lpy, 2017-04-10 21:44:11]

lpy@chromium.org changed reviewers:
+ bmcquade@chromium.org, zhenw@chromium.org

[lpy, 2017-04-10 21:44:12]

PTAL.

[lpy, 2017-04-11 18:29:51]

On 2017/04/10 21:44:12, lpy wrote:
> PTAL.

gentle ping for review.

[Bryan McQuade, 2017-04-12 01:37:20]

LGTM. Thanks so much for making this change! This has been something I've wanted
us to do for a long time.

I think this may help with the mojo work as well, as we can perhaps also move to
a model in the longer term where we don't send a whole PageLoadTiming and can
instead send one of the individual timing objects as needed. It's up to you, but
I could see that being potentially valuable.

[lpy, 2017-04-12 02:50:34]

The CQ bit was checked by lpy@chromium.org

[commit-bot: I haz the power, 2017-04-12 02:50:51]

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-04-12 03:00:08]

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

[commit-bot: I haz the power, 2017-04-12 03:00:09]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[lpy, 2017-04-12 18:50:27]

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

[lpy, 2017-04-12 18:51:00]

lpy@chromium.org changed reviewers:
+ pasko@chromium.org

[lpy, 2017-04-12 18:51:01]

+pasko@ for prerender/

PTAL.

[commit-bot: I haz the power, 2017-04-12 18:51:06]

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, 2017-04-12 19:44:35]

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

[commit-bot: I haz the power, 2017-04-12 19:44:36]

Dry run: This issue passed the CQ dry run.

[lpy, 2017-04-12 19:44:53]

The CQ bit was checked by lpy@chromium.org

[lpy, 2017-04-12 19:44:54]

The patchset sent to the CQ was uploaded after l-g-t-m from
bmcquade@chromium.org
Link to the patchset: https://codereview.chromium.org/2806863003/#ps80001
(title: "rebase")

[commit-bot: I haz the power, 2017-04-12 19:46:22]

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-04-12 20:01:10]

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

[commit-bot: I haz the power, 2017-04-12 20:01:11]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[pasko, 2017-04-13 12:37:42]

chrome/browser/prerender/ lgtm

[lpy, 2017-04-13 16:58:31]

The CQ bit was checked by lpy@chromium.org

[commit-bot: I haz the power, 2017-04-13 16:58:59]

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-04-13 17:08:09]

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

[commit-bot: I haz the power, 2017-04-13 17:08:10]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[lpy, 2017-04-13 17:11:34]

lpy@chromium.org changed reviewers:
+ mkwst@chromium.org

[lpy, 2017-04-13 17:11:35]

+mkwst@, ptal at page_load_metrics_messages

[lpy, 2017-04-13 17:15:52]

Description was changed from

==========
[Page Load Metrics] Structure PageLoadTiming.

Add another 3 categories to PageLoadTiming to group different types of timing,
which are DocumentTiming, ParseTiming and PaintTiming.

DocumentTiming includes:
* dom_content_loaded_event_start
* load_event_start
* first_layout

ParseTiming includes:
* parse_start
* parse_stop
* parse_blocked_on_script_load_duration
* parse_blocked_on_script_load_from_document_write_duration
* parse_blocked_on_script_execution_duration
* parse_blocked_on_script_execution_from_document_write_duration

PaintTiming includes:
* first_paint
* first_text_paint
* first_image_paint
* first_contentful_paint
* first_meaningful_paint
==========

to

==========
[Page Load Metrics] Structure PageLoadTiming.

Add another 3 categories to PageLoadTiming to group different types of timing,
which are DocumentTiming, ParseTiming and PaintTiming.

DocumentTiming includes:
* dom_content_loaded_event_start
* load_event_start
* first_layout

ParseTiming includes:
* parse_start
* parse_stop
* parse_blocked_on_script_load_duration
* parse_blocked_on_script_load_from_document_write_duration
* parse_blocked_on_script_execution_duration
* parse_blocked_on_script_execution_from_document_write_duration

PaintTiming includes:
* first_paint
* first_text_paint
* first_image_paint
* first_contentful_paint
* first_meaningful_paint

BUG=711351
==========

[Mike West, 2017-04-14 06:01:44]

IPC LGTM.

[lpy, 2017-04-14 06:02:13]

The CQ bit was checked by lpy@chromium.org

[commit-bot: I haz the power, 2017-04-14 06:02:34]

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-04-14 06:47:39]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1492149733211290,
"parent_rev": "923b381b904757ab4c489555759d5b8637f8d8e9", "commit_rev":
"e95b6b3993dedab3ee364fa591ad032ab243fa6a"}

[commit-bot: I haz the power, 2017-04-14 06:48:31]

Description was changed from

==========
[Page Load Metrics] Structure PageLoadTiming.

Add another 3 categories to PageLoadTiming to group different types of timing,
which are DocumentTiming, ParseTiming and PaintTiming.

DocumentTiming includes:
* dom_content_loaded_event_start
* load_event_start
* first_layout

ParseTiming includes:
* parse_start
* parse_stop
* parse_blocked_on_script_load_duration
* parse_blocked_on_script_load_from_document_write_duration
* parse_blocked_on_script_execution_duration
* parse_blocked_on_script_execution_from_document_write_duration

PaintTiming includes:
* first_paint
* first_text_paint
* first_image_paint
* first_contentful_paint
* first_meaningful_paint

BUG=711351
==========

to

==========
[Page Load Metrics] Structure PageLoadTiming.

Add another 3 categories to PageLoadTiming to group different types of timing,
which are DocumentTiming, ParseTiming and PaintTiming.

DocumentTiming includes:
* dom_content_loaded_event_start
* load_event_start
* first_layout

ParseTiming includes:
* parse_start
* parse_stop
* parse_blocked_on_script_load_duration
* parse_blocked_on_script_load_from_document_write_duration
* parse_blocked_on_script_execution_duration
* parse_blocked_on_script_execution_from_document_write_duration

PaintTiming includes:
* first_paint
* first_text_paint
* first_image_paint
* first_contentful_paint
* first_meaningful_paint

BUG=711351

Review-Url: https://codereview.chromium.org/2806863003
Cr-Commit-Position: refs/heads/master@{#464693}
Committed:
https://chromium.googlesource.com/chromium/src/+/e95b6b3993dedab3ee364fa591ad...
==========

[commit-bot: I haz the power, 2017-04-14 06:48:33]

Committed patchset #3 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/e95b6b3993dedab3ee364fa591ad...