<?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>46113</bug_id>
          
          <creation_ts>2010-09-20 12:08:46 -0700</creation_ts>
          <short_desc>Two instances of commit-queue try to process the same patch</short_desc>
          <delta_ts>2010-09-20 12:33:02 -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>New Bugs</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Other</rep_platform>
          <op_sys>OS X 10.5</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="Adam Barth">abarth</reporter>
          <assigned_to name="Adam Barth">abarth</assigned_to>
          <cc>eric</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>281727</commentid>
    <comment_count>0</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-09-20 12:08:46 -0700</bug_when>
    <thetext>Two instances of commit-queue try to process the same patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>281729</commentid>
    <comment_count>1</comment_count>
      <attachid>68119</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-09-20 12:10:10 -0700</bug_when>
    <thetext>Created attachment 68119
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>281730</commentid>
    <comment_count>2</comment_count>
      <attachid>68120</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-09-20 12:15:30 -0700</bug_when>
    <thetext>Created attachment 68120
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>281736</commentid>
    <comment_count>3</comment_count>
      <attachid>68122</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-09-20 12:24:42 -0700</bug_when>
    <thetext>Created attachment 68122
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>281738</commentid>
    <comment_count>4</comment_count>
      <attachid>68122</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-09-20 12:26:43 -0700</bug_when>
    <thetext>Comment on attachment 68122
Patch

It&apos;s sorta sad that we&apos;re not including the bug id in this message.  We could change _get_next_patch_id to be _get_next_patch and to return teh bug_id as well.  Future code might want all message to have a bug id.

We really need testing for this stuff.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>281746</commentid>
    <comment_count>5</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-09-20 12:33:02 -0700</bug_when>
    <thetext>Committed r67871: &lt;http://trac.webkit.org/changeset/67871&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>68119</attachid>
            <date>2010-09-20 12:10:10 -0700</date>
            <delta_ts>2010-09-20 12:15:27 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-46113-20100920121008.patch</filename>
            <type>text/plain</type>
            <size>1605</size>
            <attacher name="Adam Barth">abarth</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRUb29scy9D
