Remove width calculation in `Width` hint

Remove width calculation in `Width` hint

Following the discussion on the CH Intent to ship, I'm removing the width attribute
part of the `Width` calculation, to match it to the inputs
we expose to srcset's selection algorithm.

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

[Yoav Weiss, 2015-07-03 21:21:14]

yoav@yoav.ws changed reviewers:
+ mkwst@chromium.org, philipj@opera.com

[Yoav Weiss, 2015-07-03 21:30:17]

Is there a way to add tests the mess around with the runtime flags?
I'd love to test what happens when the ClientHints flag is enabled and the
WidthInSourceSize flag is disabled.

[philipj_slow, 2015-07-03 22:00:52]

On 2015/07/03 21:30:17, Yoav Weiss wrote:
> Is there a way to add tests the mess around with the runtime flags?
> I'd love to test what happens when the ClientHints flag is enabled and the
> WidthInSourceSize flag is disabled.

You'd have to create a virtual test suite, see LayoutTests/virtual/

[philipj_slow, 2015-07-03 22:02:05]

This is so little code that perhaps just removing it and then adding it back
when there's agreement will result in less overall churn. This doesn't seem like
something that would be worth its own Intent to Ship or anything.

[Yoav Weiss, 2015-07-06 05:58:29]

On 2015/07/03 22:02:05, philipj wrote:
> This is so little code that perhaps just removing it and then adding it back
> when there's agreement will result in less overall churn. This doesn't seem
like
> something that would be worth its own Intent to Ship or anything.

removed

[philipj_slow, 2015-07-06 07:08:49]

lgtm

[Yoav Weiss, 2015-07-06 07:10:17]

The CQ bit was checked by yoav@yoav.ws

[commit-bot: I haz the power, 2015-07-06 07:10:40]

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

[commit-bot: I haz the power, 2015-07-06 07:14:21]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=198304