<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>94699</bug_id>
          
          <creation_ts>2012-08-22 06:14:27 -0700</creation_ts>
          <short_desc>[GTK] Crash when finalizing WebKitWebView</short_desc>
          <delta_ts>2012-08-22 08:53:04 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKitGTK</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Xan Lopez">xan.lopez</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>alex</cc>
    
    <cc>cgarcia</cc>
    
    <cc>mrobinson</cc>
    
    <cc>xan.lopez</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>701823</commentid>
    <comment_count>0</comment_count>
    <who name="Xan Lopez">xan.lopez</who>
    <bug_when>2012-08-22 06:14:27 -0700</bug_when>
    <thetext>Seems related to the last AC/overlay patches. This happens with AC disabled too.

Program received signal SIGSEGV, Segmentation fault.
glBindFramebufferEXT () at glapi_x86-64.S:31759
31759		movq	6688(%rax), %r11
(gdb) bt
#0  glBindFramebufferEXT () at glapi_x86-64.S:31759
#1  0x00007ffff7ade18d in WebCore::GLContextGLX::~GLContextGLX() () from /home/xan/gnome/lib64/libwebkitgtk-3.0.so.0
#2  0x00007ffff7ade259 in WebCore::GLContextGLX::~GLContextGLX() () from /home/xan/gnome/lib64/libwebkitgtk-3.0.so.0
#3  0x00007ffff7adf2b9 in WebCore::RedirectedXCompositeWindow::~RedirectedXCompositeWindow() () from /home/xan/gnome/lib64/libwebkitgtk-3.0.so.0
#4  0x00007ffff6a5ff03 in WebKit::AcceleratedCompositingContext::~AcceleratedCompositingContext() () from /home/xan/gnome/lib64/libwebkitgtk-3.0.so.0
#5  0x00007ffff6a5ff29 in WebKit::AcceleratedCompositingContext::~AcceleratedCompositingContext() () from /home/xan/gnome/lib64/libwebkitgtk-3.0.so.0
#6  0x00007ffff6a9a811 in webkit_web_view_finalize(_GObject*) () from /home/xan/gnome/lib64/libwebkitgtk-3.0.so.0
#7  0x0000000000478062 in ephy_web_view_finalize (object=0x5a32020) at ../../embed/ephy-web-view.c:1093
#8  0x00007ffff51f078c in g_object_unref (_object=0x5a32020) at gobject.c:3023
#9  0x00007ffff51eb70a in g_object_run_dispose (object=0x5a32020) at gobject.c:1063
#10 0x00007ffff5b769b0 in gtk_widget_destroy (widget=0x5a32020) at gtkwidget.c:3952
#11 0x00007ffff589d918 in gtk_bin_forall (container=0x4d11b80, include_internals=0, callback=0x7ffff5b768f9 &lt;gtk_widget_destroy&gt;, callback_data=0x0) at gtkbin.c:170
#12 0x00007ffff5a6a6dc in gtk_scrolled_window_forall (container=0x4d11b80, include_internals=0, callback=0x7ffff5b768f9 &lt;gtk_widget_destroy&gt;, callback_data=0x0) at gtkscrolledwindow.c:1598
#13 0x00007ffff5903503 in gtk_container_foreach (container=0x4d11b80, callback=0x7ffff5b768f9 &lt;gtk_widget_destroy&gt;, callback_data=0x0) at gtkcontainer.c:2088
#14 0x00007ffff5901b77 in gtk_container_destroy (widget=0x4d11b80) at gtkcontainer.c:1377
#15 0x00007ffff5a698a9 in gtk_scrolled_window_destroy (widget=0x4d11b80) at gtkscrolledwindow.c:1263
#16 0x00007ffff51e7288 in g_cclosure_marshal_VOID__VOID (closure=0x4e6460, return_value=0x0, n_param_values=1, param_values=0x7fffffff9ac0, invocation_hint=0x7fffffff99f0, marshal_data=0x7ffff5a696a9) at gmarshal.c:85
#17 0x00007ffff51e4995 in g_type_class_meta_marshal (closure=0x4e6460, return_value=0x0, n_param_values=1, param_values=0x7fffffff9ac0, invocation_hint=0x7fffffff99f0, marshal_data=0x98) at gclosure.c:970
#18 0x00007ffff51e42e0 in g_closure_invoke (closure=0x4e6460, return_value=0x0, n_param_values=1, param_values=0x7fffffff9ac0, invocation_hint=0x7fffffff99f0) at gclosure.c:777
#19 0x00007ffff5201e35 in signal_emit_unlocked_R (node=0x4ef230, detail=0, instance=0x4d11b80, emission_return=0x0, instance_and_params=0x7fffffff9ac0) at gsignal.c:3667
#20 0x00007ffff5200c14 in g_signal_emit_valist (instance=0x4d11b80, signal_id=3, detail=0, var_args=0x7fffffff9de8) at gsignal.c:3300
#21 0x00007ffff520115d in g_signal_emit (instance=0x4d11b80, signal_id=3, detail=0) at gsignal.c:3356
#22 0x00007ffff5b83187 in gtk_widget_dispose (object=0x4d11b80) at gtkwidget.c:10267
#23 0x00007ffff51eb6fe in g_object_run_dispose (object=0x4d11b80) at gobject.c:1061
#24 0x00007ffff5b769b0 in gtk_widget_destroy (widget=0x4d11b80) at gtkwidget.c:3952
#25 0x00007ffff5a1cb2d in gtk_overlay_forall (overlay=0x3975e90, include_internals=0, callback=0x7ffff5b768f9 &lt;gtk_widget_destroy&gt;, callback_data=0x0) at gtkoverlay.c:568
#26 0x00007ffff5903503 in gtk_container_foreach (container=0x3975e90, callback=0x7ffff5b768f9 &lt;gtk_widget_destroy&gt;, callback_data=0x0) at gtkcontainer.c:2088
#27 0x00007ffff5901b77 in gtk_container_destroy (widget=0x3975e90) at gtkcontainer.c:1377
#28 0x00007ffff51e7288 in g_cclosure_marshal_VOID__VOID (closure=0x4e6460, return_value=0x0, n_param_values=1, param_values=0x7fffffffa310, invocation_hint=0x7fffffffa240, marshal_data=0x7ffff5901a90) at gmarshal.c:85
#29 0x00007ffff51e4995 in g_type_class_meta_marshal (closure=0x4e6460, return_value=0x0, n_param_values=1, param_values=0x7fffffffa310, invocation_hint=0x7fffffffa240, marshal_data=0x98) at gclosure.c:970
#30 0x00007ffff51e42e0 in g_closure_invoke (closure=0x4e6460, return_value=0x0, n_param_values=1, param_values=0x7fffffffa310, invocation_hint=0x7fffffffa240) at gclosure.c:777
#31 0x00007ffff5201e35 in signal_emit_unlocked_R (node=0x4ef230, detail=0, instance=0x3975e90, emission_return=0x0, instance_and_params=0x7fffffffa310) at gsignal.c:3667
#32 0x00007ffff5200c14 in g_signal_emit_valist (instance=0x3975e90, signal_id=3, detail=0, var_args=0x7fffffffa638) at gsignal.c:3300
#33 0x00007ffff520115d in g_signal_emit (instance=0x3975e90, signal_id=3, detail=0) at gsignal.c:3356
#34 0x00007ffff5b83187 in gtk_widget_dispose (object=0x3975e90) at gtkwidget.c:10267
#35 0x00007ffff51eb6fe in g_object_run_dispose (object=0x3975e90) at gobject.c:1061
#36 0x00007ffff5b769b0 in gtk_widget_destroy (widget=0x3975e90) at gtkwidget.c:3952
#37 0x00007ffff5a222c8 in gtk_paned_forall (container=0x5a2f000, include_internals=0, callback=0x7ffff5b768f9 &lt;gtk_widget_destroy&gt;, callback_data=0x0) at gtkpaned.c:1956
#38 0x00007ffff5903503 in gtk_container_foreach (container=0x5a2f000, callback=0x7ffff5b768f9 &lt;gtk_widget_destroy&gt;, callback_data=0x0) at gtkcontainer.c:2088
#39 0x00007ffff5901b77 in gtk_container_destroy (widget=0x5a2f000) at gtkcontainer.c:1377
#40 0x00007ffff51e7288 in g_cclosure_marshal_VOID__VOID (closure=0x4e6460, return_value=0x0, n_param_values=1, param_values=0x7fffffffab50, invocation_hint=0x7fffffffaa80, marshal_data=0x7ffff5901a90) at gmarshal.c:85
#41 0x00007ffff51e4995 in g_type_class_meta_marshal (closure=0x4e6460, return_value=0x0, n_param_values=1, param_values=0x7fffffffab50, invocation_hint=0x7fffffffaa80, marshal_data=0x98) at gclosure.c:970
#42 0x00007ffff51e42e0 in g_closure_invoke (closure=0x4e6460, return_value=0x0, n_param_values=1, param_values=0x7fffffffab50, invocation_hint=0x7fffffffaa80) at gclosure.c:777
#43 0x00007ffff5201e35 in signal_emit_unlocked_R (node=0x4ef230, detail=0, instance=0x5a2f000, emission_return=0x0, instance_and_params=0x7fffffffab50) at gsignal.c:3667
#44 0x00007ffff5200c14 in g_signal_emit_valist (instance=0x5a2f000, signal_id=3, detail=0, var_args=0x7fffffffae78) at gsignal.c:3300
#45 0x00007ffff520115d in g_signal_emit (instance=0x5a2f000, signal_id=3, detail=0) at gsignal.c:3356
#46 0x00007ffff5b83187 in gtk_widget_dispose (object=0x5a2f000) at gtkwidget.c:10267
#47 0x00007ffff51eb6fe in g_object_run_dispose (object=0x5a2f000) at gobject.c:1061
#48 0x00007ffff5b769b0 in gtk_widget_destroy (widget=0x5a2f000) at gtkwidget.c:3952
#49 0x00007ffff58a52dc in gtk_box_forall (container=0x56e71a0, include_internals=0, callback=0x7ffff5b768f9 &lt;gtk_widget_destroy&gt;, callback_data=0x0) at gtkbox.c:1865
#50 0x00007ffff5903503 in gtk_container_foreach (container=0x56e71a0, callback=0x7ffff5b768f9 &lt;gtk_widget_destroy&gt;, callback_data=0x0) at gtkcontainer.c:2088
#51 0x00007ffff5901b77 in gtk_container_destroy (widget=0x56e71a0) at gtkcontainer.c:1377
#52 0x00007ffff51e7288 in g_cclosure_marshal_VOID__VOID (closure=0x4e6460, return_value=0x0, n_param_values=1, param_values=0x7fffffffb3a0, invocation_hint=0x7fffffffb2d0, marshal_data=0x7ffff5901a90) at gmarshal.c:85
#53 0x00007ffff51e4995 in g_type_class_meta_marshal (closure=0x4e6460, return_value=0x0, n_param_values=1, param_values=0x7fffffffb3a0, invocation_hint=0x7fffffffb2d0, marshal_data=0x98) at gclosure.c:970
#54 0x00007ffff51e42e0 in g_closure_invoke (closure=0x4e6460, return_value=0x0, n_param_values=1, param_values=0x7fffffffb3a0, invocation_hint=0x7fffffffb2d0) at gclosure.c:777
#55 0x00007ffff5201e35 in signal_emit_unlocked_R (node=0x4ef230, detail=0, instance=0x56e71a0, emission_return=0x0, instance_and_params=0x7fffffffb3a0) at gsignal.c:3667
#56 0x00007ffff5200c14 in g_signal_emit_valist (instance=0x56e71a0, signal_id=3, detail=0, var_args=0x7fffffffb6c8) at gsignal.c:3300
#57 0x00007ffff520115d in g_signal_emit (instance=0x56e71a0, signal_id=3, detail=0) at gsignal.c:3356
#58 0x00007ffff5b83187 in gtk_widget_dispose (object=0x56e71a0) at gtkwidget.c:10267
#59 0x000000000046db91 in ephy_embed_dispose (object=0x56e71a0) at ../../embed/ephy-embed.c:394
#60 0x00007ffff51eb6fe in g_object_run_dispose (object=0x56e71a0) at gobject.c:1061
#61 0x00007ffff5b769b0 in gtk_widget_destroy (widget=0x56e71a0) at gtkwidget.c:3952
#62 0x0000000000435f44 in notebook_page_close_request_cb (notebook=0x67a010, embed=0x56e71a0, window=0x656010) at ../../src/ephy-window.c:2988
#63 0x00007ffff51e8f21 in g_cclosure_marshal_VOID__OBJECTv (closure=0x5321d0, return_value=0x0, instance=0x67a010, args=0x7fffffffbd30, marshal_data=0x0, n_params=1, param_types=0x678e60) at gmarshal.c:1312
#64 0x00007ffff51e45c4 in _g_closure_invoke_va (closure=0x5321d0, return_value=0x0, instance=0x67a010, args=0x7fffffffbd30, n_params=1, param_types=0x678e60) at gclosure.c:840
#65 0x00007ffff51fffac in g_signal_emit_valist (instance=0x67a010, signal_id=206, detail=0, var_args=0x7fffffffbd30) at gsignal.c:3211
#66 0x00007ffff52012bb in g_signal_emit_by_name (instance=0x67a010, detailed_signal=0x4a7bed &quot;tab-close-request&quot;) at gsignal.c:3393
#67 0x0000000000440ac3 in window_cmd_file_close_window (action=0x63f730, window=0x656010) at ../../src/window-commands.c:717
#68 0x00007ffff51e7288 in g_cclosure_marshal_VOID__VOID (closure=0x665870, return_value=0x0, n_param_values=1, param_values=0x7fffffffc0b0, invocation_hint=0x7fffffffbfe0, marshal_data=0x0) at gmarshal.c:85
#69 0x00007ffff51e42e0 in g_closure_invoke (closure=0x665870, return_value=0x0, n_param_values=1, param_values=0x7fffffffc0b0, invocation_hint=0x7fffffffbfe0) at gclosure.c:777
#70 0x00007ffff5201996 in signal_emit_unlocked_R (node=0x664200, detail=0, instance=0x63f730, emission_return=0x0, instance_and_params=0x7fffffffc0b0) at gsignal.c:3551
#71 0x00007ffff5200c14 in g_signal_emit_valist (instance=0x63f730, signal_id=201, detail=0, var_args=0x7fffffffc3d8) at gsignal.c:3300
#72 0x00007ffff520115d in g_signal_emit (instance=0x63f730, signal_id=201, detail=0) at gsignal.c:3356
#73 0x00007ffff587dbd3 in _gtk_action_emit_activate (action=0x63f730) at gtkaction.c:801
#74 0x00007ffff587fd31 in closure_accel_activate (closure=0x665d90, return_value=0x7fffffffc630, n_param_values=4, param_values=0x7fffffffc730, invocation_hint=0x7fffffffc660, marshal_data=0x0) at gtkaction.c:1638
#75 0x00007ffff51e42e0 in g_closure_invoke (closure=0x665d90, return_value=0x7fffffffc630, n_param_values=4, param_values=0x7fffffffc730, invocation_hint=0x7fffffffc660) at gclosure.c:777
#76 0x00007ffff5201996 in signal_emit_unlocked_R (node=0x64c770, detail=2351, instance=0x640700, emission_return=0x7fffffffc8b0, instance_and_params=0x7fffffffc730) at gsignal.c:3551
#77 0x00007ffff5200caa in g_signal_emit_valist (instance=0x640700, signal_id=189, detail=2351, var_args=0x7fffffffca98) at gsignal.c:3310
#78 0x00007ffff520115d in g_signal_emit (instance=0x640700, signal_id=189, detail=2351) at gsignal.c:3356
#79 0x00007ffff5876b97 in gtk_accel_group_activate (accel_group=0x640700, accel_quark=2351, acceleratable=0x656010, accel_key=119, accel_mods=GDK_CONTROL_MASK) at gtkaccelgroup.c:914
#80 0x00007ffff5876cb1 in gtk_accel_groups_activate (object=0x656010, accel_key=119, accel_mods=GDK_CONTROL_MASK) at gtkaccelgroup.c:952
#81 0x00007ffff5b9e9db in gtk_window_activate_key (window=0x656010, event=0x49d89c0) at gtkwindow.c:9501
#82 0x00007ffff5b98e2a in gtk_window_key_press_event (widget=0x656010, event=0x49d89c0) at gtkwindow.c:6079
#83 0x0000000000431baa in ephy_window_key_press_event (widget=0x656010, event=0x49d89c0) at ../../src/ephy-window.c:942
#84 0x00007ffff59d00f4 in _gtk_marshal_BOOLEAN__BOXED (closure=0x4f9cc0, return_value=0x7fffffffcf40, n_param_values=2, param_values=0x7fffffffd040, invocation_hint=0x7fffffffcf70, marshal_data=0x431905) at gtkmarshalers.c:85
#85 0x00007ffff51e4995 in g_type_class_meta_marshal (closure=0x4f9cc0, return_value=0x7fffffffcf40, n_param_values=2, param_values=0x7fffffffd040, invocation_hint=0x7fffffffcf70, marshal_data=0x1b0) at gclosure.c:970
#86 0x00007ffff51e42e0 in g_closure_invoke (closure=0x4f9cc0, return_value=0x7fffffffcf40, n_param_values=2, param_values=0x7fffffffd040, invocation_hint=0x7fffffffcf70) at gclosure.c:777
#87 0x00007ffff5201b12 in signal_emit_unlocked_R (node=0x4f9d10, detail=0, instance=0x656010, emission_return=0x7fffffffd190, instance_and_params=0x7fffffffd040) at gsignal.c:3589
#88 0x00007ffff5200caa in g_signal_emit_valist (instance=0x656010, signal_id=36, detail=0, var_args=0x7fffffffd378) at gsignal.c:3310
#89 0x00007ffff520115d in g_signal_emit (instance=0x656010, signal_id=36, detail=0) at gsignal.c:3356
#90 0x00007ffff5b7b5dc in gtk_widget_event_internal (widget=0x656010, event=0x49d89c0) at gtkwidget.c:6298
#91 0x00007ffff5b7ac2c in gtk_widget_event (widget=0x656010, event=0x49d89c0) at gtkwidget.c:5955
#92 0x00007ffff59cfec1 in propagate_event (widget=0x656010, event=0x49d89c0, captured=0, topmost=0x0) at gtkmain.c:2479
#93 0x00007ffff59cffd9 in gtk_propagate_event (widget=0x656010, event=0x49d89c0) at gtkmain.c:2525
#94 0x00007ffff59ceaa8 in gtk_main_do_event (event=0x49d89c0) at gtkmain.c:1713
#95 0x00007ffff560493a in _gdk_event_emit (event=0x49d89c0) at gdkevents.c:69
#96 0x00007ffff563cf9c in gdk_event_source_dispatch (source=0x541f10, callback=0, user_data=0x0) at gdkeventsource.c:358
#97 0x00007ffff50d9b14 in g_main_dispatch (context=0x51d400) at gmain.c:2707
#98 0x00007ffff50da6aa in g_main_context_dispatch (context=0x51d400) at gmain.c:3211
#99 0x00007ffff50da88d in g_main_context_iterate (context=0x51d400, block=1, dispatch=1, self=0x6140f0) at gmain.c:3282
#100 0x00007ffff50da951 in g_main_context_iteration (context=0x51d400, may_block=1) at gmain.c:3343
#101 0x00007ffff52eb077 in g_application_run (application=0x639080, argc=1, argv=0x7fffffffd9f8) at gapplication.c:1607
#102 0x000000000042e71a in main (argc=1, argv=0x7fffffffd9f8) at ../../src/ephy-main.c:493
(gdb)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>701827</commentid>
    <comment_count>1</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2012-08-22 06:18:49 -0700</bug_when>
    <thetext>I can&apos;t reproduce the crash, so it might be a driver issue or a bug in any of the dependencies.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>701839</commentid>
    <comment_count>2</comment_count>
      <attachid>159921</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2012-08-22 06:27:54 -0700</bug_when>
    <thetext>Created attachment 159921
