Layout test of PerformanceResourceTiming for multipart

Layout test of PerformanceResourceTiming for multipart

PerformanceResourceTiming fields should be correctly initialised for
multipart images.

They are not, so this CL includes an expectation that
the test will fail.

BUG=631004
Committed: https://crrev.com/d0441c41caadbe2029de72434ae7712dccf03662
Cr-Commit-Position: refs/heads/master@{#408938}

[Adam Rice, 2016-07-29 10:28:37]

Description was changed from

==========
Layout test of PerformanceResourceTiming for multipart

PerformanceResourceTiming fields should be correctly initialised for
multipart images.

BUG=631004
==========

to

==========
Layout test of PerformanceResourceTiming for multipart

PerformanceResourceTiming fields should be correctly initialised for
multipart images.

They are not, so this CL includes an expectation that
the test will fail.

BUG=631004
==========

[Adam Rice, 2016-07-29 10:29:55]

ricea@chromium.org changed reviewers:
+ ksakamoto@chromium.org

[Kunihiko Sakamoto, 2016-08-01 02:15:16]

https://codereview.chromium.org/2176153002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/http/tests/misc/resource-timing-sizes-multipart-expected.txt
(right):

https://codereview.chromium.org/2176153002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/misc/resource-timing-sizes-multipart-expected.txt:2:
FAIL PerformanceResourceTiming multipart image assert_greater_than:
encodedBodySize expected a number greater than 12241 but got 8469
I'm not very comfortable with having a test with failing expectations. Would you
comment out assertions that fail (with a TODO comment) for now, or just hold on
this CL until you have a fix of the bug?

[Adam Rice, 2016-08-01 08:46:51]

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

[commit-bot: I haz the power, 2016-08-01 08:47:10]

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

[Adam Rice, 2016-08-01 08:49:13]

https://codereview.chromium.org/2176153002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/http/tests/misc/resource-timing-sizes-multipart-expected.txt
(right):

https://codereview.chromium.org/2176153002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/misc/resource-timing-sizes-multipart-expected.txt:2:
FAIL PerformanceResourceTiming multipart image assert_greater_than:
encodedBodySize expected a number greater than 12241 but got 8469
On 2016/08/01 02:15:16, Kunihiko Sakamoto wrote:
> I'm not very comfortable with having a test with failing expectations. Would
you
> comment out assertions that fail (with a TODO comment) for now, or just hold
on
> this CL until you have a fix of the bug?

I have added a TODO.

[Kunihiko Sakamoto, 2016-08-01 09:36:26]

lgtm

[commit-bot: I haz the power, 2016-08-01 09:49:39]

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

[commit-bot: I haz the power, 2016-08-01 09:49:40]

Dry run: This issue passed the CQ dry run.

[Adam Rice, 2016-08-01 11:12:05]

The CQ bit was checked by ricea@chromium.org

[commit-bot: I haz the power, 2016-08-01 11:12:18]

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, 2016-08-01 11:15:32]

Description was changed from

==========
Layout test of PerformanceResourceTiming for multipart

PerformanceResourceTiming fields should be correctly initialised for
multipart images.

They are not, so this CL includes an expectation that
the test will fail.

BUG=631004
==========

to

==========
Layout test of PerformanceResourceTiming for multipart

PerformanceResourceTiming fields should be correctly initialised for
multipart images.

They are not, so this CL includes an expectation that
the test will fail.

BUG=631004
==========

[commit-bot: I haz the power, 2016-08-01 11:15:33]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-01 11:17:11]

Description was changed from

==========
Layout test of PerformanceResourceTiming for multipart

PerformanceResourceTiming fields should be correctly initialised for
multipart images.

They are not, so this CL includes an expectation that
the test will fail.

BUG=631004
==========

to

==========
Layout test of PerformanceResourceTiming for multipart

PerformanceResourceTiming fields should be correctly initialised for
multipart images.

They are not, so this CL includes an expectation that
the test will fail.

BUG=631004

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

[commit-bot: I haz the power, 2016-08-01 11:17:12]

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