RenderCounter inherits from RenderObject; while RenderObject declares a 'isCounter' method, RenderCounter declares a 'isRenderCounter' method. One of these methods should be renamed to have the correct behavior (method overriding to declare a RenderObject as a RenderCounter).
Created attachment 19982 [details] rename the declared method in RenderCounter
Comment on attachment 19982 [details] rename the declared method in RenderCounter sorry... wrong review flag
Comment on attachment 19982 [details] rename the declared method in RenderCounter Please add a layout test and a change log.
Created attachment 19983 [details] rename the declared method in RenderCounter + ChangeLog I dont know how to create a layout test, since this method is only called by CounterNode (which is only used by RenderCounter...)
One idea for creating a regression test that Mitz mentioned to me on IRC is to introduce an assertion at the single point where isCounter is called such that it will fail when this change makes a difference (eg, when the renderer is a RenderCounter instance. From there it should be possible to determine how to create a regression test.
Comment on attachment 19983 [details] rename the declared method in RenderCounter + ChangeLog The patch looks fine, but please create a regression test as requested by Mitz and Mark. You can find some information on our regression tests here: <http://webkit.org/quality/testing.html>. The tests all reside in the LayoutTests directory, and any new test placed in that directory will be automatically found and run by the run-webkit-tests script.
This was fixed in <http://trac.webkit.org/changeset/35667>.