<?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>136401</bug_id>
          
          <creation_ts>2014-08-29 18:17:00 -0700</creation_ts>
          <short_desc>Make timerNestingLevel threadsafe</short_desc>
          <delta_ts>2014-08-29 18:33:40 -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>WebCore Misc.</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="Gavin Barraclough">barraclough</reporter>
          <assigned_to name="Gavin Barraclough">barraclough</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>esprehn+autocc</cc>
    
    <cc>kangil.han</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1032320</commentid>
    <comment_count>0</comment_count>
    <who name="Gavin Barraclough">barraclough</who>
    <bug_when>2014-08-29 18:17:00 -0700</bug_when>
    <thetext>timerNestingLevel, used by DOMTimer to determine whether a timer is &apos;nested&apos; (repeating, possible due to a timer rescheduling itself) is a global. Since worker threads can set timers too this is not thread safe.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1032321</commentid>
    <comment_count>1</comment_count>
      <attachid>237401</attachid>
    <who name="Gavin Barraclough">barraclough</who>
    <bug_when>2014-08-29 18:19:18 -0700</bug_when>
    <thetext>Created attachment 237401
Fix</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1032324</commentid>
    <comment_count>2</comment_count>
    <who name="Gavin Barraclough">barraclough</who>
    <bug_when>2014-08-29 18:33:40 -0700</bug_when>
    <thetext>Transmitting file data ....
Committed revision 173133.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>237401</attachid>
            <date>2014-08-29 18:19:18 -0700</date>
            <delta_ts>2014-08-29 18:22:17 -0700</delta_ts>
            <desc>Fix</desc>
            <filename>136401.1.patch</filename>
            <type>text/plain</type>
            <size>4326</size>
            <attacher name="Gavin Barraclough">barraclough</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE3MzEzMikKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDI2IEBACisyMDE0LTA4LTI5ICBHYXZpbiBC
