Summary: | [WK2] [Gtk] crash when web inspector is attached, and window height is reduced. | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | arno. <a.renevier> | ||||||||
Component: | WebKitGTK | Assignee: | arno. <a.renevier> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | cgarcia, gustavo, mrobinson, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | 106050 | ||||||||||
Bug Blocks: | |||||||||||
Attachments: |
|
Description
arno.
2013-01-03 15:26:20 PST
Created attachment 181234 [details]
patch proposal: detach inspector when webViewBase height becomes too small
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API Comment on attachment 181234 [details]
patch proposal: detach inspector when webViewBase height becomes too small
Seems that it would be a bit jarring to turn off the inspector completely if you shrank the window and then made it larger again. Why not just hide the inspector if there isn't enough space?
Created attachment 181242 [details]
patch proposal: updated patch: wrong static_cast
Comment on attachment 181242 [details]
patch proposal: updated patch: wrong static_cast
I don't think detaching the inspector is the right fix, here. There are actually two bugs, we shouldn't allocate for the children a size bigger than the parent, and we should make sure the widget allocation is always >= 1 (which is what GTK does in gtk_widget_size_allocate())
Created attachment 181299 [details]
A different approach
Committed r138803: <http://trac.webkit.org/changeset/138803> |