WebKit selectors include "-webkit-slider-thumb". This does not seem to do anything on QtWebKit.
Created attachment 104433 [details] Patch
Comment on attachment 104433 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=104433&action=review > Source/WebCore/ChangeLog:8 > + Get rid of upcasting, that prevents related crashes. > + Allow to display SliderTrack and SliderThumb without <input type=range>. The changelog should explain the changes, it is not just a quick summary of what you have done. When the patch is trivial it is ok to have a simple changelog, but in general you need a good explanation. What is the crash you are fixing? Why haven't you included a layout test for it?
(In reply to comment #2) I've filed RenderThemeQt::adjustSliderThumbSize changes separately, bug 66732 > What is the crash you are fixing? Why haven't you included a layout test for it? The test case is: <html><body><p style="-webkit-appearance: slider-vertical; height: 15px"></p></body></html> Making layout test
Comment on attachment 104433 [details] Patch > Making layout test -> Clearing review flag in the meantime.
Created attachment 104909 [details] Patch
Comment on attachment 104909 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=104909&action=review > LayoutTests/ChangeLog:7 > + Reviewed by NOBODY (OOPS!). > + A short desccription would be nice > Source/WebCore/ChangeLog:11 > + Get rid of upcasting. Allow to display SliderTrack and SliderThumb > + without <input type=range>. > + > + Reviewed by NOBODY (OOPS!). > + I think it is more common to have the Reviewed by right after the bugzilla links. > Source/WebCore/platform/qt/RenderThemeQt.cpp:-922 > - RenderSlider* renderSlider = toRenderSlider(o); > - HTMLInputElement* input = renderSlider->node()->toInputElement(); > - IntRect thumbRect = sliderThumbElementOf(input)->getRect(); > - You seem to remove a lot of code without the ChangeLog telling why that was possible > Source/WebCore/platform/qt/RenderThemeQt.cpp:953 > + option.orientation = Qt::Vertical; So it is initialized to Horizontal by default? Maybe add a comment or just add the else clause
(In reply to comment #6) > > Source/WebCore/platform/qt/RenderThemeQt.cpp:953 > > + option.orientation = Qt::Vertical; > > So it is initialized to Horizontal by default? Maybe add a comment or just add the else clause yep, http://doc.qt.nokia.com/qstyleoptionslider.html#orientation-var Same code already used in RenderThemeQt twice with no comments. I don't think it's necessary to comment it now. I'll fix the rest.
Created attachment 104927 [details] Patch
Comment on attachment 104927 [details] Patch Clearing flags on attachment: 104927 Committed r93702: <http://trac.webkit.org/changeset/93702>
All reviewed patches have been landed. Closing bug.