Bug 150620 - [GTK] Epiphany with Debug mode compiled WebKitGtk+ 2.10.3 hits ASSERT_ARG on willEnterAcceleratedCompositingMode
Summary: [GTK] Epiphany with Debug mode compiled WebKitGtk+ 2.10.3 hits ASSERT_ARG on ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-10-28 04:43 PDT by Andres Gomez Garcia
Modified: 2015-11-06 01:53 PST (History)
6 users (show)

See Also:


Attachments
BT from gdb (29.43 KB, text/plain)
2015-10-28 04:43 PDT, Andres Gomez Garcia
no flags Details
Patch (2.26 KB, patch)
2015-11-04 03:56 PST, Carlos Garcia Campos
mario: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andres Gomez Garcia 2015-10-28 04:43:51 PDT
Created attachment 264208 [details]
BT from gdb

I'm using WebKitGtk+ with my own JHBuild setting:
https://github.com/tanty/jhbuild-epiphany/tree/wkgtk-devel

Epiphany 3.18.0 and WebKit 2.10.3

I'm running Epiphany with the dconf key:

"process-model" = "shared-secondary-process"

When visiting several pages, eventually, epiphany crashes.

This bug is not reproducible in a predictable way but happens quite often.

--

It seems, the crash happens when hitting:

#1  0x00007ffff13d6471 in willEnterAcceleratedCompositingMode () at /opt/gnome-os/tanty/epiphany/checkout/webkitgtk-2.10.3/Source/WebKit2/UIProcess/DrawingAreaProxyImpl.cpp:213
213         ASSERT_ARG(backingStoreStateID, backingStoreStateID <= m_currentBackingStoreStateID);
Comment 1 Andres Gomez Garcia 2015-10-28 08:25:29 PDT
I forgot to comment. The compilation was done with CMake args:

"-DPORT=GTK -DCMAKE_BUILD_TYPE=Debug -DDEVELOPER_MODE=ON -DCMAKE_C_FLAGS_DEBUG=-g1 -DCMAKE_CXX_FLAGS_DEBUG=-g1"
Comment 2 Carlos Garcia Campos 2015-11-02 01:40:27 PST
My guess is that the web process enters in AC mode due to the layout that happens in UpdateBackingStorateState, and the WillEnterAC message is sent to the UI process before the DidUpdateBackingStoreState one. I'll investigate this.
Comment 3 Carlos Garcia Campos 2015-11-04 03:56:59 PST
Created attachment 264789 [details]
Patch
Comment 4 Mario Sanchez Prada 2015-11-04 05:15:12 PST
Comment on attachment 264789 [details]
Patch

Makes sense to me, thanks!
Comment 5 Carlos Garcia Campos 2015-11-04 23:13:45 PST
Committed r192052: <http://trac.webkit.org/changeset/192052>
Comment 6 Andres Gomez Garcia 2015-11-06 01:53:07 PST
(In reply to comment #5)
> Committed r192052: <http://trac.webkit.org/changeset/192052>

Thanks a lot! :)