Make computeInlineBoxPosition() to handle unicode-bidi:isolate correctly

Make computeInlineBoxPosition() to handle unicode-bidi:isolate correctly

This patch makes |computeInlineBoxPosition()| to handle "unicode-bidi:isolate"
correctly for position at start of text by checking text direction and
bidi embedding level rather than only checking bidi embedding level.

BUG=600334
TEST=run-webkit-unit-tests --gtest_filter=VisibleUnitsTest.computeInlineBoxPositionBidiIsolate

Committed: https://crrev.com/ec6d070e13bb453972f0c385e090df9e2e5f9478
Cr-Commit-Position: refs/heads/master@{#410325}

[yosin_UTC9, 2016-08-08 06:05:12]

Description was changed from

==========
2016-08-08T15:03:39

BUG=
==========

to

==========
Make computeInlineBoxPosition() to handle unicode-bidi:isolate correctly

BUG=600334
TEST=...
==========

[yosin_UTC9, 2016-08-08 06:05:23]

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

[commit-bot: I haz the power, 2016-08-08 06:05:38]

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

[yosin_UTC9, 2016-08-08 06:05:41]

Description was changed from

==========
Make computeInlineBoxPosition() to handle unicode-bidi:isolate correctly

BUG=600334
TEST=...
==========

to

==========
Make computeInlineBoxPosition() to handle unicode-bidi:isolate correctly

BUG=600334
TEST=run-webkit-unit-tests
--gtest_filter=VisibleUnitsTest.computeInlineBoxPositionBidiIsolate
==========

[commit-bot: I haz the power, 2016-08-08 07:41:23]

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

[commit-bot: I haz the power, 2016-08-08 07:41:24]

Dry run: This issue passed the CQ dry run.

[yosin_UTC9, 2016-08-08 09:27:32]

Description was changed from

==========
Make computeInlineBoxPosition() to handle unicode-bidi:isolate correctly

BUG=600334
TEST=run-webkit-unit-tests
--gtest_filter=VisibleUnitsTest.computeInlineBoxPositionBidiIsolate
==========

to

==========
Make computeInlineBoxPosition() to handle unicode-bidi:isolate correctly

This patch makes |computeInlineBoxPosition()| to handle "unicode-bidi:isolate"
correctly for position at start of text by checking text direction and
bidi embedding level rather than only checking bidi embedding level.

BUG=600334
TEST=run-webkit-unit-tests
--gtest_filter=VisibleUnitsTest.computeInlineBoxPositionBidiIsolate
==========

[yosin_UTC9, 2016-08-08 09:27:47]

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

[yosin_UTC9, 2016-08-08 09:27:47]

PTAL

[kojii, 2016-08-08 10:07:01]

lgtm

[yosin_UTC9, 2016-08-08 10:31:35]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-08-08 10:31:50]

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-08 10:34:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-08 10:37:03]

Description was changed from

==========
Make computeInlineBoxPosition() to handle unicode-bidi:isolate correctly

This patch makes |computeInlineBoxPosition()| to handle "unicode-bidi:isolate"
correctly for position at start of text by checking text direction and
bidi embedding level rather than only checking bidi embedding level.

BUG=600334
TEST=run-webkit-unit-tests
--gtest_filter=VisibleUnitsTest.computeInlineBoxPositionBidiIsolate
==========

to

==========
Make computeInlineBoxPosition() to handle unicode-bidi:isolate correctly

This patch makes |computeInlineBoxPosition()| to handle "unicode-bidi:isolate"
correctly for position at start of text by checking text direction and
bidi embedding level rather than only checking bidi embedding level.

BUG=600334
TEST=run-webkit-unit-tests
--gtest_filter=VisibleUnitsTest.computeInlineBoxPositionBidiIsolate

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

[commit-bot: I haz the power, 2016-08-08 10:37:04]

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