[Webkit-unassigned] [Bug 186638] [GTK] Tests that are failing on the EWS test queue

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jun 18 07:18:45 PDT 2018


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

--- Comment #14 from Michael Catanzaro <mcatanzaro at igalia.com> ---
(In reply to Carlos Alberto Lopez Perez from comment #11)
> (In reply to Carlos Alberto Lopez Perez from comment #10)
> 
> > Regressions: Unexpected crashes (3)
> >   fast/hidpi/broken-image-icon-very-hidpi.html [ Crash ]
> >   http/tests/misc/cached-scripts.html [ Crash ]
> >   legacy-animation-engine/fast/css/getFloatValueForUnit.html [ Crash ]

Yes, as expected. I was planning to watch the EWS this weekend to try to find backtraces for the crashing tests, but it was broken all weekend. Anyway, I intend to help with this. I don't think it's too much effort to take the backtrace and report a bug when EWS finds a crashing test.

> Crash log for fast/hidpi/broken-image-icon-very-hidpi.html obtained on the
> EWS bot: http://termbin.com/a4of

Weird... although you should probably install gnome-icon-theme, I think this is a WebKit bug worth reporting. Our current code assumes that we have missingImage icons in the GResource, which is apparently not true. See here:

static Ref<Image> loadMissingImageIconFromTheme(const char* name)
{
    // ...
    if (iconInfo) {
        // Is the icon installed?
        return WTFMove(icon);
    }

    return loadImageFromGResource(name);
}

That's not right, we shouldn't continue on to loadImageFromGResource if the image is not guaranteed to be in the resource.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20180618/15490e81/attachment.html>


More information about the webkit-unassigned mailing list