aGFuZ2VMb2cJKHJldmlzaW9uIDY3ODY4KQorKysgV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCSh3b3Jr
aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTAtMDktMjAgIEFkYW0gQmFydGggIDxhYmFy
dGhAd2Via2l0Lm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKwor
ICAgICAgICBUd28gaW5zdGFuY2VzIG9mIGNvbW1pdC1xdWV1ZSB0cnkgdG8gcHJvY2VzcyB0aGUg
c2FtZSBwYXRjaAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9NDYxMTMKKworICAgICAgICBUaGlzIHBhdGNoIG1ha2VzIG5leHQtcGF0Y2ggYXRvbWljIHNv
IHRoYXQgdGhlIHNlcnZlciB3b24ndCB2ZW5kIHRoZQorICAgICAgICBzYW1lIHBhdGNoIHR3aWNl
IGluIHRoZSBzYW1lIGhvdXIuCisKKyAgICAgICAgKiBRdWV1ZVN0YXR1c1NlcnZlci9oYW5kbGVy
cy9uZXh0cGF0Y2gucHk6CisKIDIwMTAtMDktMjAgIEFkYW0gUm9iZW4gIDxhcm9iZW5AYXBwbGUu
Y29tPgogCiAgICAgICAgIE1ha2UgV2ViS2l0VGVzdFJ1bm5lcidzIHdhaXQtdG8tZHVtcCB3YXRj
aGRvZyB0aW1lciB3b3JrIG9uIFdpbmRvd3MKSW5kZXg6IFdlYktpdFRvb2xzL1F1ZXVlU3RhdHVz
U2VydmVyL2hhbmRsZXJzL25leHRwYXRjaC5weQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRUb29scy9R
dWV1ZVN0YXR1c1NlcnZlci9oYW5kbGVycy9uZXh0cGF0Y2gucHkJKHJldmlzaW9uIDY3ODY4KQor
KysgV2ViS2l0VG9vbHMvUXVldWVTdGF0dXNTZXJ2ZXIvaGFuZGxlcnMvbmV4dHBhdGNoLnB5CSh3
b3JraW5nIGNvcHkpCkBAIC00OCw4ICs0OCwxNiBAQCBjbGFzcyBOZXh0UGF0Y2god2ViYXBwLlJl
cXVlc3RIYW5kbGVyKToKICAgICAgICAgIyBFaXRoZXIgdGhlcmUgd2VyZSBubyB3b3JrIGl0ZW1z
LCBvciB0aGV5J3JlIGFsbCBhY3RpdmUuCiAgICAgICAgIHJldHVybiBOb25lCiAKKyAgICBkZWYg
X2Fzc2lnbihzZWxmLCBxdWV1ZV9uYW1lLCBwYXRjaF9pZCk6CisgICAgICAgIHF1ZXVlX3N0YXR1
cyA9IFF1ZXVlU3RhdHVzKCkKKyAgICAgICAgcXVldWVfc3RhdHVzLnF1ZXVlX25hbWUgPSBxdWV1
ZV9uYW1lCisgICAgICAgIHF1ZXVlX3N0YXR1cy5hY3RpdmVfcGF0Y2hfaWQgPSBwYXRjaF9pZAor
ICAgICAgICBxdWV1ZV9zdGF0dXMubWVzc2FnZSA9ICJBc3NpZ25lZCBmb3IgcHJvY2Vzc2luZyIK
KyAgICAgICAgcXVldWVfc3RhdHVzLnB1dCgpCisKICAgICBkZWYgZ2V0KHNlbGYsIHF1ZXVlX25h
bWUpOgogICAgICAgICBwYXRjaF9pZCA9IHNlbGYuX2dldF9uZXh0X3BhdGNoX2lkKHF1ZXVlX25h
bWUpCisgICAgICAgIHNlbGYuX2Fzc2lnbihxdWV1ZV9uYW1lLCBwYXRjaF9pZCkKICAgICAgICAg
aWYgbm90IHBhdGNoX2lkOgogICAgICAgICAgICAgc2VsZi5lcnJvcig0MDQpCiAgICAgICAgICAg
ICByZXR1cm4K
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>68120</attachid>
            <date>2010-09-20 12:15:30 -0700</date>
            <delta_ts>2010-09-20 12:24:38 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-46113-20100920121529.patch</filename>
            <type>text/plain</type>
            <size>1648</size>
            <attacher name="Adam Barth">abarth</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRUb29scy9D
