Make <iframe> containing SVG behave as every other <iframe>

Make <iframe> containing SVG behave as every other <iframe>

When an SVG is contained inside an <iframe>, its dimensions should not
be part of size calculations of the <iframe>. Neither should the SVG's
own width and height be ignored when sizing the <svg> inside the
<iframe>

The specification waters are murky here but this aligns Blink with
Firefox and IE.

BUG=308992, 364051
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=172467

[davve, 2014-04-23 15:18:12]

Something you think we should do, Philip?

Aligning with the other feels right here, but perhaps there is specification
work to do before we can land it. WDYT?

[pdr., 2014-04-23 16:54:37]

On 2014/04/23 15:18:12, David Vest wrote:
> Something you think we should do, Philip?
> 
> Aligning with the other feels right here, but perhaps there is specification
> work to do before we can land it. WDYT?

I do think we should do this, and I like the way you implemented it. LGTM

Because we're aligning with the other major browsers, I think we should go ahead
with this patch. In parallel, could you try to get the spec clarified around
this?

[davve, 2014-04-24 07:12:57]

On 2014/04/23 16:54:37, pdr wrote:
> On 2014/04/23 15:18:12, David Vest wrote:
> > Something you think we should do, Philip?
> > 
> > Aligning with the other feels right here, but perhaps there is specification
> > work to do before we can land it. WDYT?
> 
> I do think we should do this, and I like the way you implemented it. LGTM
> 
> Because we're aligning with the other major browsers, I think we should go
ahead
> with this patch. In parallel, could you try to get the spec clarified around
> this?

Thanks for the review! I'll do that.

[davve, 2014-04-24 07:22:54]

The CQ bit was checked by davve@opera.com

[commit-bot: I haz the power, 2014-04-24 07:23:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/davve@opera.com/246933007/20001

[commit-bot: I haz the power, 2014-04-24 08:28:23]

Change committed as 172467