Set width and height attributes for <img> if image is loaded from srcset

Set width and height attributes for <img> if image is loaded from srcset

For device with high DPR, the image could be loaded from srcset. When
the img element is serialized, srcset attribute is skipped. We need to
set width and height attributes to retain the original size of the
image.

BUG=674999
TEST=new tests added

Review-Url: https://codereview.chromium.org/2624953002
Cr-Commit-Position: refs/heads/master@{#443058}
Committed: https://chromium.googlesource.com/chromium/src/+/a908d62ac9640cc8e9e54521c8ab8133919ef24a

[jianli, 2017-01-11 02:08:45]

The CQ bit was checked by jianli@chromium.org to run a CQ dry run

[jianli, 2017-01-11 02:09:02]

jianli@chromium.org changed reviewers:
+ tkent@chromium.org

[commit-bot: I haz the power, 2017-01-11 02:09:32]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tkent, 2017-01-11 02:15:32]

lgtm

https://codereview.chromium.org/2624953002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/web/WebFrameSerializer.cpp (right):

https://codereview.chromium.org/2624953002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/web/WebFrameSerializer.cpp:217: // has a higher DPR, a
different image from srcsrt could be loaded instead.
srcsrt -> srcset

[jianli, 2017-01-11 02:18:50]

The CQ bit was checked by jianli@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-11 02:19:35]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-11 04:07:23]

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

[commit-bot: I haz the power, 2017-01-11 04:07:23]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[jianli, 2017-01-11 23:12:30]

https://codereview.chromium.org/2624953002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/web/WebFrameSerializer.cpp (right):

https://codereview.chromium.org/2624953002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/web/WebFrameSerializer.cpp:217: // has a higher DPR, a
different image from srcsrt could be loaded instead.
On 2017/01/11 02:15:32, tkent wrote:
> srcsrt -> srcset

Done.

[jianli, 2017-01-11 23:12:42]

The CQ bit was checked by jianli@chromium.org

[jianli, 2017-01-11 23:12:43]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2624953002/#ps40001
(title: "Fix another typo")

[commit-bot: I haz the power, 2017-01-11 23:13:33]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-11 23:48:22]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1484176362716230,
"parent_rev": "68e998adb1ea9fd998afd4dafd372d3d4729a54b", "commit_rev":
"a908d62ac9640cc8e9e54521c8ab8133919ef24a"}

[commit-bot: I haz the power, 2017-01-11 23:49:43]

Description was changed from

==========
Set width and height attributes for <img> if image is loaded from srcset

For device with high DPR, the image could be loaded from srcset. When
the img element is serialized, srcset attribute is skipped. We need to
set width and height attributes to retain the original size of the
image.

BUG=674999
TEST=new tests added
==========

to

==========
Set width and height attributes for <img> if image is loaded from srcset

For device with high DPR, the image could be loaded from srcset. When
the img element is serialized, srcset attribute is skipped. We need to
set width and height attributes to retain the original size of the
image.

BUG=674999
TEST=new tests added

Review-Url: https://codereview.chromium.org/2624953002
Cr-Commit-Position: refs/heads/master@{#443058}
Committed:
https://chromium.googlesource.com/chromium/src/+/a908d62ac9640cc8e9e54521c8ab...
==========

[commit-bot: I haz the power, 2017-01-11 23:49:47]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/a908d62ac9640cc8e9e54521c8ab...