Bug 75615

Summary: [Qt][WK2]REGRESSION(r104068): editing/pasteboard/copy-element-with-conflicting-background-color-from-rule.html made the next test fail
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ossy, rniwa, webkit-sed
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 75330    
Bug Blocks: 79666    

Description Csaba Osztrogonác 2012-01-05 04:33:48 PST
--- /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 .
Comment 1 Csaba Osztrogonác 2012-01-05 04:50:47 PST
I skipped it to paint the bot green: http://trac.webkit.org/changeset/104141
Comment 2 Csaba Osztrogonác 2012-01-05 07:33:26 PST
(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 ...
Comment 3 Csaba Osztrogonác 2012-01-05 07:37:39 PST
I skipped the culprit: http://trac.webkit.org/changeset/104162
Comment 4 Csaba Osztrogonác 2012-01-05 07:40:08 PST
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:
 | "
Comment 5 Ryosuke Niwa 2012-02-27 08:00:12 PST
Maybe Qt DRT has some bug in clipboard code?
Comment 6 Csaba Osztrogonác 2012-10-09 01:34:16 PDT
These tests aren't skipped now.