Workaround

This patch doesn&apos;t fix the issue, since I can&apos;t reproduce it, but at least it will fix the crash if AC is never enabled (which is actually the common case in wk1)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>701840</commentid>
    <comment_count>3</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2012-08-22 06:28:29 -0700</bug_when>
    <thetext>Xan, could you confirm the patch fixes the crash for you in ephy?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>701860</commentid>
    <comment_count>4</comment_count>
    <who name="Xan Lopez">xan.lopez</who>
    <bug_when>2012-08-22 06:54:12 -0700</bug_when>
    <thetext>(In reply to comment #3)
&gt; Xan, could you confirm the patch fixes the crash for you in ephy?

This fixes the crash reported in this bug, yep</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>701914</commentid>
    <comment_count>5</comment_count>
      <attachid>159921</attachid>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2012-08-22 08:05:34 -0700</bug_when>
    <thetext>Comment on attachment 159921
Workaround

Hrm. This will cause the window to be created and destroyed in situations like going back and forth in history, so I don&apos;t think the comment is correct.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>701926</commentid>
    <comment_count>6</comment_count>
      <attachid>159921</attachid>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2012-08-22 08:15:35 -0700</bug_when>
    <thetext>Comment on attachment 159921
Workaround

View in context: https://bugs.webkit.org/attachment.cgi?id=159921&amp;action=review

&gt; Source/WebKit/gtk/WebCoreSupport/AcceleratedCompositingContextGL.cpp:-244
&gt; -        // Shrink the offscreen window to save memory while accelerated compositing is turned off.
&gt; -        m_redirectedWindow-&gt;resize(IntSize(1, 1));
&gt; +        m_redirectedWindow = nullptr;
&gt;          m_rootLayer = nullptr;
&gt;          m_nonCompositedContentLayer = nullptr;
&gt;          m_textureMapper = nullptr;
&gt;          return;
&gt;      }
&gt;  
&gt; -    if (graphicsLayer &amp;&amp; !enabled())
&gt; -        m_redirectedWindow-&gt;resize(getWebViewSize(m_webView));
&gt; -

