Atomic scaling in ImageResource::imageSize()

Atomic scaling in ImageResource::imageSize()

Since https://codereview.chromium.org/1634133003/ there is only
hasRelativeSize(), no hasRelative{Width,Height}(). The code affected
by this patch was written under the latter assumption. With only
hasRelativeSize() the code can be simplified a bit to either scale or
return early.

Also remove useless ASSERT. |multiplier| is always non-equal to one in
that code branch so the assert doesn't check anything meaningful.

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

[davve, 2016-02-03 16:59:37]

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

[commit-bot: I haz the power, 2016-02-03 17:00:45]

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

[commit-bot: I haz the power, 2016-02-03 18:37:47]

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

[commit-bot: I haz the power, 2016-02-03 18:37:48]

Dry run: This issue passed the CQ dry run.

[davve, 2016-02-04 08:52:41]

Description was changed from

==========
Atomic scaling in ImageResource::imageSize()

Since https://codereview.chromium.org/1634133003/ there is only
hasRelativeSize(), no hasRelative{Width,Height}(). The code affected
by this patch was written under the latter assumption. With only
hasRelativeSize() the code can be simplified a bit to either scale or
return early.

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

to

==========
Atomic scaling in ImageResource::imageSize()

Since https://codereview.chromium.org/1634133003/ there is only
hasRelativeSize(), no hasRelative{Width,Height}(). The code affected
by this patch was written under the latter assumption. With only
hasRelativeSize() the code can be simplified a bit to either scale or
return early.

Also remove useless ASSERT. |multiplier| is always non-equal to one in
that code branch so the assert doesn't check anything meaningful.

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

[davve, 2016-02-04 08:53:34]

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

[davve, 2016-02-04 08:53:35]

Follow-up to https://codereview.chromium.org/1634133003/

[davve, 2016-02-04 08:53:41]

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:54:08]

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

[fs, 2016-02-04 09:11:50]

lgtm

[davve, 2016-02-04 09:15:00]

The CQ bit was unchecked by davve@opera.com

[davve, 2016-02-04 09:15:00]

The CQ bit was checked by davve@opera.com

[commit-bot: I haz the power, 2016-02-04 09:15:53]

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

[commit-bot: I haz the power, 2016-02-04 10:26:48]

Description was changed from

==========
Atomic scaling in ImageResource::imageSize()

Since https://codereview.chromium.org/1634133003/ there is only
hasRelativeSize(), no hasRelative{Width,Height}(). The code affected
by this patch was written under the latter assumption. With only
hasRelativeSize() the code can be simplified a bit to either scale or
return early.

Also remove useless ASSERT. |multiplier| is always non-equal to one in
that code branch so the assert doesn't check anything meaningful.

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

to

==========
Atomic scaling in ImageResource::imageSize()

Since https://codereview.chromium.org/1634133003/ there is only
hasRelativeSize(), no hasRelative{Width,Height}(). The code affected
by this patch was written under the latter assumption. With only
hasRelativeSize() the code can be simplified a bit to either scale or
return early.

Also remove useless ASSERT. |multiplier| is always non-equal to one in
that code branch so the assert doesn't check anything meaningful.

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

[commit-bot: I haz the power, 2016-02-04 10:26:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-04 10:28:05]

Description was changed from

==========
Atomic scaling in ImageResource::imageSize()

Since https://codereview.chromium.org/1634133003/ there is only
hasRelativeSize(), no hasRelative{Width,Height}(). The code affected
by this patch was written under the latter assumption. With only
hasRelativeSize() the code can be simplified a bit to either scale or
return early.

Also remove useless ASSERT. |multiplier| is always non-equal to one in
that code branch so the assert doesn't check anything meaningful.

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

to

==========
Atomic scaling in ImageResource::imageSize()

Since https://codereview.chromium.org/1634133003/ there is only
hasRelativeSize(), no hasRelative{Width,Height}(). The code affected
by this patch was written under the latter assumption. With only
hasRelativeSize() the code can be simplified a bit to either scale or
return early.

Also remove useless ASSERT. |multiplier| is always non-equal to one in
that code branch so the assert doesn't check anything meaningful.

BUG=581357

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

[commit-bot: I haz the power, 2016-02-04 10:28:06]

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