aGFuZ2VMb2cJKHJldmlzaW9uIDY3ODY4KQorKysgV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCSh3b3Jr
aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTAtMDktMjAgIEFkYW0gQmFydGggIDxhYmFy
dGhAd2Via2l0Lm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKwor
ICAgICAgICBUd28gaW5zdGFuY2VzIG9mIGNvbW1pdC1xdWV1ZSB0cnkgdG8gcHJvY2VzcyB0aGUg
c2FtZSBwYXRjaAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9NDYxMTMKKworICAgICAgICBUaGlzIHBhdGNoIG1ha2VzIG5leHQtcGF0Y2ggYXRvbWljIHNv
IHRoYXQgdGhlIHNlcnZlciB3b24ndCB2ZW5kIHRoZQorICAgICAgICBzYW1lIHBhdGNoIHR3aWNl
IGluIHRoZSBzYW1lIGhvdXIuCisKKyAgICAgICAgKiBRdWV1ZVN0YXR1c1NlcnZlci9oYW5kbGVy
cy9uZXh0cGF0Y2gucHk6CisKIDIwMTAtMDktMjAgIEFkYW0gUm9iZW4gIDxhcm9iZW5AYXBwbGUu
Y29tPgogCiAgICAgICAgIE1ha2UgV2ViS2l0VGVzdFJ1bm5lcidzIHdhaXQtdG8tZHVtcCB3YXRj
aGRvZyB0aW1lciB3b3JrIG9uIFdpbmRvd3MKSW5kZXg6IFdlYktpdFRvb2xzL1F1ZXVlU3RhdHVz
U2VydmVyL2hhbmRsZXJzL25leHRwYXRjaC5weQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRUb29scy9R
dWV1ZVN0YXR1c1NlcnZlci9oYW5kbGVycy9uZXh0cGF0Y2gucHkJKHJldmlzaW9uIDY3ODY4KQor
KysgV2ViS2l0VG9vbHMvUXVldWVTdGF0dXNTZXJ2ZXIvaGFuZGxlcnMvbmV4dHBhdGNoLnB5CSh3
b3JraW5nIGNvcHkpCkBAIC00OCw5ICs0OCwxNyBAQCBjbGFzcyBOZXh0UGF0Y2god2ViYXBwLlJl
cXVlc3RIYW5kbGVyKToKICAgICAgICAgIyBFaXRoZXIgdGhlcmUgd2VyZSBubyB3b3JrIGl0ZW1z
LCBvciB0aGV5J3JlIGFsbCBhY3RpdmUuCiAgICAgICAgIHJldHVybiBOb25lCiAKKyAgICBkZWYg
X2Fzc2lnbihzZWxmLCBxdWV1ZV9uYW1lLCBwYXRjaF9pZCk6CisgICAgICAgIHF1ZXVlX3N0YXR1
cyA9IFF1ZXVlU3RhdHVzKCkKKyAgICAgICAgcXVldWVfc3RhdHVzLnF1ZXVlX25hbWUgPSBxdWV1
ZV9uYW1lCisgICAgICAgIHF1ZXVlX3N0YXR1cy5hY3RpdmVfcGF0Y2hfaWQgPSBwYXRjaF9pZAor
ICAgICAgICBxdWV1ZV9zdGF0dXMubWVzc2FnZSA9ICJBc3NpZ25lZCBmb3IgcHJvY2Vzc2luZyIK
KyAgICAgICAgcXVldWVfc3RhdHVzLnB1dCgpCisKICAgICBkZWYgZ2V0KHNlbGYsIHF1ZXVlX25h
bWUpOgogICAgICAgICBwYXRjaF9pZCA9IHNlbGYuX2dldF9uZXh0X3BhdGNoX2lkKHF1ZXVlX25h
bWUpCiAgICAgICAgIGlmIG5vdCBwYXRjaF9pZDoKICAgICAgICAgICAgIHNlbGYuZXJyb3IoNDA0
KQogICAgICAgICAgICAgcmV0dXJuCisgICAgICAgIHNlbGYuX2Fzc2lnbihxdWV1ZV9uYW1lLCBw
YXRjaF9pZCkKICAgICAgICAgc2VsZi5yZXNwb25zZS5vdXQud3JpdGUocGF0Y2hfaWQpCg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>68122</attachid>
            <date>2010-09-20 12:24:42 -0700</date>
            <delta_ts>2010-09-20 12:26:43 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-46113-20100920122440.patch</filename>
            <type>text/plain</type>
            <size>1660</size>
            <attacher name="Adam Barth">abarth</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRUb29scy9D
