Add a test of getBoundingClientRect() when white-space:pre for complex scripts

Add a test of getBoundingClientRect() when white-space:pre for complex scripts

At one point the caret was placed at an incorrect position for complex
scripts. This was caused by advances and/or bounds calculation was
incorrect when white-space:pre is applied to a text run of complex
scripts.

It was fixed in later patches. This CL adds a test for the failure to
prevent it happening again.

BUG=513994
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=200878

[kojii, 2015-08-19 10:24:11]

kojii@chromium.org changed reviewers:
+ drott@chromium.org, eae@chromium.org

[kojii, 2015-08-19 10:24:11]

PTAL.

[eae, 2015-08-19 15:38:10]

LGTM

https://codereview.chromium.org/1303673005/diff/20001/LayoutTests/fast/text/i...
File LayoutTests/fast/text/international/bounds-white-space-pre.html (right):

https://codereview.chromium.org/1303673005/diff/20001/LayoutTests/fast/text/i...
LayoutTests/fast/text/international/bounds-white-space-pre.html:18: var
expectedNode = nodes[0].firstChild;
Nit,

var expectedNode = container.firstElementChild.firstChild;
var targetNode = container.lastElementChild.firstChild;;

is easier and negates the need for the query selector call.

[kojii, 2015-08-20 05:00:30]

https://codereview.chromium.org/1303673005/diff/20001/LayoutTests/fast/text/i...
File LayoutTests/fast/text/international/bounds-white-space-pre.html (right):

https://codereview.chromium.org/1303673005/diff/20001/LayoutTests/fast/text/i...
LayoutTests/fast/text/international/bounds-white-space-pre.html:18: var
expectedNode = nodes[0].firstChild;
On 2015/08/19 15:38:10, eae wrote:
> Nit,
> 
> var expectedNode = container.firstElementChild.firstChild;
> var targetNode = container.lastElementChild.firstChild;;
> 
> is easier and negates the need for the query selector call.

Done.

[kojii, 2015-08-20 05:00:36]

The CQ bit was checked by kojii@chromium.org

[kojii, 2015-08-20 05:00:36]

The patchset sent to the CQ was uploaded after l-g-t-m from eae@chromium.org
Link to the patchset: https://codereview.chromium.org/1303673005/#ps40001
(title: "eae review")

[commit-bot: I haz the power, 2015-08-20 05:00:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1303673005/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1303673005/40001

[commit-bot: I haz the power, 2015-08-20 05:40:05]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=200878