NEW 126068
Replace virtual isListBox functions with enum and standalone function 
https://bugs.webkit.org/show_bug.cgi?id=126068
Summary Replace virtual isListBox functions with enum and standalone function
Antti Koivisto
Reported 2013-12-20 07:48:01 PST
Fast and modern.
Attachments
patch (22.42 KB, patch)
2013-12-20 07:53 PST, Antti Koivisto 		kling: review+
buildbot: commit-queue-
DetailsFormatted DiffDiff
Archive of layout-test-results from webkit-ews-09 for mac-mountainlion-wk2 (458.88 KB, application/zip)
2013-12-20 08:57 PST, Build Bot 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Antti Koivisto
Comment 1 2013-12-20 07:53:27 PST
Created attachment 219757 [details] patch
Andreas Kling
Comment 2 2013-12-20 08:00:19 PST
Comment on attachment 219757 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=219757&action=review > Source/WebCore/ChangeLog:3 > + Replace virtual isRenderFoo functions with bits and standalone functions Title seems a bit broad when it's only for RenderListBox so far.
Antti Koivisto
Comment 3 2013-12-20 08:24:53 PST
> Title seems a bit broad when it's only for RenderListBox so far. It is an aspirational title!
Build Bot
Comment 4 2013-12-20 08:57:49 PST
Comment on attachment 219757 [details] patch Attachment 219757 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/50568045 New failing tests: editing/execCommand/format-block-multiple-paragraphs-in-pre.html
Build Bot
Comment 5 2013-12-20 08:57:51 PST
Created attachment 219761 [details] Archive of layout-test-results from webkit-ews-09 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-09 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
Note You need to log in before you can comment on or make changes to this bug.