RESOLVED FIXED 70240
Move CSSRule specific virtuals from StyleBase down into CSSRule. 
https://bugs.webkit.org/show_bug.cgi?id=70240
Summary Move CSSRule specific virtuals from StyleBase down into CSSRule.
Andreas Kling
Reported 2011-10-17 08:07:23 PDT
The isFooRule() virtuals and insertedIntoParent() can be moved down to CSSRule now.
Attachments
Proposed patch (4.17 KB, patch)
2011-10-17 08:11 PDT, Andreas Kling 		darin: review+
gyuyoung.kim: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andreas Kling
Comment 1 2011-10-17 08:11:34 PDT
Created attachment 111266 [details] Proposed patch
WebKit Review Bot
Comment 2 2011-10-17 08:13:28 PDT
Attachment 111266 [details] did not pass style-queue: Failed to run "['Tools/Scripts/update-webkit', '--chromium']" exit_code: 2 Updating OpenSource Current branch master is up to date. Updating chromium port dependencies using gclient... Error: Can't switch the checkout to http://v8.googlecode.com/svn/branches/3.6@9637; UUID don't match and there is local changes in /mnt/git/webkit-style-queue/Source/WebKit/chromium/v8. Delete the directory and try again. Re-trying 'depot_tools/gclient sync' Error: Can't switch the checkout to http://v8.googlecode.com/svn/branches/3.6@9637; UUID don't match and there is local changes in /mnt/git/webkit-style-queue/Source/WebKit/chromium/v8. Delete the directory and try again. Re-trying 'depot_tools/gclient sync' Error: Can't switch the checkout to http://v8.googlecode.com/svn/branches/3.6@9637; UUID don't match and there is local changes in /mnt/git/webkit-style-queue/Source/WebKit/chromium/v8. Delete the directory and try again. Error: 'depot_tools/gclient sync' failed 3 tries and returned 256 at Tools/Scripts/update-webkit-chromium line 107. Re-trying 'depot_tools/gclient sync' No such file or directory at Tools/Scripts/update-webkit line 104. If any of these errors are false positives, please file a bug against check-webkit-style.
Gyuyoung Kim
Comment 3 2011-10-17 08:31:01 PDT
Comment on attachment 111266 [details] Proposed patch Attachment 111266 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/10086090
Andreas Kling
Comment 4 2011-10-17 08:40:59 PDT
(In reply to comment #3) > (From update of attachment 111266 [details]) > Attachment 111266 [details] did not pass efl-ews (efl): > Output: http://queues.webkit.org/results/10086090 D'oh. XSLImportRule still uses isImportRule().
Collabora GTK+ EWS bot
Comment 5 2011-10-17 12:51:58 PDT
Comment on attachment 111266 [details] Proposed patch Attachment 111266 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/10086597
WebKit Review Bot
Comment 6 2011-10-17 16:21:13 PDT
Comment on attachment 111266 [details] Proposed patch Attachment 111266 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/10120018
Andreas Kling
Comment 7 2011-10-18 05:45:10 PDT
Committed r97742: <http://trac.webkit.org/changeset/97742>
Note You need to log in before you can comment on or make changes to this bug.