Bug 105039 - [GTK][AC] Fix assertion failures for AC with clutter
Summary: [GTK][AC] Fix assertion failures for AC with clutter
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL: http://www.webkit.org/blog-files/3d-t...
Keywords:
Depends on:
Blocks: 105699
  Show dependency tree / graph
 
Reported: 2012-12-14 09:59 PST by ChangSeok Oh
Modified: 2012-12-23 17:19 PST (History)
4 users (show)

See Also:

Attachments
Patch (2.74 KB, patch)
2012-12-14 10:22 PST, ChangSeok Oh 		no flags Details | Formatted Diff | Diff
Patch (2.75 KB, patch)
2012-12-17 07:19 PST, ChangSeok Oh 		no flags Details | Formatted Diff | Diff
Patch (2.80 KB, patch)
2012-12-18 08:15 PST, ChangSeok Oh 		no flags Details | Formatted Diff | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description ChangSeok Oh 2012-12-14 09:59:46 PST 
I found two assertion failures while running AC(clutter backend) test page in debug build.
Comment 1 ChangSeok Oh 2012-12-14 10:22:55 PST 
Created attachment 179496 [details]
Patch
Comment 2 WebKit Review Bot 2012-12-14 11:17:08 PST 
Comment on attachment 179496 [details]
Patch

Attachment 179496 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/15313771

New failing tests:
inspector-protocol/debugger-terminate-dedicated-worker-while-paused.html
Comment 3 Build Bot 2012-12-14 12:10:35 PST 
Comment on attachment 179496 [details]
Patch

Attachment 179496 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/15309778

New failing tests:
fast/frames/sandboxed-iframe-attribute-parsing.html
Comment 4 ChangSeok Oh 2012-12-17 07:19:56 PST 
Created attachment 179737 [details]
Patch
Comment 5 Gustavo Noronha (kov) 2012-12-17 11:10:39 PST 
Comment on attachment 179737 [details]
Patch

Makes sense, but can you add notImplemented() calls for the AcceleratedCompositingContext methods you removed the asserts from?
Comment 6 ChangSeok Oh 2012-12-18 08:15:24 PST 
Created attachment 179945 [details]
Patch
Comment 7 WebKit Review Bot 2012-12-18 18:33:37 PST 
Comment on attachment 179945 [details]
Patch

Clearing flags on attachment: 179945

Committed r138100: <http://trac.webkit.org/changeset/138100>
Comment 8 WebKit Review Bot 2012-12-18 18:33:41 PST 
All reviewed patches have been landed.  Closing bug.