Yield HTMLDocumentParser for high priority tasks in blink scheduler.

Yield HTMLDocumentParser for high priority tasks in blink scheduler.

Before this CL, main thread continued to process chunks from
background parser when there were pending high priority tasks.

This CL changes HTMLDocumentParser to query blink scheduler so we can
yield when more high priority tasks are queued.

BUG=421300
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183407

[kouhei (in TOK), 2014-10-08 03:40:42]

kouhei@chromium.org changed reviewers:
+ eseidel@chromium.org, haraken@chromium.org, skyostil@google.com

[eseidel, 2014-10-08 03:52:22]

This seems like a good idea, but ideally this is the HTMLParserScheduler's job
to decide when the parser should run.  Maybe this if block should just get
turned into a small HTMLParserScheduler function.

Regardless, lgtm.

[kouhei (in TOK), 2014-10-08 04:22:35]

On 2014/10/08 03:52:22, eseidel wrote:
> This seems like a good idea, but ideally this is the HTMLParserScheduler's job
> to decide when the parser should run.  Maybe this if block should just get
> turned into a small HTMLParserScheduler function.
> 
> Regardless, lgtm.

Thanks!

Yes. I agree we should refactor the code, but as I wrote on the bug, I'm not
sure where is the right place for the scheduling code.
I'd like to land this patch as is and iterate on refactoring.

[Sami, 2014-10-08 10:21:11]

Great, lgtm.

Another thing to consider is hooking this up to use the idle task API which
rmcilroy@ recently added to the scheduler:
https://codereview.chromium.org/595023002/

[kouhei (in TOK), 2014-10-08 11:35:06]

Thanks for review!

[kouhei (in TOK), 2014-10-08 11:35:09]

The CQ bit was checked by kouhei@chromium.org

[commit-bot: I haz the power, 2014-10-08 11:35:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/634983003/1

[commit-bot: I haz the power, 2014-10-08 11:39:19]

Committed patchset #1 (id:1) as 183407