Use seperate expected files for fast/frames/set-parent-src-synchronously.{html,xhtml}

Use seperate expected files for fast/frames/set-parent-src-synchronously.{html,xhtml}

The two layout tests fast/frames/set-parent-src-synchronously.html and
fast/frames/set-parent-src-synchronously.xhtml have (slightly) different
behaviors but share the same expected file.

This patch renames the two tests so that they use separate expected files,
so that potential test flakiness is reduced.

BUG=n/a

Review-Url: https://codereview.chromium.org/2889213004
Cr-Commit-Position: refs/heads/master@{#473134}
Committed: https://chromium.googlesource.com/chromium/src/+/19a83f8917e6a6df07d91cdaf3db4dc256f82390

[Xiaocheng, 2017-05-19 07:06:31]

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

[commit-bot: I haz the power, 2017-05-19 07:07:04]

Dry run: CQ is trying da patch.

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

[Xiaocheng, 2017-05-19 07:13:12]

Description was changed from

==========
Use seperate expected files for
fast/frames/set-parent-src-synchronously.{html,xhtml}

BUG=n/a
==========

to

==========
Use seperate expected files for
fast/frames/set-parent-src-synchronously.{html,xhtml}

The two layout tests fast/frames/set-parent-src-synchronously.html and
fast/frames/set-parent-src-synchronously.xhtml have (slightly) different
behaviors but share the same expected file.

This patch renames the two tests so that they use separate expected files,
so that potential test flakiness is reduced.

BUG=n/a
==========

[Xiaocheng, 2017-05-19 07:21:24]

xiaochengh@chromium.org changed reviewers:
+ kojii@chromium.org, yosin@chromium.org

[Xiaocheng, 2017-05-19 07:21:24]

PTAL.

It seems to be a layout bug (or feature?) that whitespace collapsing is done
differently for html and xhtml. The contents of the two layout tests are the
same:

some text bla bla bla.
<iframe></iframe>

When the page width is set appropriately so that line-breaking is done between
"bla." and <iframe>, the "\n" between them is handled differently:
- html: collapsed away
- xml: not collapsed away. It's actually laid out as a trailing space after ".",
and can be selected by mouse.

This issue is blocking my patch [1] that stops TextIterator from emitting extra
spaces before replaced elements (e.g., <iframe>). However, due to the different
behaviors, TextIterator will emit different text for html and xhtml. So they
have to use different expected files.

[1] crrev.com/2887273003

[yosin_UTC9, 2017-05-19 07:41:38]

lgtm

[Xiaocheng, 2017-05-19 07:56:28]

The CQ bit was unchecked by xiaochengh@chromium.org

[Xiaocheng, 2017-05-19 07:56:29]

The CQ bit was checked by xiaochengh@chromium.org

[commit-bot: I haz the power, 2017-05-19 07:56:40]

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-05-19 09:06:38]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495180589573640, "parent_rev":
"7446a544808a998f3698178f1cf309f627592c70", "commit_rev":
"19a83f8917e6a6df07d91cdaf3db4dc256f82390"}

[commit-bot: I haz the power, 2017-05-19 09:06:57]

Description was changed from

==========
Use seperate expected files for
fast/frames/set-parent-src-synchronously.{html,xhtml}

The two layout tests fast/frames/set-parent-src-synchronously.html and
fast/frames/set-parent-src-synchronously.xhtml have (slightly) different
behaviors but share the same expected file.

This patch renames the two tests so that they use separate expected files,
so that potential test flakiness is reduced.

BUG=n/a
==========

to

==========
Use seperate expected files for
fast/frames/set-parent-src-synchronously.{html,xhtml}

The two layout tests fast/frames/set-parent-src-synchronously.html and
fast/frames/set-parent-src-synchronously.xhtml have (slightly) different
behaviors but share the same expected file.

This patch renames the two tests so that they use separate expected files,
so that potential test flakiness is reduced.

BUG=n/a

Review-Url: https://codereview.chromium.org/2889213004
Cr-Commit-Position: refs/heads/master@{#473134}
Committed:
https://chromium.googlesource.com/chromium/src/+/19a83f8917e6a6df07d91cdaf3db...
==========

[commit-bot: I haz the power, 2017-05-19 09:07:01]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/19a83f8917e6a6df07d91cdaf3db...