aGFuZ2VMb2cJKHJldmlzaW9uIDY3ODY4KQorKysgV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCSh3b3Jr
aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTAtMDktMjAgIEFkYW0gQmFydGggIDxhYmFy
dGhAd2Via2l0Lm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKwor
ICAgICAgICBUd28gaW5zdGFuY2VzIG9mIGNvbW1pdC1xdWV1ZSB0cnkgdG8gcHJvY2VzcyB0aGUg
c2FtZSBwYXRjaAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9NDYxMTMKKworICAgICAgICBUaGlzIHBhdGNoIG1ha2VzIG5leHQtcGF0Y2ggYXRvbWljIHNv
IHRoYXQgdGhlIHNlcnZlciB3b24ndCB2ZW5kIHRoZQorICAgICAgICBzYW1lIHBhdGNoIHR3aWNl
IGluIHRoZSBzYW1lIGhvdXIuCisKKyAgICAgICAgKiBRdWV1ZVN0YXR1c1NlcnZlci9oYW5kbGVy
cy9uZXh0cGF0Y2gucHk6CisKIDIwMTAtMDktMjAgIEFkYW0gUm9iZW4gIDxhcm9iZW5AYXBwbGUu
Y29tPgogCiAgICAgICAgIE1ha2UgV2ViS2l0VGVzdFJ1bm5lcidzIHdhaXQtdG8tZHVtcCB3YXRj
aGRvZyB0aW1lciB3b3JrIG9uIFdpbmRvd3MKSW5kZXg6IFdlYktpdFRvb2xzL1F1ZXVlU3RhdHVz
U2VydmVyL2hhbmRsZXJzL25leHRwYXRjaC5weQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRUb29scy9R
dWV1ZVN0YXR1c1NlcnZlci9oYW5kbGVycy9uZXh0cGF0Y2gucHkJKHJldmlzaW9uIDY3ODY4KQor
KysgV2ViS2l0VG9vbHMvUXVldWVTdGF0dXNTZXJ2ZXIvaGFuZGxlcnMvbmV4dHBhdGNoLnB5CSh3
b3JraW5nIGNvcHkpCkBAIC00OCw5ICs0OCwxNyBAQCBjbGFzcyBOZXh0UGF0Y2god2ViYXBwLlJl
cXVlc3RIYW5kbGVyKToKICAgICAgICAgIyBFaXRoZXIgdGhlcmUgd2VyZSBubyB3b3JrIGl0ZW1z
LCBvciB0aGV5J3JlIGFsbCBhY3RpdmUuCiAgICAgICAgIHJldHVybiBOb25lCiAKKyAgICBkZWYg
X2Fzc2lnbihzZWxmLCBxdWV1ZV9uYW1lLCBwYXRjaF9pZCk6CisgICAgICAgIHF1ZXVlX3N0YXR1
cyA9IHF1ZXVlc3RhdHVzLlF1ZXVlU3RhdHVzKCkKKyAgICAgICAgcXVldWVfc3RhdHVzLnF1ZXVl
X25hbWUgPSBxdWV1ZV9uYW1lCisgICAgICAgIHF1ZXVlX3N0YXR1cy5hY3RpdmVfcGF0Y2hfaWQg
PSBwYXRjaF9pZAorICAgICAgICBxdWV1ZV9zdGF0dXMubWVzc2FnZSA9ICJBc3NpZ25lZCBmb3Ig
cHJvY2Vzc2luZyIKKyAgICAgICAgcXVldWVfc3RhdHVzLnB1dCgpCisKICAgICBkZWYgZ2V0KHNl
bGYsIHF1ZXVlX25hbWUpOgogICAgICAgICBwYXRjaF9pZCA9IHNlbGYuX2dldF9uZXh0X3BhdGNo
X2lkKHF1ZXVlX25hbWUpCiAgICAgICAgIGlmIG5vdCBwYXRjaF9pZDoKICAgICAgICAgICAgIHNl
bGYuZXJyb3IoNDA0KQogICAgICAgICAgICAgcmV0dXJuCisgICAgICAgIHNlbGYuX2Fzc2lnbihx
dWV1ZV9uYW1lLCBwYXRjaF9pZCkKICAgICAgICAgc2VsZi5yZXNwb25zZS5vdXQud3JpdGUocGF0
Y2hfaWQpCg==
</data>
<flag name="review"
          id="57580"
          type_id="1"
          status="+"
          setter="eric"
    />
          </attachment>
      

    </bug>

</bugzilla>