Bug 117574 – [EFL][WK1] Some tests of editing/selection fail to render properly

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit EFL

[Gyuyoung Kim]

Draw the wrong patterns in comparison to the EFL-WK2.

editing/selection/3690703-2.html
editing/selection/3690703.html
editing/selection/3690719.html
editing/selection/4397952.html
editing/selection/4895428-3.html
editing/selection/4975120.html
editing/selection/5007143-2.html
editing/selection/5007143.html
editing/selection/5057506.html
editing/selection/5195166-1.html
editing/selection/7152-2.html
editing/selection/caret-and-focus-ring.html
editing/selection/caret-before-select.html
editing/selection/caret-ltr-right.html
editing/selection/caret-rtl-2-left.html
editing/selection/caret-rtl-right.html

[Sun-woo Nam]

Created attachment 215727 [details]
Patch

[Ryuan Choi]

Comment on attachment 215727 [details]
Patch

Thank you.

For your information,
"Unreviewed" means that you don't need to get r+.

Instead, you should ask cq? to get cq+.

If you are using webkit-patch,
you can use --no-review --request-commit option

[Gyuyoung Kim]

Comment on attachment 215727 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=215727&action=review

> LayoutTests/ChangeLog:3
> +        [EFL] Need to update the EFL test expectations.

Different bug title with existing one.

> LayoutTests/platform/efl-wk1/TestExpectations:117
>  webkit.org/b/117574 editing/selection/caret-and-focus-ring.html [ ImageOnlyFailure ]

Please change this bug url using a bug you filed. This bug will be "Resolved Fixed."

[Sun-woo Nam]

Created attachment 215847 [details]
Patch

[Gyuyoung Kim]

Comment on attachment 215847 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=215847&action=review

> LayoutTests/platform/efl-wk1/TestExpectations:485
> +

Do not add unnecessary line.

[Sun-woo Nam]

Created attachment 215851 [details]
Patch

[Sun-woo Nam]

Created attachment 215852 [details]
Patch

[Gyuyoung Kim]

Comment on attachment 215852 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=215852&action=review

> LayoutTests/ChangeLog:11
> +        Unreviewed, EFL gardening.

Generally, patch description is placed below this line.

[Sun-woo Nam]

Created attachment 215854 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 215854 [details]
Patch

Clearing flags on attachment: 215854

Committed r158532: <http://trac.webkit.org/changeset/158532>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.