Bug 97087 – [EFL][DRT]DumpRenderTree needs to reset focus state when test starts.

• Component: WebKit EFL
• Status: RESOLVED FIXED

[Michal Pakula vel Rutka]

When running layout tests in editing/undo category in WebKit1 editing/undo/undo-indent and editing/undo/undo-misspellings have flaky results, as editing delegates does not match expected one. However if those test where ran separately, results where as expected. The difference comes from running editing/undo/undo-iframe-location-change before tests mentioned above: this test leaves frame in unfocused state and proper delegates are not triggered.

[Michal Pakula vel Rutka]

Created attachment 164701 [details]
proposed patch

The patch adds main frame focusing similar to WebKit-GTK's DumpRenderTree view focusing in dump() function. Then tests will start with main frame focused regardless of earlier test's frame focus state.

[Grzegorz Czajkowski]

LGTM.

Just wondering, did you try to set the frame focus in DumpRenderTreeChrome::resetDefaultsToConsistentValues()? IMHO it's good place to restore the original settings before running test.

[Michal Pakula vel Rutka]

(In reply to comment #2)
> LGTM.
> 
> Just wondering, did you try to set the frame focus in DumpRenderTreeChrome::resetDefaultsToConsistentValues()? IMHO it's good place to restore the original settings before running test.

I checked and it also fixes the problem. I will check if it does not introduce any regressions, and then post a new patch.

[Michal Pakula vel Rutka]

Created attachment 164746 [details]
setting focus moved into resetDefaultsToConsistentValues

[Grzegorz Czajkowski]

Comment on attachment 164746 [details]
setting focus moved into resetDefaultsToConsistentValues

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

Thanks for your verification of newly proposed approach and changes. Looks better.

> LayoutTests/ChangeLog:8
> +        Remove two flaky tests from TestExpectations.

Please add a note that after setting frame focus the tests are passing now.

> Tools/ChangeLog:8
> +        Add focusing a main frame on each test dump.

on each test dump -> on settings reset?

[Michal Pakula vel Rutka]

Created attachment 164859 [details]
change log fixes

[Gyuyoung Kim]

Comment on attachment 164859 [details]
change log fixes

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

> Tools/DumpRenderTree/efl/DumpRenderTreeChrome.cpp:291
>      ewk_view_text_direction_set(mainView(), EWK_TEXT_DIRECTION_DEFAULT);

It would be good if you add a new line to here.

[Michal Pakula vel Rutka]

Created attachment 164864 [details]
moved focusing function below

[Gyuyoung Kim]

Comment on attachment 164864 [details]
moved focusing function below

Looks fine.

[Grzegorz Czajkowski]

Comment on attachment 164864 [details]
moved focusing function below

LGTM. Thanks.

[WebKit Review Bot]

Comment on attachment 164864 [details]
moved focusing function below

Clearing flags on attachment: 164864

Committed r129110: <http://trac.webkit.org/changeset/129110>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.