I updated my webkit tree to rev 118364 and ran the following commands: $ svn diff LayoutTests/platform/chromium/test_expectations.txt [no diffs are reported] $ Tools/Scripts/webkit-patch rebaseline-expectations Retrieving results for chromium-linux-x86 from Webkit Linux 32. Retrieving results for chromium-linux-x86_64 from Webkit Linux. Retrieving results for chromium-mac-leopard from Webkit Mac10.5. Retrieving results for chromium-mac-lion from Webkit Mac10.7. Retrieving results for chromium-mac-snowleopard from Webkit Mac10.6. Retrieving results for chromium-win-vista from Webkit Vista. Retrieving results for chromium-win-win7 from Webkit Win7. Retrieving results for chromium-win-xp from Webkit Win. [no tests had to be rebaselined] $ svn diff LayoutTests/platform/chromium/test_expectations.txt Index: LayoutTests/platform/chromium/test_expectations.txt =================================================================== --- LayoutTests/platform/chromium/test_expectations.txt (revision 118364) +++ LayoutTests/platform/chromium/test_expectations.txt (working copy) @@ -3804,3 +3804,707 @@ BUGWK87364 DEBUG : fast/dom/shadow/drop-event-for-input-in-shadow.html = TEXT PASS BUGWK87364 DEBUG : fast/dom/shadow/drop-event-in-shadow.html = TEXT PASS + +// TEMPORARY overrides of +// src/third_party/WebKit/LayoutTests/platform/chromium/test_expectations.txt +// that are associated with changes to the Skia code. +// [and so on... 8 copies of http://src.chromium.org/svn/trunk/src/skia/skia_test_expectations.txt and http://src.chromium.org/svn/trunk/src/webkit/tools/layout_tests/test_expectations.txt have been appended]
Just to be clear... I am very happy that the overrides expectations files are being observed by the tool. :-) I just don't want them to be permanently added to the base expectations file. I will try to fix this myself, but if anyone else wants to jump on it, go right ahead...
given that I'm revamping how we handle multiple expectations files at the moment (in order to support cascading files), it's probably better for me to own this.
This bug is affecting my workflow to the extent it's no longer acceptable. We need to either roll out r117789 or fix this bug today.
*** Bug 88166 has been marked as a duplicate of this bug. ***
I will be working on this this afternoon. Sorry for the delay; the transit of venus sidetracked me more than I expected ...
Created attachment 146093 [details] Ignore overrides in rebaseline
This this patch doesn't work then I'm going to roll out r117789. It's not acceptable to break one of the most mission critical tool in webkit like this for days.
Comment on attachment 146093 [details] Ignore overrides in rebaseline looks fine for now while I can work out a better long-term fix
Committed r119605: <http://trac.webkit.org/changeset/119605>