[webkit-changes] [WebKit/WebKit] 9cc46b: [GTK] `TestWebViewEditor` `/webkit/WebKitWebView/e...

Vitaly Dyachkov noreply at github.com
Wed Sep 6 07:36:00 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 9cc46b09b21f4b584ef5b4ea9d89519df6968462
      https://github.com/WebKit/WebKit/commit/9cc46b09b21f4b584ef5b4ea9d89519df6968462
  Author: Vitaly Dyachkov <vitaly at igalia.com>
  Date:   2023-09-06 (Wed, 06 Sep 2023)

  Changed paths:
    M Source/WebKit/UIProcess/API/glib/WebKitEditorState.cpp
    M Tools/TestWebKitAPI/Tests/WebKitGtk/TestWebViewEditor.cpp
    M Tools/TestWebKitAPI/glib/TestExpectations.json

  Log Message:
  -----------
  [GTK] `TestWebViewEditor` `/webkit/WebKitWebView/editable/editable` is flaky
https://bugs.webkit.org/show_bug.cgi?id=261028

Reviewed by Carlos Garcia Campos.

It seems like waiting for the `WebKitWebView`'s `draw` signal doesn't
guarantee that `WebKitEditorState` will be updated after setting
`WebKitWebView` as editable. This leads to flakiness.

The better way to make sure `WebKitEditorState` has been updated is to
wait until the cut action becomes available. To achieve this, this patch
introduces a new `WebKitEditorState`'s `changed` signal.

The test uses this new signal to ensure that `WebKitWebView` is indeed
editable before attempting to cut the selection.

* Source/WebKit/UIProcess/API/glib/WebKitEditorState.cpp:
(webkit_editor_state_class_init):
(webkitEditorStateChanged):
* Tools/TestWebKitAPI/Tests/WebKitGtk/TestWebViewEditor.cpp:
(loadContentsAndTryToCutSelection):
* Tools/TestWebKitAPI/glib/TestExpectations.json:

Canonical link: https://commits.webkit.org/267671@main




More information about the webkit-changes mailing list