<?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>40182</bug_id>
          
          <creation_ts>2010-06-04 12:20:32 -0700</creation_ts>
          <short_desc>[GTK] EventSender should call gtk_main_do_event instead of invoking signal handlers directly</short_desc>
          <delta_ts>2010-11-02 17:15:25 -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>PC</rep_platform>
          <op_sys>Linux</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="Martin Robinson">mrobinson</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>xan.lopez</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>234330</commentid>
    <comment_count>0</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2010-06-04 12:20:32 -0700</bug_when>
    <thetext>Calling gtk_main_do_event is &quot;more correct.&quot; In particular, it will allow us to call gtk_get_current_event() in WebCore/WebKit without breaking LayoutTests.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>234333</commentid>
    <comment_count>1</comment_count>
      <attachid>57905</attachid>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2010-06-04 12:26:46 -0700</bug_when>
    <thetext>Created attachment 57905
Use gtk_main_do_event instead of invoking signal handlers directly</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>234336</commentid>
    <comment_count>2</comment_count>
      <attachid>57905</attachid>
    <who name="Xan Lopez">xan.lopez</who>
    <bug_when>2010-06-04 12:35:44 -0700</bug_when>
    <thetext>Comment on attachment 57905
Use gtk_main_do_event instead of invoking signal handlers directly

Looks good.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>234339</commentid>
    <comment_count>3</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2010-06-04 12:41:48 -0700</bug_when>
    <thetext>Committed r60700: &lt;http://trac.webkit.org/changeset/60700&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>258335</commentid>
    <comment_count>4</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2010-07-30 12:38:37 -0700</bug_when>
    <thetext>*** Bug 29203 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>57905</attachid>
            <date>2010-06-04 12:26:46 -0700</date>
            <delta_ts>2010-11-02 17:15:25 -0700</delta_ts>
            <desc>Use gtk_main_do_event instead of invoking signal handlers directly</desc>
            <filename>bug-40182-20100604122645.patch</filename>
            <type>text/plain</type>
            <size>2537</size>
            <attacher name="Martin Robinson">mrobinson</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCBmYWZlYTE1M2ZiMjE4Zjk1MjRlYzM5OThlOGRjNDk5Mzg0MzI0NDdiLi4xY2EwNzVh
