lowlevel/img.sky flaky crashes

lowlevel/img.sky flaky crashes

We don't need to pump pending speculations if we don't have any.

R=esprehn@chromium.org, eseidel@chromium.org
BUG=447763
Committed: https://chromium.googlesource.com/external/mojo/+/1ca63718c089f6221f196eeb6d93e921987d45f6

[esprehn, 2015-01-14 19:00:46]

esprehn@chromium.org changed reviewers:
+ esprehn@chromium.org

[esprehn, 2015-01-14 19:00:47]

lgtm

https://codereview.chromium.org/852043003/diff/1/sky/engine/core/html/parser/...
File sky/engine/core/html/parser/HTMLDocumentParser.cpp (right):

https://codereview.chromium.org/852043003/diff/1/sky/engine/core/html/parser/...
sky/engine/core/html/parser/HTMLDocumentParser.cpp:387: return;
Why don't we always detach the parser when stopping it? The comment here:

https://github.com/domokit/mojo/blob/f990d6d37b72dfb491d7b9f266f258ec31b6027f...

seems like we should. Strangely it seems like we stop the parser before firing
unload events, and detach it later.

If we unified detach and stop we could make this ASSERT(isStopped());

[abarth-chromium, 2015-01-14 19:04:11]

https://codereview.chromium.org/852043003/diff/1/sky/engine/core/html/parser/...
File sky/engine/core/html/parser/HTMLDocumentParser.cpp (right):

https://codereview.chromium.org/852043003/diff/1/sky/engine/core/html/parser/...
sky/engine/core/html/parser/HTMLDocumentParser.cpp:387: return;
Oops.  This branch shouldn't be here.  That was just something I did for
debugging.  I'll remove it.

[abarth-chromium, 2015-01-14 21:32:25]

Committed patchset #1 (id:1) manually as
1ca63718c089f6221f196eeb6d93e921987d45f6 (presubmit successful).