<?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>54518</bug_id>
          
          <creation_ts>2011-02-15 17:54:20 -0800</creation_ts>
          <short_desc>Fix race condition in JavaScriptAudioNode</short_desc>
          <delta_ts>2011-02-15 18:13:23 -0800</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>0</everconfirmed>
          <reporter name="Chris Rogers">crogers</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>kbr</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>351785</commentid>
    <comment_count>0</comment_count>
    <who name="Chris Rogers">crogers</who>
    <bug_when>2011-02-15 17:54:20 -0800</bug_when>
    <thetext>Fix race condition in JavaScriptAudioNode</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>351787</commentid>
    <comment_count>1</comment_count>
      <attachid>82564</attachid>
    <who name="Chris Rogers">crogers</who>
    <bug_when>2011-02-15 17:56:06 -0800</bug_when>
    <thetext>Created attachment 82564
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>351789</commentid>
    <comment_count>2</comment_count>
    <who name="Chris Rogers">crogers</who>
    <bug_when>2011-02-15 17:58:36 -0800</bug_when>
    <thetext>m_isRequestOutstanding needs to be set before callOnMainThread() to avoid race condition with its value being set in the main thread.

This fixes chromium bug:
http://code.google.com/p/chromium/issues/detail?id=71702

The race condition could also happen for other ports.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>351790</commentid>
    <comment_count>3</comment_count>
      <attachid>82564</attachid>
    <who name="Kenneth Russell">kbr</who>
    <bug_when>2011-02-15 18:01:12 -0800</bug_when>
    <thetext>Comment on attachment 82564
Patch

Looks fine to me. Tricky.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>351799</commentid>
    <comment_count>4</comment_count>
    <who name="Chris Rogers">crogers</who>
    <bug_when>2011-02-15 18:13:23 -0800</bug_when>
    <thetext>Committed r78671: &lt;http://trac.webkit.org/changeset/78671&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>82564</attachid>
            <date>2011-02-15 17:56:06 -0800</date>
            <delta_ts>2011-02-15 18:01:12 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-54518-20110215175605.patch</filename>
            <type>text/plain</type>
            <size>1357</size>
            <attacher name="Chris Rogers">crogers</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDc4NjYyKQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTEtMDItMTUgIENocmlzIFJv
Z2VycyAgPGNyb2dlcnNAZ29vZ2xlLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkg
KE9PUFMhKS4KKworICAgICAgICBGaXggcmFjZSBjb25kaXRpb24gaW4gSmF2YVNjcmlwdEF1ZGlv
Tm9kZQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTQ1
MTgKKworICAgICAgICBObyBuZXcgdGVzdHMgc2luY2UgYXVkaW8gQVBJIGlzIG5vdCB5ZXQgaW1w
bGVtZW50ZWQuCisKKyAgICAgICAgKiB3ZWJhdWRpby9KYXZhU2NyaXB0QXVkaW9Ob2RlLmNwcDoK
KyAgICAgICAgKFdlYkNvcmU6OkphdmFTY3JpcHRBdWRpb05vZGU6OnByb2Nlc3MpOgorCiAyMDEx
LTAyLTEwICBKZXJlbXkgT3Jsb3cgIDxqb3Jsb3dAY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJl
dmlld2VkIGJ5IE5hdGUgQ2hhcGluLgpJbmRleDogU291cmNlL1dlYkNvcmUvd2ViYXVkaW8vSmF2
YVNjcmlwdEF1ZGlvTm9kZS5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvd2ViYXVk
aW8vSmF2YVNjcmlwdEF1ZGlvTm9kZS5jcHAJKHJldmlzaW9uIDc4NjE5KQorKysgU291cmNlL1dl
YkNvcmUvd2ViYXVkaW8vSmF2YVNjcmlwdEF1ZGlvTm9kZS5jcHAJKHdvcmtpbmcgY29weSkKQEAg
LTIwNSw4ICsyMDUsOCBAQCB2b2lkIEphdmFTY3JpcHRBdWRpb05vZGU6OnByb2Nlc3Moc2l6ZV90
CiAgICAgICAgICAgICAKICAgICAgICAgICAgIC8vIEZpcmUgdGhlIGV2ZW50IG9uIHRoZSBtYWlu
IHRocmVhZCwgbm90IHRoaXMgb25lICh3aGljaCBpcyB0aGUgcmVhbHRpbWUgYXVkaW8gdGhyZWFk
KS4KICAgICAgICAgICAgIG1fZG91YmxlQnVmZmVySW5kZXhGb3JFdmVudCA9IG1fZG91YmxlQnVm
ZmVySW5kZXg7Ci0gICAgICAgICAgICBjYWxsT25NYWluVGhyZWFkKGZpcmVQcm9jZXNzRXZlbnRE
aXNwYXRjaCwgdGhpcyk7CiAgICAgICAgICAgICBtX2lzUmVxdWVzdE91dHN0YW5kaW5nID0gdHJ1
ZTsKKyAgICAgICAgICAgIGNhbGxPbk1haW5UaHJlYWQoZmlyZVByb2Nlc3NFdmVudERpc3BhdGNo
LCB0aGlzKTsKICAgICAgICAgfQogCiAgICAgICAgIHN3YXBCdWZmZXJzKCk7Cg==
</data>
<flag name="review"
          id="74236"
          type_id="1"
          status="+"
          setter="kbr"
    />
          </attachment>
      

    </bug>

</bugzilla>