[webkit-reviews] review requested: [Bug 29431] [Qt] REGRESSION:(r50665) QWebFrame::setScrollBarPolicy(Qt::Vertical, Qt::ScrollBarAlwaysOff) has no effect. : [Attachment 52325] patch 1 - adds lock scrollbars concept to ScrollView

[bugzilla-daemon at webkit.org]

Antonio Gomes (tonikitoo) <tonikitoo at webkit.org> has asked  for review:
Bug 29431: [Qt] REGRESSION:(r50665)
QWebFrame::setScrollBarPolicy(Qt::Vertical,Qt::ScrollBarAlwaysOff) has no
effect.
https://bugs.webkit.org/show_bug.cgi?id=29431

Attachment 52325: patch 1 - adds lock scrollbars concept to ScrollView
https://bugs.webkit.org/attachment.cgi?id=52325&action=review

------- Additional Comments from Antonio Gomes (tonikitoo)
<tonikitoo at webkit.org>
Patch introduces a lock scrollbars concept to ScrollView, as in
WebDynamicScrollBarsView.mm/h on WebKit/mac. It is needed because in QtWebKit,
we have Api for setting both vertical and horizontal scrollbars on/off/auto.
When it is set to off, for example, it should remain as such, unless unset.


@hyatt, do you think this approach is feasible?