MTVmZjE3YjhjYWM3ZTM2ZmYyMjUyM2Q5ZTAxMTQ2OGM3IDEwMDY0NAotLS0gYS9XZWJLaXRUb29s
cy9DaGFuZ2VMb2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTggQEAK
KzIwMTAtMDYtMDQgIE1hcnRpbiBSb2JpbnNvbiAgPG1yb2JpbnNvbkBpZ2FsaWEuY29tPgorCisg
ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFtHVEtdIEV2ZW50
U2VuZGVyIHNob3VsZCBjYWxsIGd0a19tYWluX2RvX2V2ZW50IGluc3RlYWQgb2YgaW52b2tpbmcg
c2lnbmFsIGhhbmRsZXJzIGRpcmVjdGx5CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn
L3Nob3dfYnVnLmNnaT9pZD00MDE4MgorCisgICAgICAgIEluc3RlYWQgb2YgaW52b2tpbmcgc2ln
bmFsIGhhbmRsZXJzIGRpcmVjdGx5IHZpYSBnX3NpZ25hbF9lbWl0X2J5X25hbWUsIGNhbGwKKyAg
ICAgICAgZ3RrX21haW5fZG9fZXZlbnQuIFRoaXMgd2lsbCBhbGxvdyB1cyB0byBjYWxsIGd0a19n
ZXRfY3VycmVudF9ldmVudCgpIGluIFdlYktpdAorICAgICAgICB3aXRob3V0IGJyZWFraW5nIERS
VC4KKworICAgICAgICAqIER1bXBSZW5kZXJUcmVlL2d0ay9FdmVudFNlbmRlci5jcHA6CisgICAg
ICAgIChkaXNwYXRjaEV2ZW50KTogVXNlIGd0a19tYWluX2RvX2V2ZW50IGluc3RlYWQgb2YgaW52
b2tpbmcgc2lnbmFsIGhhbmRsZXJzIGRpcmVjdGx5LgorICAgICAgICAoa2V5RG93bkNhbGxiYWNr
KTogVXNlIGRpc3BhdGNoRXZlbnQgaGVyZSB3aGljaCBub3cgdXNlcyBndGtfbWFpbl9kb19ldmVu
dC4KKwogMjAxMC0wNi0wNCAgWWFlbCBBaGFyb24gIDx5YWVsLmFoYXJvbkBub2tpYS5jb20+CiAK
ICAgICAgICAgUmV2aWV3ZWQgYnkgU2ltb24gSGF1c21hbm4uCmRpZmYgLS1naXQgYS9XZWJLaXRU
b29scy9EdW1wUmVuZGVyVHJlZS9ndGsvRXZlbnRTZW5kZXIuY3BwIGIvV2ViS2l0VG9vbHMvRHVt
cFJlbmRlclRyZWUvZ3RrL0V2ZW50U2VuZGVyLmNwcAppbmRleCA5ZWI0NzkxNDM5MTg2NjljYTc4
YmMzYWFhNDc2NDU0MmFiN2MwYmViLi4yZDc3YThhZGNlMmY4MzgzMTQ5OTk5OWNhMTcwYWQ4YTM0
Y2I5YTM0IDEwMDY0NAotLS0gYS9XZWJLaXRUb29scy9EdW1wUmVuZGVyVHJlZS9ndGsvRXZlbnRT
ZW5kZXIuY3BwCisrKyBiL1dlYktpdFRvb2xzL0R1bXBSZW5kZXJUcmVlL2d0ay9FdmVudFNlbmRl
ci5jcHAKQEAgLTM2NiwxNSArMzY2LDcgQEAgc3RhdGljIHZvaWQgZGlzcGF0Y2hFdmVudChHZGtF
dmVudCBldmVudCkKICAgICBpZiAoIXZpZXcpCiAgICAgICAgIHJldHVybjsKIAotICAgIGdib29s
ZWFuIHJldHVyblZhbHVlOwotICAgIGlmIChldmVudC50eXBlID09IEdES19CVVRUT05fUFJFU1Mg
fHwgZXZlbnQudHlwZSA9PSBHREtfMkJVVFRPTl9QUkVTUyB8fCBldmVudC50eXBlID09IEdES18z
QlVUVE9OX1BSRVNTKQotICAgICAgICBnX3NpZ25hbF9lbWl0X2J5X25hbWUodmlldywgImJ1dHRv
bl9wcmVzc19ldmVudCIsICZldmVudCwgJnJldHVyblZhbHVlKTsKLSAgICBlbHNlIGlmIChldmVu
dC50eXBlID09IEdES19CVVRUT05fUkVMRUFTRSkKLSAgICAgICAgZ19zaWduYWxfZW1pdF9ieV9u
YW1lKHZpZXcsICJidXR0b25fcmVsZWFzZV9ldmVudCIsICZldmVudCwgJnJldHVyblZhbHVlKTsK
LSAgICBlbHNlIGlmIChldmVudC50eXBlID09IEdES19NT1RJT05fTk9USUZZKQotICAgICAgICBn
X3NpZ25hbF9lbWl0X2J5X25hbWUodmlldywgIm1vdGlvbl9ub3RpZnlfZXZlbnQiLCAmZXZlbnQs
ICZyZXR1cm5WYWx1ZSk7Ci0gICAgZWxzZSBpZiAoZXZlbnQudHlwZSA9PSBHREtfU0NST0xMKQot
ICAgICAgICBndGtfbWFpbl9kb19ldmVudCgmZXZlbnQpOworICAgIGd0a19tYWluX2RvX2V2ZW50
KCZldmVudCk7CiB9CiAKIHZvaWQgcmVwbGF5U2F2ZWRFdmVudHMoKQpAQCAtNTM0LDEzICs1MjYs
MTEgQEAgc3RhdGljIEpTVmFsdWVSZWYga2V5RG93bkNhbGxiYWNrKEpTQ29udGV4dFJlZiBjb250
ZXh0LCBKU09iamVjdFJlZiBmdW5jdGlvbiwgSlMKICAgICAgICAgZ19mcmVlKGtleXMpOwogICAg
IH0KIAotICAgIGdib29sZWFuIHJldHVybl92YWw7CiAgICAgZXZlbnQua2V5LnR5cGUgPSBHREtf
S0VZX1BSRVNTOwotCi0gICAgZ19zaWduYWxfZW1pdF9ieV9uYW1lKHZpZXcsICJrZXktcHJlc3Mt
ZXZlbnQiLCAmZXZlbnQua2V5LCAmcmV0dXJuX3ZhbCk7CisgICAgZGlzcGF0Y2hFdmVudChldmVu
dCk7CiAKICAgICBldmVudC5rZXkudHlwZSA9IEdES19LRVlfUkVMRUFTRTsKLSAgICBnX3NpZ25h
bF9lbWl0X2J5X25hbWUodmlldywgImtleS1yZWxlYXNlLWV2ZW50IiwgJmV2ZW50LmtleSwgJnJl
dHVybl92YWwpOworICAgIGRpc3BhdGNoRXZlbnQoZXZlbnQpOwogCiAgICAgcmV0dXJuIEpTVmFs
dWVNYWtlVW5kZWZpbmVkKGNvbnRleHQpOwogfQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>