Bug 72240

Summary: Add test expectations for gray-scale-jpeg-with-color-profile.html
Product: WebKit Reporter: noel gordon <noel.gordon>
Component: Tools / TestsAssignee: noel gordon <noel.gordon>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, eric, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 49950    
Bug Blocks: 59670    
Attachments:
Description Flags
webkit-patch rebaseline-expectations
none
It's getting better all the time ... none

Comment 1 noel gordon 2011-11-13 22:50:27 PST
Created attachment 114881 [details]
webkit-patch rebaseline-expectations
Comment 2 WebKit Review Bot 2011-11-13 22:53:44 PST
Attachment 114881 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/fast..." exit_code: 1

Last 3072 characters of output:
arent-src-synchronously.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3851:  Path does not exist. svg/custom/getBBox-path.svg  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3853:  Path does not exist. svg/zoom/page/zoom-svg-as-relative-image.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3854:  Path does not exist. svg/zoom/page/relative-sized-document-scrollbars.svg  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3856:  Path does not exist. storage/indexeddb/key-type-array.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3858:  Path does not exist. storage/indexeddb/factory-deletedatabase-interactions.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3860:  Path does not exist. fast/js/array-functions-non-arrays.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3861:  Path does not exist. fast/js/eval-cross-window.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3862:  Path does not exist. fast/js/regexp-caching.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3863:  Path does not exist. fast/js/toString-overrides.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3864:  Path does not exist. fast/js/toString-recursion.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3865:  Path does not exist. http/tests/security/xss-eval.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3867:  Path does not exist. fast/dom/javascript-url-exception-isolation.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3868:  Path does not exist. fast/borders/inline-mask-overlay-image-outset-vertical-rl.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3870:  Path does not exist. fast/dom/rtl-scroll-to-leftmost-and-resize.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3872:  Path does not exist. http/tests/inspector/resource-tree/appcache-iframe-manifests.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3874:  Path does not exist. accessibility/adjacent-continuations-cause-assertion-failure.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3876:  Path does not exist. inspector/debugger/script-formatter.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3879:  Path does not exist. fast/js/mozilla/strict/15.4.5.1.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3881:  Path does not exist. fast/canvas/canvas-transforms-fillRect-shadow.html  [test/expectations] [2]
LayoutTests/platform/chromium/test_expectations.txt:3883:  Path does not exist. media/track/tracklist-is-reachable.html  [test/expectations] [2]
Total errors found: 2133 in 3 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 noel gordon 2011-11-13 23:06:04 PST
I might buy the Beatles Catalog on iTunes instead :)  I was synced to ToT on snow-leopard at the time, warning at the top was:

(WARNING: Could not determine the port for LayoutTests/platform/chromium/test_expectations.txt. Using 'test' port, but platform-specific expectations will fail the check.

Maybe I need to give a specific platform arg (--platform all) to webkit-patch rebaseline-expectations or some such ...
Comment 4 Adam Barth 2011-11-13 23:11:51 PST
I bet Eric broke this with his recent changes to how we get the port object.
Comment 5 Eric Seidel (no email) 2011-11-13 23:13:56 PST
How do I test this?
Comment 6 noel gordon 2011-11-15 00:15:11 PST
Fixed on http://trac.webkit.org/changeset/100231 so a test drive ...
Comment 7 noel gordon 2011-11-15 00:17:34 PST
Created attachment 115110 [details]
It's getting better all the time ...
Comment 8 Eric Seidel (no email) 2011-11-15 03:13:01 PST
Comment on attachment 115110 [details]
It's getting better all the time ...

OK.
Comment 9 Eric Seidel (no email) 2011-11-15 03:13:18 PST
Why did it need rebaseline?
Comment 10 WebKit Review Bot 2011-11-15 03:31:11 PST
Comment on attachment 115110 [details]
It's getting better all the time ...

Clearing flags on attachment: 115110

Committed r100264: <http://trac.webkit.org/changeset/100264>
Comment 11 WebKit Review Bot 2011-11-15 03:31:18 PST
All reviewed patches have been landed.  Closing bug.
Comment 12 noel gordon 2011-11-15 15:57:42 PST
(In reply to comment #9)
> Why did it need rebaseline?

Had "Rebaseline" on my brain for some reason, and wrongly.

No rebaseline needed, we're adding missing expectations.  Will re-title the bug, and link back to the issue at hand.