Skip to content

GitLab

T
tangled
Commit 1d3a26c5 authored by Yoshifumi Inoue's avatar Yoshifumi Inoue Committed by Commit Bot
Browse files
Options

Get rid of a space before <br> in editing/selection/drag-in-iframe.html 

This patch gets rid of a space before <br> from "drag-in-iframe.html" to pass
this test in both legacy layout and EditingNG for ease of enabling EditingNG.

Space before <br> should be collapsed this test[1], but legacy layout does
so and EditingNG does to follow the spec[1]. This differences isn't part of
these test file to drag in <iframe>.

[1] https://drafts.csswg.org/css-text-3/#white-space-property

Bug: 707656
Change-Id: I42cc0f7b9cdc1c93dc64472669fc46c08b1a7c1f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2248384
Auto-Submit: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: default avatarKoji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#779191}
parent 49809ade
Hide whitespace changes
Inline Side-by-side
Showing with 1 addition and 1 deletion
third_party/blink/web_tests/editing/selection/drag-in-iframe.html
View file @ 1d3a26c5
......@@ -7,7 +7,7 @@
below and verify that a drag caret appears inside the frame.
</p>
<iframe id="subframe" style="border: 1px solid black;"
srcdoc="<body contenteditable><span id='target'>There</span> should be a caret here --> <br>See it?</body>"></iframe>
srcdoc="<body contenteditable><span id='target'>There</span> should be a caret here --><br>See it?</body>"></iframe>
<script>
const t = async_test('Drag-In-Iframge');
window.addEventListener('load', t.step_func_done(() => {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment