[Webkit-unassigned] [Bug 21210] New: [GTK] Linux/GTK: GtkLauncher throwing Gtk-CRITICAL errors

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=21210

           Summary: [GTK] Linux/GTK: GtkLauncher throwing Gtk-CRITICAL
                    errors
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: cmsigler at gmail.com


Hi,

I think this behaviour cropped up around the time David Hyatt was working on
ScrollView/widget refactoring.  GtkLauncher had been performing cleanly in my
quick Acid{1,2,3} testing prior to this.  Now it throws these errors:

(lt-GtkLauncher:nnnnn): Gtk-CRITICAL **: gtk_widget_queue_draw_area: assertion
`GTK_IS_WIDGET (widget)' failed

when going to the Acid2 and esp. Acid3 test pages.

So, I've looked at WebCore/platform/gtk/WidgetGtk.cpp, and
GTK_WIDGET(containingWindow()) seems to be throwing these errors, indicating
containingWindow() is not a widget(?).  Widget::invalidateRect(const IntRect&
rect) isn't called in too many places (under GTK), and I don't see any calls
where the value of rect passed to it would be invalid.

Based on my experience, I'm guessing that sometimes the value of rect being
passed is unset/NULL...?  I've futzed with the code and can't see any such
errors -- of course, I'm not very familiar with the code base....

Hope this error report might point out a simple thinko.  TIA.

Clemmitt


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.