[webkit-reviews] review requested: [Bug 31281] r50665 broke around 100 Qt layout tests : [Attachment 42860] proposed patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Nov 10 06:46:10 PST 2009


Csaba Osztrogonac <ossy at webkit.org> has asked  for review:
Bug 31281: r50665 broke around 100 Qt layout tests
https://bugs.webkit.org/show_bug.cgi?id=31281

Attachment 42860: proposed patch
https://bugs.webkit.org/attachment.cgi?id=42860&action=review

------- Additional Comments from Csaba Osztrogonac <ossy at webkit.org>
FrameView::FrameView() set initial value of m_canHaveScrollbars(true), but
DumpRenderTree of Qt disable scrollbars by default:

WebKitTools/DumpRenderTree/qt/DumpRenderTree.cpp :
252	    m_page->mainFrame()->setScrollBarPolicy(Qt::Horizontal,
Qt::ScrollBarAlwaysOff);
253	    m_page->mainFrame()->setScrollBarPolicy(Qt::Vertical,
Qt::ScrollBarAlwaysOff);

If we call updateCanHaveScrollbars(), it makes QtBuildBot happy. :)

Only we have only one problem: The new fast/overflow/scrollbar-restored.html
won't work on Qt, because scrollbars are disabled by DumpRenderTree. After
resizing scrollbars won't be appear if it is disabled ...

Should we enable scrollbars? It would make some tests fail ...
Or should we skip only this new test?


More information about the webkit-reviews mailing list