Inline SVGImage::setContainerSize() and remove extra resize call

Inline SVGImage::setContainerSize() and remove extra resize call

DrawInternal() is responsible for resizing the frame to the new
container size. Setting the frame size to the old container size
temporarily ought to be a redundant operation.

BUG=581357
Committed: https://crrev.com/15f0bca570b644f01b0d75e0967450993183c1d7
Cr-Commit-Position: refs/heads/master@{#373517}

[davve, 2016-02-04 08:48:39]

The CQ bit was checked by davve@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-02-04 08:48:47]

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

[commit-bot: I haz the power, 2016-02-04 10:08:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-04 10:08:13]

Dry run: This issue passed the CQ dry run.

[davve, 2016-02-04 10:29:50]

davve@opera.com changed reviewers:
+ fs@opera.com

[davve, 2016-02-04 10:29:50]

This extra resize call has confused me from time to time. Let's see if we can
remove it.

If it can be removed, it would make it clearer how and when containerSize() is
called. Would in turn make things easier going forward.

[fs, 2016-02-04 11:17:02]

On 2016/02/04 at 10:29:50, davve wrote:
> This extra resize call has confused me from time to time. Let's see if we can
remove it.
> 
> If it can be removed, it would make it clearer how and when containerSize() is
called. Would in turn make things easier going forward.

Makes sense. LGTM

[davve, 2016-02-04 12:44:45]

The CQ bit was checked by davve@opera.com

[commit-bot: I haz the power, 2016-02-04 12:45:15]

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

[commit-bot: I haz the power, 2016-02-04 12:50:18]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-04 12:51:38]

Description was changed from

==========
Inline SVGImage::setContainerSize() and remove extra resize call

DrawInternal() is responsible for resizing the frame to the new
container size. Setting the frame size to the old container size
temporarily ought to be a redundant operation.

BUG=581357
==========

to

==========
Inline SVGImage::setContainerSize() and remove extra resize call

DrawInternal() is responsible for resizing the frame to the new
container size. Setting the frame size to the old container size
temporarily ought to be a redundant operation.

BUG=581357

Committed: https://crrev.com/15f0bca570b644f01b0d75e0967450993183c1d7
Cr-Commit-Position: refs/heads/master@{#373517}
==========

[commit-bot: I haz the power, 2016-02-04 12:51:39]

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