Add an experiment group via Settings to tokenize synchronously

Add an experiment group via Settings to tokenize synchronously

This patch adds an experiment group that tokenizes synchronously
on the main thread in response to a recieve data IPC.

If this regresses performance, we can augment the CL slightly
to only synchronously tokenize when the main parser has no
work.

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

Committed: https://crrev.com/2fcf4bab78301b1c7b64b1b47547e7e5a70da65f
Cr-Commit-Position: refs/heads/master@{#405035}

[Charlie Harrison, 2016-07-11 18:31:54]

Description was changed from

==========
Add an experiment group via Settings to tokenize synchronously


Clean up TODOs in TestNavigationManager

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

to

==========
Add an experiment group via Settings to tokenize synchronously


Clean up TODOs in TestNavigationManager

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

[Charlie Harrison, 2016-07-11 19:01:47]

Description was changed from

==========
Add an experiment group via Settings to tokenize synchronously


Clean up TODOs in TestNavigationManager

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

to

==========
Add an experiment group via Settings to tokenize synchronously

This patch adds an experiment group that tokenizes synchronously
on the main thread in response to a recieve data IPC.

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

[Charlie Harrison, 2016-07-11 19:02:28]

Description was changed from

==========
Add an experiment group via Settings to tokenize synchronously

This patch adds an experiment group that tokenizes synchronously
on the main thread in response to a recieve data IPC.

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

to

==========
Add an experiment group via Settings to tokenize synchronously

This patch adds an experiment group that tokenizes synchronously
on the main thread in response to a recieve data IPC.

If this regresses performance, we can augment the CL slightly
to only synchronously tokenize when the main parser has no
work.

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

[Charlie Harrison, 2016-07-11 19:03:00]

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

[Charlie Harrison, 2016-07-11 19:03:00]

PTAL!

[kouhei (in TOK), 2016-07-12 01:38:52]

kouhei@chromium.org changed reviewers:
+ kinuko@chromium.org

[kouhei (in TOK), 2016-07-12 01:38:52]

lgtm
+kinuko: Would you take a look at content/

[kinuko, 2016-07-12 02:37:49]

Looks like this can be landed without my stamp, but lgtm anyways

[Charlie Harrison, 2016-07-12 03:00:55]

Thanks!

[Charlie Harrison, 2016-07-12 03:01:02]

The CQ bit was checked by csharrison@chromium.org

[commit-bot: I haz the power, 2016-07-12 03:03: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-12 06:36:07]

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

[commit-bot: I haz the power, 2016-07-12 06:36:08]

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

[Charlie Harrison, 2016-07-12 12:17:17]

The CQ bit was checked by csharrison@chromium.org

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

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-12 15:40:32]

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

[commit-bot: I haz the power, 2016-07-12 15:40:33]

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

[Charlie Harrison, 2016-07-12 19:03:57]

The CQ bit was checked by csharrison@chromium.org

[commit-bot: I haz the power, 2016-07-12 19:05:55]

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-12 21:23:29]

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

[commit-bot: I haz the power, 2016-07-12 21:23:30]

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

[Charlie Harrison, 2016-07-13 03:16:49]

The CQ bit was checked by csharrison@chromium.org

[commit-bot: I haz the power, 2016-07-13 03:18:45]

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-13 05:03:51]

Description was changed from

==========
Add an experiment group via Settings to tokenize synchronously

This patch adds an experiment group that tokenizes synchronously
on the main thread in response to a recieve data IPC.

If this regresses performance, we can augment the CL slightly
to only synchronously tokenize when the main parser has no
work.

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

to

==========
Add an experiment group via Settings to tokenize synchronously

This patch adds an experiment group that tokenizes synchronously
on the main thread in response to a recieve data IPC.

If this regresses performance, we can augment the CL slightly
to only synchronously tokenize when the main parser has no
work.

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

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

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-13 05:06:07]

Description was changed from

==========
Add an experiment group via Settings to tokenize synchronously

This patch adds an experiment group that tokenizes synchronously
on the main thread in response to a recieve data IPC.

If this regresses performance, we can augment the CL slightly
to only synchronously tokenize when the main parser has no
work.

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

to

==========
Add an experiment group via Settings to tokenize synchronously

This patch adds an experiment group that tokenizes synchronously
on the main thread in response to a recieve data IPC.

If this regresses performance, we can augment the CL slightly
to only synchronously tokenize when the main parser has no
work.

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

Committed: https://crrev.com/2fcf4bab78301b1c7b64b1b47547e7e5a70da65f
Cr-Commit-Position: refs/heads/master@{#405035}
==========

[commit-bot: I haz the power, 2016-07-13 05:06:08]

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