Ensure RemoteFontFaceSource doesn't try to start a load in a detached document

Ensure RemoteFontFaceSource doesn't try to start a load in a detached document

BUG=603428
Committed: https://crrev.com/fa123915549f2be2b85ed62f2a43ab1b8c5aace1
Cr-Commit-Position: refs/heads/master@{#387718}

[Nate Chapin, 2016-04-15 21:06:35]

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

[Nate Chapin, 2016-04-15 21:06:36]

mkwst: Does this seem reasonable? I'm planning on refactoring to make this
mistake more difficult to make in https://codereview.chromium.org/1889973002/

[Mike West, 2016-04-15 21:21:09]

LGTM to fix the crasher, though I agree that it's a bit awkward. The other patch
you linked looks interesting, but it's way too late for me to look at it in
anything like detail.

This looks fine for the moment. *shrug*

[Nate Chapin, 2016-04-15 21:24:37]

On 2016/04/15 21:21:09, Mike West wrote:
> LGTM to fix the crasher, though I agree that it's a bit awkward. The other
patch
> you linked looks interesting, but it's way too late for me to look at it in
> anything like detail.
> 
> This looks fine for the moment. *shrug*

Yeah, I'm not thrilled with this, but it's better than pushing out that crash :/

[Nate Chapin, 2016-04-15 21:24:43]

The CQ bit was checked by japhet@chromium.org

[commit-bot: I haz the power, 2016-04-15 21:25:11]

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

[commit-bot: I haz the power, 2016-04-15 22:12:59]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-15 22:14:48]

Description was changed from

==========
Ensure RemoteFontFaceSource doesn't try to start a load in a detached document

BUG=603428
==========

to

==========
Ensure RemoteFontFaceSource doesn't try to start a load in a detached document

BUG=603428

Committed: https://crrev.com/fa123915549f2be2b85ed62f2a43ab1b8c5aace1
Cr-Commit-Position: refs/heads/master@{#387718}
==========

[commit-bot: I haz the power, 2016-04-15 22:14:49]

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