Instead of destroying the redirected window here, I think it&apos;d be be safer to resize it. This is important because it&apos;s quite common to change the root compositing layer to null and then back. For instance this happens when moving back and forward between pages. This will still avoid creating the redirected window when AC is never used.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>701929</commentid>
    <comment_count>7</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2012-08-22 08:20:00 -0700</bug_when>
    <thetext>(In reply to comment #5)
&gt; (From update of attachment 159921 [details])
&gt; Hrm. This will cause the window to be created and destroyed in situations like going back and forth in history, so I don&apos;t think the comment is correct.

Ah!, ok, I thought it only happened when A was enabled/disabled. I&apos;ll update the patch then.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>701934</commentid>
    <comment_count>8</comment_count>
      <attachid>159940</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2012-08-22 08:22:32 -0700</bug_when>
    <thetext>Created attachment 159940
Updated patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>701936</commentid>
    <comment_count>9</comment_count>
      <attachid>159940</attachid>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2012-08-22 08:24:00 -0700</bug_when>
    <thetext>Comment on attachment 159940
Updated patch

View in context: https://bugs.webkit.org/attachment.cgi?id=159940&amp;action=review

&gt; Source/WebKit/gtk/WebCoreSupport/AcceleratedCompositingContextGL.cpp:-244
&gt; -    if (graphicsLayer &amp;&amp; !enabled())
&gt; -        m_redirectedWindow-&gt;resize(getWebViewSize(m_webView));
&gt; -

I believe this is still required because we need to kick the old window to the proper size.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>701941</commentid>
    <comment_count>10</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2012-08-22 08:30:05 -0700</bug_when>
    <thetext>(In reply to comment #9)
&gt; (From update of attachment 159940 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=159940&amp;action=review
&gt; 
&gt; &gt; Source/WebKit/gtk/WebCoreSupport/AcceleratedCompositingContextGL.cpp:-244
&gt; &gt; -    if (graphicsLayer &amp;&amp; !enabled())
&gt; &gt; -        m_redirectedWindow-&gt;resize(getWebViewSize(m_webView));
&gt; &gt; -
&gt; 
&gt; I believe this is still required because we need to kick the old window to the proper size.

This already happens, when graphicsLayer != NULL, which is always teh case here because there&apos;s an early return when it&apos;s NULL, initalize() is called. If already enabled initialize returns early, but if not enabled, if the window already exists it&apos;s resized.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>701974</commentid>
    <comment_count>11</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2012-08-22 08:53:04 -0700</bug_when>
    <thetext>Committed r126307: &lt;http://trac.webkit.org/changeset/126307&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>159921</attachid>
            <date>2012-08-22 06:27:54 -0700</date>
            <delta_ts>2012-08-22 08:22:32 -0700</delta_ts>
            <desc>Workaround</desc>
            <filename>wk-ac-crash.diff</filename>
            <type>text/plain</type>
            <size>3223</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvZ3RrL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQv
Z3RrL0NoYW5nZUxvZwppbmRleCAyZWY3OTE5Li45NGExZmI1IDEwMDY0NAotLS0gYS9Tb3VyY2Uv
V2ViS2l0L2d0ay9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdC9ndGsvQ2hhbmdlTG9nCkBA
IC0xLDMgKzEsMjIgQEAKKzIwMTItMDgtMjIgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNp
YUBpZ2FsaWEuY29tPgorCisgICAgICAgIFtHVEtdIENyYXNoIHdoZW4gZmluYWxpemluZyBXZWJL
aXRXZWJWaWV3CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p
ZD05NDY5OQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg
IENyZWF0ZSB0aGUgb2Zmc2NyZWVuIHdpbmRvdyB0aGUgZmlyc3QgdGltZSBhY2NlbGVyYXRlZCBj
b21wb3NpdGluZworICAgICAgICBpcyBlbmFibGVkLCBhbmQgZGVzdHJveSBpdCB3aGVuIGl0J3Mg
ZGlzYWJsZWQuIEVuYWJsaW5nIGFuZAorICAgICAgICBkaXNhYmxpbmcgQUMgYXQgcnVuIHRpbWUg
ZG9lc24ndCBoYXBwZW4gb2Z0ZW4gc28KKyAgICAgICAgY3JlYXRpbmcvZGVzdHJveWluZyB0aGUg
d2luZG93IGV2ZXJ5IHRpbWUsIGxpa2Ugd2UgY3VycmVudGx5IGRvCisgICAgICAgIHdpdGggYWxs
IG90aGVyIG9iamVjdHMgaW4gdGhlIEFjY2VsZXJhdGVkQ29tcG9zaXRpbmdDb250ZXh0LAorICAg
ICAgICBzaG91bGRuJ3QgYmUgYSBwcm9ibGVtLgorCisgICAgICAgICogV2ViQ29yZVN1cHBvcnQv
QWNjZWxlcmF0ZWRDb21wb3NpdGluZ0NvbnRleHRHTC5jcHA6CisgICAgICAgIChXZWJLaXQ6OkFj
Y2VsZXJhdGVkQ29tcG9zaXRpbmdDb250ZXh0OjpBY2NlbGVyYXRlZENvbXBvc2l0aW5nQ29udGV4
dCk6CisgICAgICAgIChXZWJLaXQ6OkFjY2VsZXJhdGVkQ29tcG9zaXRpbmdDb250ZXh0Ojppbml0
aWFsaXplKToKKyAgICAgICAgKFdlYktpdDo6QWNjZWxlcmF0ZWRDb21wb3NpdGluZ0NvbnRleHQ6
OnNldFJvb3RDb21wb3NpdGluZ0xheWVyKToKKwogMjAxMi0wOC0yMSAgSm9hbm1hcmllIERpZ2dz
ICA8amRpZ2dzQGlnYWxpYS5jb20+CiAgICAgICAgIFtHdGtdIE5vIGFjY2Vzc2libGUgY2FyZXQt
bW92ZWQgZXZlbnRzIGZvdW5kIGluIGNlcnRhaW4gY29udGVudAogICAgICAgICBodHRwczovL2J1
Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NzI4MTEKZGlmZiAtLWdpdCBhL1NvdXJjZS9X
ZWJLaXQvZ3RrL1dlYkNvcmVTdXBwb3J0L0FjY2VsZXJhdGVkQ29tcG9zaXRpbmdDb250ZXh0R0wu
Y3BwIGIvU291cmNlL1dlYktpdC9ndGsvV2ViQ29yZVN1cHBvcnQvQWNjZWxlcmF0ZWRDb21wb3Np
dGluZ0NvbnRleHRHTC5jcHAKaW5kZXggMjliYTM0OS4uNWJjMDFkMSAxMDA2NDQKLS0tIGEvU291
cmNlL1dlYktpdC9ndGsvV2ViQ29yZVN1cHBvcnQvQWNjZWxlcmF0ZWRDb21wb3NpdGluZ0NvbnRl
eHRHTC5jcHAKKysrIGIvU291cmNlL1dlYktpdC9ndGsvV2ViQ29yZVN1cHBvcnQvQWNjZWxlcmF0
ZWRDb21wb3NpdGluZ0NvbnRleHRHTC5jcHAKQEAgLTUwLDcgKzUwLDYgQEAgbmFtZXNwYWNlIFdl
YktpdCB7CiBBY2NlbGVyYXRlZENvbXBvc2l0aW5nQ29udGV4dDo6QWNjZWxlcmF0ZWRDb21wb3Np
dGluZ0NvbnRleHQoV2ViS2l0V2ViVmlldyogd2ViVmlldykKICAgICA6IG1fd2ViVmlldyh3ZWJW
aWV3KQogICAgICwgbV9sYXllckZsdXNoVGltZXJDYWxsYmFja0lkKDApCi0gICAgLCBtX3JlZGly
ZWN0ZWRXaW5kb3coUmVkaXJlY3RlZFhDb21wb3NpdGVXaW5kb3c6OmNyZWF0ZShJbnRTaXplKDEs
IDEpKSkKICAgICAsIG1fbGFzdEZsdXNoVGltZSgwKQogICAgICwgbV9yZWRyYXdQZW5kaW5nVGlt
ZSgwKQogICAgICwgbV9uZWVkc0V4dHJhRmx1c2goZmFsc2UpCkBAIC02OSwxMCArNjgsMTQgQEAg
dm9pZCBBY2NlbGVyYXRlZENvbXBvc2l0aW5nQ29udGV4dDo6aW5pdGlhbGl6ZSgpCiAgICAgaWYg
KG1fcm9vdExheWVyKQogICAgICAgICByZXR1cm47CiAKKyAgICBJbnRTaXplIHBhZ2VTaXplID0g
Z2V0V2ViVmlld1NpemUobV93ZWJWaWV3KTsKKyAgICBpZiAoIW1fcmVkaXJlY3RlZFdpbmRvdykK
KyAgICAgICAgbV9yZWRpcmVjdGVkV2luZG93ID0gUmVkaXJlY3RlZFhDb21wb3NpdGVXaW5kb3c6
OmNyZWF0ZShwYWdlU2l6ZSk7CisgICAgZWxzZQorICAgICAgICBtX3JlZGlyZWN0ZWRXaW5kb3ct
PnJlc2l6ZShwYWdlU2l6ZSk7CisKICAgICBtX3Jvb3RMYXllciA9IEdyYXBoaWNzTGF5ZXI6OmNy
ZWF0ZSh0aGlzKTsKICAgICBtX3Jvb3RMYXllci0+c2V0RHJhd3NDb250ZW50KGZhbHNlKTsKLQot
ICAgIEludFNpemUgcGFnZVNpemUgPSBnZXRXZWJWaWV3U2l6ZShtX3dlYlZpZXcpOwogICAgIG1f
cm9vdExheWVyLT5zZXRTaXplKHBhZ2VTaXplKTsKIAogICAgIC8vIFRoZSBub24tY29tcG9zaXRl
ZCBjb250ZW50cyBhcmUgYSBjaGlsZCBvZiB0aGUgcm9vdCBsYXllci4KQEAgLTIzMSwxNyArMjM0
LDEzIEBAIHZvaWQgQWNjZWxlcmF0ZWRDb21wb3NpdGluZ0NvbnRleHQ6OnNldFJvb3RDb21wb3Np
dGluZ0xheWVyKEdyYXBoaWNzTGF5ZXIqIGdyYXBoCiAgICAgaWYgKCFncmFwaGljc0xheWVyKSB7
CiAgICAgICAgIHN0b3BBbnlQZW5kaW5nTGF5ZXJGbHVzaCgpOwogCi0gICAgICAgIC8vIFNocmlu
ayB0aGUgb2Zmc2NyZWVuIHdpbmRvdyB0byBzYXZlIG1lbW9yeSB3aGlsZSBhY2NlbGVyYXRlZCBj
b21wb3NpdGluZyBpcyB0dXJuZWQgb2ZmLgotICAgICAgICBtX3JlZGlyZWN0ZWRXaW5kb3ctPnJl
c2l6ZShJbnRTaXplKDEsIDEpKTsKKyAgICAgICAgbV9yZWRpcmVjdGVkV2luZG93ID0gbnVsbHB0
cjsKICAgICAgICAgbV9yb290TGF5ZXIgPSBudWxscHRyOwogICAgICAgICBtX25vbkNvbXBvc2l0
ZWRDb250ZW50TGF5ZXIgPSBudWxscHRyOwogICAgICAgICBtX3RleHR1cmVNYXBwZXIgPSBudWxs
cHRyOwogICAgICAgICByZXR1cm47CiAgICAgfQogCi0gICAgaWYgKGdyYXBoaWNzTGF5ZXIgJiYg
IWVuYWJsZWQoKSkKLSAgICAgICAgbV9yZWRpcmVjdGVkV2luZG93LT5yZXNpemUoZ2V0V2ViVmll
d1NpemUobV93ZWJWaWV3KSk7Ci0KICAgICAvLyBBZGQgdGhlIGFjY2VsZXJhdGVkIGxheWVyIHRy
ZWUgaGllcmFyY2h5LgogICAgIGluaXRpYWxpemUoKTsKICAgICBtX25vbkNvbXBvc2l0ZWRDb250
ZW50TGF5ZXItPnJlbW92ZUFsbENoaWxkcmVuKCk7Cg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>159940</attachid>
            <date>2012-08-22 08:22:32 -0700</date>
            <delta_ts>2012-08-22 08:24:00 -0700</delta_ts>
            <desc>Updated patch</desc>
            <filename>wk-ac-crash2.diff</filename>
            <type>text/plain</type>
            <size>2658</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvZ3RrL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQv
Z3RrL0NoYW5nZUxvZwppbmRleCAzMGI3NWVkLi42YWYxYjVlIDEwMDY0NAotLS0gYS9Tb3VyY2Uv
V2ViS2l0L2d0ay9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdC9ndGsvQ2hhbmdlTG9nCkBA
IC0xLDUgKzEsMjEgQEAKIDIwMTItMDgtMjIgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNp
YUBpZ2FsaWEuY29tPgogCisgICAgICAgIFtHVEtdIENyYXNoIHdoZW4gZmluYWxpemluZyBXZWJL
aXRXZWJWaWV3CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p
ZD05NDY5OQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg
IENyZWF0ZSB0aGUgb2Zmc2NyZWVuIHdpbmRvdyB0aGUgZmlyc3QgdGltZSBhY2NlbGVyYXRlZCBj
b21wb3NpdGluZworICAgICAgICBpcyBlbmFibGVkLCBzbyB0aGF0IGlmIGl0J3MgbmV2ZXIgZW5h
YmxlZCB0aGUgd2luZG93IHdvbid0IGJlCisgICAgICAgIGNyZWF0ZWQuCisKKyAgICAgICAgKiBX
ZWJDb3JlU3VwcG9ydC9BY2NlbGVyYXRlZENvbXBvc2l0aW5nQ29udGV4dEdMLmNwcDoKKyAgICAg
ICAgKFdlYktpdDo6QWNjZWxlcmF0ZWRDb21wb3NpdGluZ0NvbnRleHQ6OkFjY2VsZXJhdGVkQ29t
cG9zaXRpbmdDb250ZXh0KToKKyAgICAgICAgKFdlYktpdDo6QWNjZWxlcmF0ZWRDb21wb3NpdGlu
Z0NvbnRleHQ6OmluaXRpYWxpemUpOgorICAgICAgICAoV2ViS2l0OjpBY2NlbGVyYXRlZENvbXBv
c2l0aW5nQ29udGV4dDo6c2V0Um9vdENvbXBvc2l0aW5nTGF5ZXIpOgorCisyMDEyLTA4LTIyICBD
YXJsb3MgR2FyY2lhIENhbXBvcyAgPGNnYXJjaWFAaWdhbGlhLmNvbT4KKwogICAgICAgICBbR1RL
XSBQcmVmZXJyZWQgbGFuZ3VhZ2VzIGFuZCBzcGVsbGNoZWNrZXIgQVBJcyBhcmUgbm90IGNvbnNp
c3RlbnQgaW4gV2ViS2l0MgogICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9OTQ2ODMKIApkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdC9ndGsvV2ViQ29yZVN1
cHBvcnQvQWNjZWxlcmF0ZWRDb21wb3NpdGluZ0NvbnRleHRHTC5jcHAgYi9Tb3VyY2UvV2ViS2l0
L2d0ay9XZWJDb3JlU3VwcG9ydC9BY2NlbGVyYXRlZENvbXBvc2l0aW5nQ29udGV4dEdMLmNwcApp
bmRleCAyOWJhMzQ5Li45YTJjYWUwIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0L2d0ay9XZWJD
b3JlU3VwcG9ydC9BY2NlbGVyYXRlZENvbXBvc2l0aW5nQ29udGV4dEdMLmNwcAorKysgYi9Tb3Vy
Y2UvV2ViS2l0L2d0ay9XZWJDb3JlU3VwcG9ydC9BY2NlbGVyYXRlZENvbXBvc2l0aW5nQ29udGV4
dEdMLmNwcApAQCAtNTAsNyArNTAsNiBAQCBuYW1lc3BhY2UgV2ViS2l0IHsKIEFjY2VsZXJhdGVk
Q29tcG9zaXRpbmdDb250ZXh0OjpBY2NlbGVyYXRlZENvbXBvc2l0aW5nQ29udGV4dChXZWJLaXRX
ZWJWaWV3KiB3ZWJWaWV3KQogICAgIDogbV93ZWJWaWV3KHdlYlZpZXcpCiAgICAgLCBtX2xheWVy
Rmx1c2hUaW1lckNhbGxiYWNrSWQoMCkKLSAgICAsIG1fcmVkaXJlY3RlZFdpbmRvdyhSZWRpcmVj
dGVkWENvbXBvc2l0ZVdpbmRvdzo6Y3JlYXRlKEludFNpemUoMSwgMSkpKQogICAgICwgbV9sYXN0
Rmx1c2hUaW1lKDApCiAgICAgLCBtX3JlZHJhd1BlbmRpbmdUaW1lKDApCiAgICAgLCBtX25lZWRz
RXh0cmFGbHVzaChmYWxzZSkKQEAgLTY5LDEwICs2OCwxNCBAQCB2b2lkIEFjY2VsZXJhdGVkQ29t
cG9zaXRpbmdDb250ZXh0Ojppbml0aWFsaXplKCkKICAgICBpZiAobV9yb290TGF5ZXIpCiAgICAg
ICAgIHJldHVybjsKIAorICAgIEludFNpemUgcGFnZVNpemUgPSBnZXRXZWJWaWV3U2l6ZShtX3dl
YlZpZXcpOworICAgIGlmICghbV9yZWRpcmVjdGVkV2luZG93KQorICAgICAgICBtX3JlZGlyZWN0
ZWRXaW5kb3cgPSBSZWRpcmVjdGVkWENvbXBvc2l0ZVdpbmRvdzo6Y3JlYXRlKHBhZ2VTaXplKTsK
KyAgICBlbHNlCisgICAgICAgIG1fcmVkaXJlY3RlZFdpbmRvdy0+cmVzaXplKHBhZ2VTaXplKTsK
KwogICAgIG1fcm9vdExheWVyID0gR3JhcGhpY3NMYXllcjo6Y3JlYXRlKHRoaXMpOwogICAgIG1f
cm9vdExheWVyLT5zZXREcmF3c0NvbnRlbnQoZmFsc2UpOwotCi0gICAgSW50U2l6ZSBwYWdlU2l6
ZSA9IGdldFdlYlZpZXdTaXplKG1fd2ViVmlldyk7CiAgICAgbV9yb290TGF5ZXItPnNldFNpemUo
cGFnZVNpemUpOwogCiAgICAgLy8gVGhlIG5vbi1jb21wb3NpdGVkIGNvbnRlbnRzIGFyZSBhIGNo
aWxkIG9mIHRoZSByb290IGxheWVyLgpAQCAtMjM5LDkgKzI0Miw2IEBAIHZvaWQgQWNjZWxlcmF0
ZWRDb21wb3NpdGluZ0NvbnRleHQ6OnNldFJvb3RDb21wb3NpdGluZ0xheWVyKEdyYXBoaWNzTGF5
ZXIqIGdyYXBoCiAgICAgICAgIHJldHVybjsKICAgICB9CiAKLSAgICBpZiAoZ3JhcGhpY3NMYXll
ciAmJiAhZW5hYmxlZCgpKQotICAgICAgICBtX3JlZGlyZWN0ZWRXaW5kb3ctPnJlc2l6ZShnZXRX
ZWJWaWV3U2l6ZShtX3dlYlZpZXcpKTsKLQogICAgIC8vIEFkZCB0aGUgYWNjZWxlcmF0ZWQgbGF5
ZXIgdHJlZSBoaWVyYXJjaHkuCiAgICAgaW5pdGlhbGl6ZSgpOwogICAgIG1fbm9uQ29tcG9zaXRl
ZENvbnRlbnRMYXllci0+cmVtb3ZlQWxsQ2hpbGRyZW4oKTsK
</data>
<flag name="review"
          id="170368"
          type_id="1"
          status="+"
          setter="mrobinson"
    />
    <flag name="commit-queue"
          id="170369"
          type_id="3"
          status="-"
          setter="mrobinson"
    />
          </attachment>
      

    </bug>

</bugzilla>