Summary: | REGRESSION(r57292): Safari/Win and Chromium/Win no longer pass the acid3 test. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dimitri Glazkov (Google) <dglazkov> | ||||||
Component: | CSS | Assignee: | Dimitri Glazkov (Google) <dglazkov> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | abarth, aroben, eric, hyatt, jamesr, mjs, webkit.review.bot | ||||||
Priority: | P1 | Keywords: | InRadar | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | PC | ||||||||
OS: | All | ||||||||
Bug Depends on: | 24300 | ||||||||
Bug Blocks: | |||||||||
Attachments: |
|
Description
Dimitri Glazkov (Google)
2010-04-20 16:39:21 PDT
will fix tomorrow, btw. MSVC likes to make enum bitfields signed. _insideLink probably just needs to be bumped up to three bits. That or the type changed from EInsideLink to unsigned. Created attachment 53962 [details]
Patch
Comment on attachment 53962 [details]
Patch
The correct fix is not to make the bitfield larger. Instead we type the bitfield "unsigned" and add a comment with the type:
unsigned _insideLink : 2; // EInsideLink
See the line above.
We might be able to change check-webkit-style to catch this.
Created attachment 53966 [details]
Patch
Thanks Darin! I should've thought of this. dhyatt did all the casting for me already. Attachment 53966 [details] did not pass style-queue:
Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebCore/rendering/style/RenderStyle.h:206: _insideLink is incorrectly named. Don't use underscores in your identifier names. [readability/naming] [4]
Total errors found: 1 in 2 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Landed as http://trac.webkit.org/changeset/57994. http://trac.webkit.org/changeset/57994 might have broken Qt Linux Release |