Bug 75615 – [Qt][WK2]REGRESSION(r104068): editing/pasteboard/copy-element-with-conflicting-background-color-from-rule.html made the next test fail

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Csaba Osztrogonác]

--- /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/editing/pasteboard/copy-in-password-field-expected.txt 
+++ /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/editing/pasteboard/copy-in-password-field-actual.txt 
@@ -3,9 +3,9 @@
 EDITING DELEGATE: webViewDidChangeSelection:WebViewDidChangeSelectionNotification
 EDITING DELEGATE: webViewDidChangeSelection:WebViewDidChangeSelectionNotification
 EDITING DELEGATE: webViewDidChangeSelection:WebViewDidChangeSelectionNotification
-EDITING DELEGATE: shouldInsertText:[not modified] replacingDOMRange:range from 0 of #text > DIV > #shadow-root to 14 of #text > DIV > #shadow-root givenAction:WebViewInsertActionPasted
+EDITING DELEGATE: shouldInsertText: replacingDOMRange:range from 0 of #text > DIV > #shadow-root to 14 of #text > DIV > #shadow-root givenAction:WebViewInsertActionPasted
 EDITING DELEGATE: webViewDidChangeSelection:WebViewDidChangeSelectionNotification
-EDITING DELEGATE: shouldChangeSelectedDOMRange:range from 0 of DIV > #shadow-root to 0 of DIV > #shadow-root toDOMRange:range from 14 of #text > DIV > #shadow-root to 14 of #text > DIV > #shadow-root affinity:NSSelectionAffinityDownstream stillSelecting:FALSE
+EDITING DELEGATE: shouldChangeSelectedDOMRange:range from 0 of DIV > #shadow-root to 0 of DIV > #shadow-root toDOMRange:range from 0 of DIV > #shadow-root to 0 of DIV > #shadow-root affinity:NSSelectionAffinityDownstream stillSelecting:FALSE
 EDITING DELEGATE: webViewDidChangeSelection:WebViewDidChangeSelectionNotification
 EDITING DELEGATE: webViewDidChange:WebViewDidChangeNotification
 This test verifies that copying is disabled for password fields by attempting to copy from a password field and paste into a textfield. If the test passes, you'll see a of 'PASS' message below, and the textfield will remain unmodified.
@@ -13,5 +13,5 @@
 (The test is only meaningful in debug builds, since paste is disabled in release builds.)
 
   
-PASS document.getElementById("text").value is "[not modified]"
+FAIL document.getElementById("text").value should be [not modified]. Was .

[Csaba Osztrogonác]

I skipped it to paint the bot green: http://trac.webkit.org/changeset/104141

[Csaba Osztrogonác]

(In reply to comment #1)
> I skipped it to paint the bot green: http://trac.webkit.org/changeset/104141

Unfortunately it didn't help, now the next test fails. :-/ It seems the editing/pasteboard/copy-element-with-conflicting-background-color-from-rule.html is the culprit now. It fails and makes the next test fail too. But the failing of editing/pasteboard/copy-element-with-conflicting-background-color-from-rule.html is only a flakey fail, because it passes in the retry phase ... I can't reproduce this flakiness in smaller example, only the whole testcase ... Pfff ... how I hate this kind of sideeffect bugs ...

[Csaba Osztrogonác]

I skipped the culprit: http://trac.webkit.org/changeset/104162

[Csaba Osztrogonác]

flakey fail: the green "Copy this line" line is missing from actual png

next test fail after it:
--- /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/editing/pasteboard/copy-paste-bidi-expected.txt 
+++ /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/editing/pasteboard/copy-paste-bidi-actual.txt 
@@ -4,7 +4,7 @@
 | "1234׊×××:"
 
 first test - after:
-| "1234׊×××:<#selection-caret>"
+| <br>
 
 second test - before:
 | "

[Ryosuke Niwa]

Maybe Qt DRT has some bug in clipboard code?

[Csaba Osztrogonác]

These tests aren't skipped now.