<?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>67501</bug_id>
          
          <creation_ts>2011-09-02 10:33:25 -0700</creation_ts>
          <short_desc>set DOMTimer&apos;s default min timeout from 10ms to 4ms as per w3 WG</short_desc>
          <delta_ts>2011-09-02 22:18:56 -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>DOM</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>DUPLICATE</resolution>
          <dup_id>61214</dup_id>
          
          <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="Srikumar B">srikumar.b</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>srikumar.b</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>461300</commentid>
    <comment_count>0</comment_count>
    <who name="Srikumar B">srikumar.b</who>
    <bug_when>2011-09-02 10:33:25 -0700</bug_when>
    <thetext>As per the W3 working group standard http://www.whatwg.org/specs/web-apps/current-work/multipage/timers.html#timers, the default timeout for setTimeout() , setInterval() should be 4ms in DOMTimer but it is currently set to 10ms (s_minDefaultTimerInterval)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>461353</commentid>
    <comment_count>1</comment_count>
      <attachid>106163</attachid>
    <who name="Srikumar B">srikumar.b</who>
    <bug_when>2011-09-02 11:15:24 -0700</bug_when>
    <thetext>Created attachment 106163
Patch to change the default timeout value

DOMTimer default value changed from 10ms to 4ms as per the w3 WG http://www.whatwg.org/specs/web-apps/current-work/multipage/timers.html#timers

As the value is a minimum timeout, i think there is no straight forward way to test this case. I made sure all the existing DOMTimer tests are passing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>461738</commentid>
    <comment_count>2</comment_count>
      <attachid>106163</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2011-09-02 22:18:43 -0700</bug_when>
    <thetext>Comment on attachment 106163
Patch to change the default timeout value

It was intentionally left for ports to decide when/if they want to change the minimum interval. Grepping sources, it appears that not all ports chose to change it, so it may not appropriate to change the default at this time.

Also, duping to an existing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>461739</commentid>
    <comment_count>3</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2011-09-02 22:18:56 -0700</bug_when>
    <thetext>

*** This bug has been marked as a duplicate of bug 61214 ***</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>106163</attachid>
            <date>2011-09-02 11:15:24 -0700</date>
            <delta_ts>2011-09-02 22:18:43 -0700</delta_ts>
            <desc>Patch to change the default timeout value</desc>
            <filename>67501.patch</filename>
            <type>text/plain</type>
            <size>1321</size>
            <attacher name="Srikumar B">srikumar.b</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDk0NDMxKQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTggQEAKKzIwMTEtMDktMDIgIFNyaWt1bWFy
IEJvbmRhICA8c3Jpa3VtYXIuYkBnbWFpbC5jb20+CisKKyAgICAgICAgc2V0IGRlZmF1bHQgbWlu
aW11bSB0aW1lciBpbnRlcnZhbCB0byA0bXMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5v
cmcvc2hvd19idWcuY2dpP2lkPTY3NTAxCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChP
T1BTISkuCisKKyAgICAgICAgTm8gbmV3IHRlc3RzCisKKyAgICAgICAgQXMgcGVyIHRoZSB1cGRh
dGVkIFczIHN0YW5kYXJkLCBkZWZhdWx0IG1pbmltdW0gdGltZXIgaW52ZXJ2YWwgCisgICAgICAg
IHNob3VsZCBiZSA0bXMuCisKKworICAgICAgICAqIHBhZ2UvRE9NVGltZXIuY3BwOgorCiAyMDEx
LTA5LTAyICBBZHJpZW5uZSBXYWxrZXIgIDxlbm5lQGdvb2dsZS5jb20+CiAKICAgICAgICAgW2No
cm9taXVtXSBSZW1vdmUgTGF5ZXJSZW5kZXJlckNocm9taXVtIHJlZmVyZW5jZXMgZnJvbSB0aGUg
TGF5ZXJDaHJvbWl1bSB0cmVlCkluZGV4OiBTb3VyY2UvV2ViQ29yZS9wYWdlL0RPTVRpbWVyLmNw
cAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9wYWdlL0RPTVRpbWVyLmNwcAkocmV2aXNp
b24gOTQzNDkpCisrKyBTb3VyY2UvV2ViQ29yZS9wYWdlL0RPTVRpbWVyLmNwcAkod29ya2luZyBj
b3B5KQpAQCAtNDEsNyArNDEsNyBAQCBuYW1lc3BhY2UgV2ViQ29yZSB7CiBzdGF0aWMgY29uc3Qg
aW50IG1heEludGVydmFsRm9yVXNlckdlc3R1cmVGb3J3YXJkaW5nID0gMTAwMDsgLy8gT25lIHNl
Y29uZCBtYXRjaGVzIEdlY2tvLgogc3RhdGljIGNvbnN0IGludCBtYXhUaW1lck5lc3RpbmdMZXZl
bCA9IDU7CiBzdGF0aWMgY29uc3QgZG91YmxlIG9uZU1pbGxpc2Vjb25kID0gMC4wMDE7Ci1kb3Vi
bGUgRE9NVGltZXI6OnNfbWluRGVmYXVsdFRpbWVySW50ZXJ2YWwgPSAwLjAxMDsgLy8gMTAgbWls
bGlzZWNvbmRzCitkb3VibGUgRE9NVGltZXI6OnNfbWluRGVmYXVsdFRpbWVySW50ZXJ2YWwgPSAw
LjAwNDsgLy8gNCBtaWxsaXNlY29uZHMKIAogc3RhdGljIGludCB0aW1lck5lc3RpbmdMZXZlbCA9
IDA7CiAgICAgCg==
</data>
<flag name="review"
          id="102525"
          type_id="1"
          status="-"
          setter="ap"
    />
    <flag name="commit-queue"
          id="102526"
          type_id="3"
          status="-"
          setter="ap"
    />
          </attachment>
      

    </bug>

</bugzilla>