YXJyYWNsb3VnaCAgPGJhcnJhY2xvdWdoQGFwcGxlLmNvbT4KKworICAgICAgICBNYWtlIHRpbWVy
TmVzdGluZ0xldmVsIHRocmVhZHNhZmUKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv
c2hvd19idWcuY2dpP2lkPTEzNjQwMQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09Q
UyEpLgorCisgICAgICAgIHRpbWVyTmVzdGluZ0xldmVsLCB1c2VkIGJ5IERPTVRpbWVyIHRvIGRl
dGVybWluZSB3aGV0aGVyIGEgdGltZXIgaXMgJ25lc3RlZCcKKyAgICAgICAgKHJlcGVhdGluZywg
cG9zc2libGUgZHVlIHRvIGEgdGltZXIgcmVzY2hlZHVsaW5nIGl0c2VsZikgaXMgYSBnbG9iYWwu
IFNpbmNlCisgICAgICAgIHdvcmtlciB0aHJlYWRzIGNhbiBzZXQgdGltZXJzIHRvbyB0aGlzIGlz
IG5vdCB0aHJlYWQgc2FmZS4KKworICAgICAgICAqIGRvbS9TY3JpcHRFeGVjdXRpb25Db250ZXh0
LmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OlNjcmlwdEV4ZWN1dGlvbkNvbnRleHQ6OlNjcmlwdEV4
ZWN1dGlvbkNvbnRleHQpOgorICAgICAgICAgICAgLSBhZGRlZCBpbml0aWFsaXplIG1fdGltZXJO
ZXN0aW5nTGV2ZWwKKyAgICAgICAgKiBkb20vU2NyaXB0RXhlY3V0aW9uQ29udGV4dC5oOgorICAg
ICAgICAoV2ViQ29yZTo6U2NyaXB0RXhlY3V0aW9uQ29udGV4dDo6dGltZXJOZXN0aW5nTGV2ZWwp
OgorICAgICAgICAoV2ViQ29yZTo6U2NyaXB0RXhlY3V0aW9uQ29udGV4dDo6c2V0VGltZXJOZXN0
aW5nTGV2ZWwpOgorICAgICAgICAgICAgLSBhZGRlZCBhY2Nlc3NvcnMKKyAgICAgICAgKiBwYWdl
L0RPTVRpbWVyLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkRPTVRpbWVyOjpET01UaW1lcik6Cisg
ICAgICAgIChXZWJDb3JlOjpET01UaW1lcjo6ZmlyZWQpOgorICAgICAgICAgICAgLSBtb3ZlIHRp
bWVyTmVzdGluZ0xldmVsIHRvIHRoZSBjb250ZXh0CisKIDIwMTQtMDgtMjkgIEdhdmluIEJhcnJh
Y2xvdWdoICA8YmFycmFjbG91Z2hAYXBwbGUuY29tPgogCiAgICAgICAgIERPTVRpbWVyOjptX25l
c3RpbmdMZXZlbCBpcyBwcm9uZSB0byBvdmVyZmxvdwpJbmRleDogU291cmNlL1dlYkNvcmUvZG9t
L1NjcmlwdEV4ZWN1dGlvbkNvbnRleHQuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3Jl
L2RvbS9TY3JpcHRFeGVjdXRpb25Db250ZXh0LmNwcAkocmV2aXNpb24gMTczMTE0KQorKysgU291
cmNlL1dlYkNvcmUvZG9tL1NjcmlwdEV4ZWN1dGlvbkNvbnRleHQuY3BwCSh3b3JraW5nIGNvcHkp
CkBAIC04MCw2ICs4MCw3IEBAIFNjcmlwdEV4ZWN1dGlvbkNvbnRleHQ6OlNjcmlwdEV4ZWN1dGlv
bkMKICAgICAsIG1fcmVhc29uRm9yU3VzcGVuZGluZ0FjdGl2ZURPTU9iamVjdHMoc3RhdGljX2Nh
c3Q8QWN0aXZlRE9NT2JqZWN0OjpSZWFzb25Gb3JTdXNwZW5zaW9uPigtMSkpCiAgICAgLCBtX2Fj
dGl2ZURPTU9iamVjdHNBcmVTdG9wcGVkKGZhbHNlKQogICAgICwgbV9hY3RpdmVET01PYmplY3RB
ZGRpdGlvbkZvcmJpZGRlbihmYWxzZSkKKyAgICAsIG1fdGltZXJOZXN0aW5nTGV2ZWwoMCkKICNp
ZiAhQVNTRVJUX0RJU0FCTEVECiAgICAgLCBtX2luU2NyaXB0RXhlY3V0aW9uQ29udGV4dERlc3Ry
dWN0b3IoZmFsc2UpCiAgICAgLCBtX2FjdGl2ZURPTU9iamVjdFJlbW92YWxGb3JiaWRkZW4oZmFs
c2UpCkluZGV4OiBTb3VyY2UvV2ViQ29yZS9kb20vU2NyaXB0RXhlY3V0aW9uQ29udGV4dC5oCj09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL2RvbS9TY3JpcHRFeGVjdXRpb25Db250ZXh0LmgJ
KHJldmlzaW9uIDE3MzExNCkKKysrIFNvdXJjZS9XZWJDb3JlL2RvbS9TY3JpcHRFeGVjdXRpb25D
b250ZXh0LmgJKHdvcmtpbmcgY29weSkKQEAgLTE3NSw2ICsxNzUsOSBAQCBwdWJsaWM6CiAgICAg
dmlydHVhbCBib29sIHVud3JhcENyeXB0b0tleShjb25zdCBWZWN0b3I8dWludDhfdD4mIHdyYXBw
ZWRLZXksIFZlY3Rvcjx1aW50OF90PiYga2V5KSA9IDA7CiAjZW5kaWYKIAorICAgIGludCB0aW1l
ck5lc3RpbmdMZXZlbCgpIGNvbnN0IHsgcmV0dXJuIG1fdGltZXJOZXN0aW5nTGV2ZWw7IH0KKyAg
ICB2b2lkIHNldFRpbWVyTmVzdGluZ0xldmVsKGludCB0aW1lck5lc3RpbmdMZXZlbCkgeyBtX3Rp
bWVyTmVzdGluZ0xldmVsID0gdGltZXJOZXN0aW5nTGV2ZWw7IH0KKwogcHJvdGVjdGVkOgogICAg
IGNsYXNzIEFkZENvbnNvbGVNZXNzYWdlVGFzayA6IHB1YmxpYyBUYXNrIHsKICAgICBwdWJsaWM6
CkBAIC0yMjMsNiArMjI2LDcgQEAgcHJpdmF0ZToKICNlbmRpZgogCiAgICAgYm9vbCBtX2FjdGl2
ZURPTU9iamVjdEFkZGl0aW9uRm9yYmlkZGVuOworICAgIGludCBtX3RpbWVyTmVzdGluZ0xldmVs
OwogCiAjaWYgIUFTU0VSVF9ESVNBQkxFRAogICAgIGJvb2wgbV9pblNjcmlwdEV4ZWN1dGlvbkNv
bnRleHREZXN0cnVjdG9yOwpJbmRleDogU291cmNlL1dlYkNvcmUvcGFnZS9ET01UaW1lci5jcHAK
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvcGFnZS9ET01UaW1lci5jcHAJKHJldmlzaW9u
IDE3MzEzMikKKysrIFNvdXJjZS9XZWJDb3JlL3BhZ2UvRE9NVGltZXIuY3BwCSh3b3JraW5nIGNv
cHkpCkBAIC00OSw4ICs0OSw2IEBAIHN0YXRpYyBjb25zdCBpbnQgbWF4SW50ZXJ2YWxGb3JVc2Vy
R2VzdHUKIHN0YXRpYyBjb25zdCBpbnQgbWF4VGltZXJOZXN0aW5nTGV2ZWwgPSA1Owogc3RhdGlj
IGNvbnN0IGRvdWJsZSBvbmVNaWxsaXNlY29uZCA9IDAuMDAxOwogCi1zdGF0aWMgaW50IHRpbWVy
TmVzdGluZ0xldmVsID0gMDsKLSAgICAKIHN0YXRpYyBpbmxpbmUgYm9vbCBzaG91bGRGb3J3YXJk
VXNlckdlc3R1cmUoaW50IGludGVydmFsLCBpbnQgbmVzdGluZ0xldmVsKQogewogICAgIHJldHVy
biBVc2VyR2VzdHVyZUluZGljYXRvcjo6cHJvY2Vzc2luZ1VzZXJHZXN0dXJlKCkKQEAgLTYwLDcg
KzU4LDcgQEAgc3RhdGljIGlubGluZSBib29sIHNob3VsZEZvcndhcmRVc2VyR2VzdAogCiBET01U
aW1lcjo6RE9NVGltZXIoU2NyaXB0RXhlY3V0aW9uQ29udGV4dCogY29udGV4dCwgc3RkOjp1bmlx
dWVfcHRyPFNjaGVkdWxlZEFjdGlvbj4gYWN0aW9uLCBpbnQgaW50ZXJ2YWwsIGJvb2wgc2luZ2xl
U2hvdCkKICAgICA6IFN1c3BlbmRhYmxlVGltZXIoY29udGV4dCkKLSAgICAsIG1fbmVzdGluZ0xl
dmVsKHRpbWVyTmVzdGluZ0xldmVsKQorICAgICwgbV9uZXN0aW5nTGV2ZWwoY29udGV4dC0+dGlt
ZXJOZXN0aW5nTGV2ZWwoKSkKICAgICAsIG1fYWN0aW9uKFdURjo6bW92ZShhY3Rpb24pKQogICAg
ICwgbV9vcmlnaW5hbEludGVydmFsKGludGVydmFsKQogICAgICwgbV9zaG91bGRGb3J3YXJkVXNl
ckdlc3R1cmUoc2hvdWxkRm9yd2FyZFVzZXJHZXN0dXJlKGludGVydmFsLCBtX25lc3RpbmdMZXZl
bCkpCkBAIC0xMzAsNyArMTI4LDcgQEAgdm9pZCBET01UaW1lcjo6ZmlyZWQoKQogICAgICAgICBB
U1NFUlQoIWRvY3VtZW50LT5mcmFtZSgpLT50aW1lcnNQYXVzZWQoKSk7CiAgICAgfQogI2VuZGlm
Ci0gICAgdGltZXJOZXN0aW5nTGV2ZWwgPSBzdGQ6Om1pbihtX25lc3RpbmdMZXZlbCArIDEsIG1h
eFRpbWVyTmVzdGluZ0xldmVsKTsKKyAgICBjb250ZXh0LT5zZXRUaW1lck5lc3RpbmdMZXZlbChz
dGQ6Om1pbihtX25lc3RpbmdMZXZlbCArIDEsIG1heFRpbWVyTmVzdGluZ0xldmVsKSk7CiAKICAg
ICBBU1NFUlQoIWlzU3VzcGVuZGVkKCkpOwogICAgIEFTU0VSVCghY29udGV4dC0+YWN0aXZlRE9N
T2JqZWN0c0FyZVN1c3BlbmRlZCgpKTsKQEAgLTE5Miw3ICsxOTAsNyBAQCB2b2lkIERPTVRpbWVy
OjpmaXJlZCgpCiAKICAgICBJbnNwZWN0b3JJbnN0cnVtZW50YXRpb246OmRpZEZpcmVUaW1lcihj
b29raWUpOwogCi0gICAgdGltZXJOZXN0aW5nTGV2ZWwgPSAwOworICAgIGNvbnRleHQtPnNldFRp
bWVyTmVzdGluZ0xldmVsKDApOwogfQogCiB2b2lkIERPTVRpbWVyOjpkaWRTdG9wKCkK
</data>
<flag name="review"
          id="262129"
          type_id="1"
          status="+"
          setter="thorton"
    />
          </attachment>
      

    </bug>

</bugzilla>