LayoutTests/fast/repaint/positioned-input-text-cursor-repaint.html - Issue 709613003: To ensure consistency between layouts, we need to reset any overriden top for centering the textfie…

Unified Diff: LayoutTests/fast/repaint/positioned-input-text-cursor-repaint.html

Issue 709613003: To ensure consistency between layouts, we need to reset any overriden top for centering the textfie… (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: Created 6 years, 1 month ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

« no previous file with comments | « LayoutTests/TestExpectations ('k') | LayoutTests/fast/repaint/positioned-input-text-cursor-repaint-expected.html » ('j') | LayoutTests/fast/repaint/positioned-input-text-cursor-repaint-expected.html » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: LayoutTests/fast/repaint/positioned-input-text-cursor-repaint.html

diff --git a/LayoutTests/fast/repaint/positioned-input-text-cursor-repaint.html b/LayoutTests/fast/repaint/positioned-input-text-cursor-repaint.html

new file mode 100644

index 0000000000000000000000000000000000000000..368ff7cd31b47046dcc20b347c8cd0f62356222b

--- /dev/null

+++ b/LayoutTests/fast/repaint/positioned-input-text-cursor-repaint.html

@@ -0,0 +1,38 @@

+<!DOCTYPE html>

Julien - ping for review 2014/11/11 16:57:58 This test is not in the right directory: fast/repa

pals 2014/11/12 13:29:08 Done.

+<script>

+var i = 0;

+var refreshIntervalId;

+function repaintTest()

+ refreshIntervalId = setInterval(function(){moveInputElement();}, 10);

Julien - ping for review 2014/11/11 16:57:58 setInterval is *almost never* a good solution. If

pals 2014/11/12 13:29:08 The bug does not reproduce in case of fulllayout.

Julien - ping for review 2014/11/19 01:13:27 simplifiedlayout is just a type of layout. You sho

+function moveInputElement()

+ var input = document.getElementById('input');

+ i++;

+ input.style.left = i*10 + 'px';

+ input.style.width = '200px';

+ if (i >= 3) {

+ clearInterval(refreshIntervalId);

+ if (window.testRunner)

+ testRunner.notifyDone();

+ }

+if (window.testRunner)

+ testRunner.waitUntilDone();

+window.onload = repaintTest();

+</script>

+<style>

+input {

+ position: absolute;

+ top: 0;

+ left: 0;

+ height: 50px;

+</style>

Julien - ping for review 2014/11/11 16:57:58 Tests should have a description of what they test

pals 2014/11/12 13:29:08 Done.

+<input id="input" type="text" autofocus>