Summary: | rewrite fast/dom/Window/window-screen-properties to be slightly more portable | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dirk Pranke <dpranke> | ||||||||
Component: | Tools / Tests | Assignee: | Dirk Pranke <dpranke> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | commit-queue, dimich | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | PC | ||||||||||
OS: | OS X 10.5 | ||||||||||
Attachments: |
|
Description
Dirk Pranke
2009-11-11 14:07:07 PST
Created attachment 43000 [details]
proposed rewrite of patch
Comment on attachment 43000 [details]
proposed rewrite of patch
The patch seems to contain unrelated png files.
Also, while you are at it, does it make sense to verify that pixelDepth==colorDepth or is it not actually always the case?
r- because of extra png files
grr. Not only were there pngs that snuck in (I'm not sure how), there was a typo in the test file. per http://www.quirksmode.org/dom/w3c_cssom.html , there may be differences between pixeldepth and colordepth on older unix machines (where bpp <= 8). I don't know if we even support bpp <= 8, so I could certainly add that check if we wanted. Created attachment 43013 [details]
revised patch - corrects typo in test case, drops mistakenly included pngs
Comment on attachment 43013 [details]
revised patch - corrects typo in test case, drops mistakenly included pngs
Almost there. I think you also need to update platform/win/fast/dom/Window/window-screen-properties-expected.txt
Err, not to update but rather delete since your updated test removes a need for a platform-specific snapshot. Created attachment 43021 [details]
drop unnecessary win expectation
Good catch, actually we should be able to delete it.
Comment on attachment 43021 [details] drop unnecessary win expectation Clearing flags on attachment: 43021 Committed r50857: <http://trac.webkit.org/changeset/50857> All reviewed patches have been landed. Closing bug. |