SVGFontFaceElement::removedFrom should not use FontSelector when document is not active.

SVGFontFaceElement::removedFrom should not use FontSelector when document is not active.

BUG=348401
TEST=fast/css/use-incorrect-svg-crash.html
TBR=morrita@chromium.org, dglazkov@chromium.org

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

[kouhei (in TOK), 2014-03-03 04:33:20]

nullchecking add lgtm with nits.

[kouhei (in TOK), 2014-03-03 04:33:28]

https://codereview.chromium.org/183743012/diff/1/LayoutTests/fast/css/resourc...
File LayoutTests/fast/css/resources/incorrect-font.svg (right):

https://codereview.chromium.org/183743012/diff/1/LayoutTests/fast/css/resourc...
LayoutTests/fast/css/resources/incorrect-font.svg:1: <?xml version="1.0"
standalone="no"?>
Would you add a comment where it fails to parse?

[kouhei (in TOK), 2014-03-03 04:53:47]

Additional comments lgtm.

[tasak, 2014-03-03 04:56:44]

+morrita

Would you tell me about whether imported documents can be active or not?

[tasak, 2014-03-03 05:05:20]

+haraken
Would you review this CL?
I need to land this patch as soon as possible.

So I will fix the FIXME after landing this patch.

[haraken, 2014-03-03 05:08:01]

Given the urgency, rubberstamp LGTM.

I'd like to have dimitri and morrita-san to take a look.

[tasak, 2014-03-03 05:10:20]

On 2014/03/03 05:08:01, haraken wrote:
> Given the urgency, rubberstamp LGTM.
> 
> I'd like to have dimitri and morrita-san to take a look.

Thank you, haraken.

I added TBR=morrita1, dglazkov to the description.

[tasak, 2014-03-03 05:10:26]

The CQ bit was checked by tasak@google.com

[commit-bot: I haz the power, 2014-03-03 05:10:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tasak@google.com/183743012/20001

[commit-bot: I haz the power, 2014-03-03 07:33:23]

Change committed as 168271