Adjust test expectations for infinite-size images.

Adjust test expectations for infinite-size images.

The test for infinite-size images is expecting the width to be equal to
the maximum possible value of a blink layout value. However, with
use-zoom-for-dsf on and device scale factor !=1 all image widths are
adjusted by the scale factor. So the test's expectation needs to be
changed because the effective maximum value is now different.

BUG=622000
Committed: https://crrev.com/e691b41f4226ecb00323a46fedb97d0ae2b67047
Cr-Commit-Position: refs/heads/master@{#401397}

[Bret, 2016-06-22 00:42:19]

Description was changed from

==========
Adjust test expectations for infinite-size images.

The test for infinite-size images is expecting the width to be equal to
the maximum possible value of a blink layout value. However, with
use-zoom-for-dsf on and device scale factor !=1 all image widths are
adjusted by the scale factor. So the test's expectation needs to be
changed because the effective maximum value is now different.

BUG=622000
==========

to

==========
Adjust test expectations for infinite-size images.

The test for infinite-size images is expecting the width to be equal to
the maximum possible value of a blink layout value. However, with
use-zoom-for-dsf on and device scale factor !=1 all image widths are
adjusted by the scale factor. So the test's expectation needs to be
changed because the effective maximum value is now different.

BUG=622000
==========

[Bret, 2016-06-22 00:42:19]

bsep@chromium.org changed reviewers:
+ mkwst@chromium.org, oshima@chromium.org

[oshima, 2016-06-22 01:11:51]

non owner lgtm with a nit

https://codereview.chromium.org/2085153002/diff/20001/components/test_runner/...
File components/test_runner/web_test_delegate.h (right):

https://codereview.chromium.org/2085153002/diff/20001/components/test_runner/...
components/test_runner/web_test_delegate.h:159: // Returns whether or not the
use-zoom-for-dsf flag is enabled.
TODO(oshima): Remove return value when all platforms migrate to
use-zoom-for-dsf.

https://codereview.chromium.org/2085153002/diff/20001/components/test_runner/...
components/test_runner/web_test_delegate.h:160: virtual bool
SetDeviceScaleFactor(float factor) = 0;
I have a slight preference to have a separate IsUsZoomForDSFEnabled(), but I'll
defer to mkswt@

[Bret, 2016-06-22 17:42:27]

https://codereview.chromium.org/2085153002/diff/20001/components/test_runner/...
File components/test_runner/web_test_delegate.h (right):

https://codereview.chromium.org/2085153002/diff/20001/components/test_runner/...
components/test_runner/web_test_delegate.h:159: // Returns whether or not the
use-zoom-for-dsf flag is enabled.
On 2016/06/22 01:11:51, oshima wrote:
> TODO(oshima): Remove return value when all platforms migrate to
> use-zoom-for-dsf.

Added in TestRunnerForSpecificView since that's the core thing that needs
deleting later.

https://codereview.chromium.org/2085153002/diff/20001/components/test_runner/...
components/test_runner/web_test_delegate.h:160: virtual bool
SetDeviceScaleFactor(float factor) = 0;
On 2016/06/22 01:11:51, oshima wrote:
> I have a slight preference to have a separate IsUsZoomForDSFEnabled(), but
I'll
> defer to mkswt@

I kinda like that better too, so I changed it.

[Mike West, 2016-06-22 18:57:48]

On 2016/06/22 at 01:11:51, oshima wrote:
> I have a slight preference to have a separate IsUsZoomForDSFEnabled(), but
I'll defer to mkswt@

mkwst@ says LGTM!

[Bret, 2016-06-22 19:27:48]

The CQ bit was checked by bsep@chromium.org

[Bret, 2016-06-22 19:27:48]

The patchset sent to the CQ was uploaded after l-g-t-m from oshima@chromium.org
Link to the patchset: https://codereview.chromium.org/2085153002/#ps40001
(title: "small refactor")

[commit-bot: I haz the power, 2016-06-22 19:28:14]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2085153002/40001

[commit-bot: I haz the power, 2016-06-22 20:56:39]

Description was changed from

==========
Adjust test expectations for infinite-size images.

The test for infinite-size images is expecting the width to be equal to
the maximum possible value of a blink layout value. However, with
use-zoom-for-dsf on and device scale factor !=1 all image widths are
adjusted by the scale factor. So the test's expectation needs to be
changed because the effective maximum value is now different.

BUG=622000
==========

to

==========
Adjust test expectations for infinite-size images.

The test for infinite-size images is expecting the width to be equal to
the maximum possible value of a blink layout value. However, with
use-zoom-for-dsf on and device scale factor !=1 all image widths are
adjusted by the scale factor. So the test's expectation needs to be
changed because the effective maximum value is now different.

BUG=622000
==========

[commit-bot: I haz the power, 2016-06-22 20:56:40]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-22 20:58:41]

Description was changed from

==========
Adjust test expectations for infinite-size images.

The test for infinite-size images is expecting the width to be equal to
the maximum possible value of a blink layout value. However, with
use-zoom-for-dsf on and device scale factor !=1 all image widths are
adjusted by the scale factor. So the test's expectation needs to be
changed because the effective maximum value is now different.

BUG=622000
==========

to

==========
Adjust test expectations for infinite-size images.

The test for infinite-size images is expecting the width to be equal to
the maximum possible value of a blink layout value. However, with
use-zoom-for-dsf on and device scale factor !=1 all image widths are
adjusted by the scale factor. So the test's expectation needs to be
changed because the effective maximum value is now different.

BUG=622000

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

[commit-bot: I haz the power, 2016-06-22 20:58:43]

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