Let blink scheduler know when background parsing a document

Let blink scheduler know when background parsing a document

This is necessary to make Virtual time behave deterministically.
Without knowing that a document is being parsed in the background
we might think loading is safe to run timers (which it isn't if
you want determinism).

BUG=546953
Committed: https://crrev.com/1d65ae16d48a6639cae89b64f032b47e31c82c7e
Cr-Commit-Position: refs/heads/master@{#408668}

[alex clarke (OOO till 29th), 2016-07-27 15:25:35]

alexclarke@chromium.org changed reviewers:
+ mkwst@chromium.org, skyostil@chromium.org

[Sami, 2016-07-27 16:45:10]

lgtm.

[Mike West, 2016-07-29 13:49:09]

//third_party/WebKit LGTM % two naming nits.

https://codereview.chromium.org/2192493002/diff/1/third_party/WebKit/public/p...
File third_party/WebKit/public/platform/WebFrameScheduler.h (right):

https://codereview.chromium.org/2192493002/diff/1/third_party/WebKit/public/p...
third_party/WebKit/public/platform/WebFrameScheduler.h:59: virtual void
setDocumentParsedInBackground(bool) { }
Nit: "parsed" reads to me as "a document has finished parsing". If this is meant
to represent the parser actively working its way through a document, perhaps
`setDocumentParsingInBackground` is a better description?

https://codereview.chromium.org/2192493002/diff/1/third_party/WebKit/public/p...
File third_party/WebKit/public/platform/WebViewScheduler.h (right):

https://codereview.chromium.org/2192493002/diff/1/third_party/WebKit/public/p...
third_party/WebKit/public/platform/WebViewScheduler.h:58:
PAUSE_IF_NETWORK_FETCHES_PENDING
Nit: Is there a name for this enum that could represent the state you're now
checking?

[alex clarke (OOO till 29th), 2016-07-29 14:07:11]

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

[alex clarke (OOO till 29th), 2016-07-29 14:07:27]

https://codereview.chromium.org/2192493002/diff/1/third_party/WebKit/public/p...
File third_party/WebKit/public/platform/WebFrameScheduler.h (right):

https://codereview.chromium.org/2192493002/diff/1/third_party/WebKit/public/p...
third_party/WebKit/public/platform/WebFrameScheduler.h:59: virtual void
setDocumentParsedInBackground(bool) { }
On 2016/07/29 13:49:08, Mike West wrote:
> Nit: "parsed" reads to me as "a document has finished parsing". If this is
meant
> to represent the parser actively working its way through a document, perhaps
> `setDocumentParsingInBackground` is a better description?

Good point. Done.

https://codereview.chromium.org/2192493002/diff/1/third_party/WebKit/public/p...
File third_party/WebKit/public/platform/WebViewScheduler.h (right):

https://codereview.chromium.org/2192493002/diff/1/third_party/WebKit/public/p...
third_party/WebKit/public/platform/WebViewScheduler.h:58:
PAUSE_IF_NETWORK_FETCHES_PENDING
On 2016/07/29 13:49:08, Mike West wrote:
> Nit: Is there a name for this enum that could represent the state you're now
> checking?

Done.

[commit-bot: I haz the power, 2016-07-29 14:07:33]

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-29 14:15:41]

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

[commit-bot: I haz the power, 2016-07-29 14:15:42]

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_...)

[alex clarke (OOO till 29th), 2016-07-29 15:03:52]

The CQ bit was checked by alexclarke@chromium.org

[alex clarke (OOO till 29th), 2016-07-29 15:03:52]

The patchset sent to the CQ was uploaded after l-g-t-m from mkwst@chromium.org,
skyostil@chromium.org
Link to the patchset: https://codereview.chromium.org/2192493002/#ps40001
(title: "Missed a couple of things to rename")

[commit-bot: I haz the power, 2016-07-29 15:04:06]

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-29 15:47:14]

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

[commit-bot: I haz the power, 2016-07-29 15:47:16]

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_...)

[alex clarke (OOO till 29th), 2016-07-29 15:55:15]

The CQ bit was checked by alexclarke@chromium.org

[alex clarke (OOO till 29th), 2016-07-29 15:55:15]

The patchset sent to the CQ was uploaded after l-g-t-m from mkwst@chromium.org,
skyostil@chromium.org
Link to the patchset: https://codereview.chromium.org/2192493002/#ps60001
(title: "Fix some test failures")

[commit-bot: I haz the power, 2016-07-29 15:55:32]

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-29 17:11:05]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-07-29 17:12:22]

Description was changed from

==========
Let blink scheduler know when background parsing a document

This is necessary to make Virtual time behave deterministically.
Without knowing that a document is being parsed in the background
we might think loading is safe to run timers (which it isn't if
you want determinism).

BUG=546953
==========

to

==========
Let blink scheduler know when background parsing a document

This is necessary to make Virtual time behave deterministically.
Without knowing that a document is being parsed in the background
we might think loading is safe to run timers (which it isn't if
you want determinism).

BUG=546953

Committed: https://crrev.com/1d65ae16d48a6639cae89b64f032b47e31c82c7e
Cr-Commit-Position: refs/heads/master@{#408668}
==========

[commit-bot: I haz the power, 2016-07-29 17:12:24]

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