<?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>154559</bug_id>
          
          <creation_ts>2016-02-22 14:57:51 -0800</creation_ts>
          <short_desc>Some timer alignment cleanup.</short_desc>
          <delta_ts>2016-02-22 23:37:35 -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>WebKit Misc.</component>
          <version>Other</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>cdumez</cc>
    
    <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>1167208</commentid>
    <comment_count>0</comment_count>
    <who name="Gavin Barraclough">barraclough</who>
    <bug_when>2016-02-22 14:57:51 -0800</bug_when>
    <thetext>Document shouldn&apos;t override Page&apos;s timer alignment policy with a lower alignment, and m_domTimerAlignmentInterval isn&apos;t really a Settings, it&apos;s just the current state for the page.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1167233</commentid>
    <comment_count>1</comment_count>
      <attachid>271963</attachid>
    <who name="Gavin Barraclough">barraclough</who>
    <bug_when>2016-02-22 16:16:23 -0800</bug_when>
    <thetext>Created attachment 271963
Fix</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1167252</commentid>
    <comment_count>2</comment_count>
      <attachid>271963</attachid>
    <who name="Anders Carlsson">andersca</who>
    <bug_when>2016-02-22 17:01:43 -0800</bug_when>
    <thetext>Comment on attachment 271963
Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=271963&amp;action=review

You should change this to use std::chrono::milliseconds!

&gt; Source/WebCore/page/Page.h:620
&gt; -    bool m_timerThrottlingEnabled;
&gt; +    bool m_timerThrottlingEnabledTime;
&gt; +    double m_timerAlignmentInterval;

Can this use Optional instead?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1167275</commentid>
    <comment_count>3</comment_count>
    <who name="Gavin Barraclough">barraclough</who>
    <bug_when>2016-02-22 17:45:13 -0800</bug_when>
    <thetext>(In reply to comment #2)
&gt; Comment on attachment 271963 [details]
&gt; Fix
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=271963&amp;action=review
&gt; 
&gt; You should change this to use std::chrono::milliseconds!

Heh, probably! - sure I&apos;ll look into this. (But as a separate change - these intervals are all currently doubles &amp; I should probably switch double -&gt; chrono::milliseconds in a bunch of other places at the same time).

&gt; &gt; Source/WebCore/page/Page.h:620
&gt; &gt; -    bool m_timerThrottlingEnabled;
&gt; &gt; +    bool m_timerThrottlingEnabledTime;
&gt; &gt; +    double m_timerAlignmentInterval;
&gt; 
&gt; Can this use Optional instead?

Interesting. I don&apos;t think this should be an Optional - when timer throttling is disabled the timer alignment property does still hold a value (DOMTimer::defaultAlignmentInterval() - which theoretically could be non-zero).

Also I plan on replacing m_timerThrottlingEnabled with the timestamp at which time throttling is enabled, so we know how long it has been - come to think of it, that probably should be an Optional.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1167297</commentid>
    <comment_count>4</comment_count>
      <attachid>271963</attachid>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2016-02-22 18:48:39 -0800</bug_when>
    <thetext>Comment on attachment 271963
Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=271963&amp;action=review

r=me % nits

&gt; Source/WebCore/dom/Document.cpp:2934
&gt; +    if (Page* page = this-&gt;page())

auto*

&gt; Source/WebCore/page/Page.cpp:208
&gt; +    , m_timerThrottlingEnabledTime(false)

I prefer the previous name since this is a boolean.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1167322</commentid>
    <comment_count>5</comment_count>
    <who name="Gavin Barraclough">barraclough</who>
    <bug_when>2016-02-22 23:28:54 -0800</bug_when>
    <thetext>&gt; &gt; Source/WebCore/page/Page.cpp:208
&gt; &gt; +    , m_timerThrottlingEnabledTime(false)
&gt; 
&gt; I prefer the previous name since this is a boolean.

Oops, yes!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1167323</commentid>
    <comment_count>6</comment_count>
    <who name="Gavin Barraclough">barraclough</who>
    <bug_when>2016-02-22 23:37:35 -0800</bug_when>
    <thetext>Transmitting file data ......
Committed revision 196971.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>271963</attachid>
            <date>2016-02-22 16:16:23 -0800</date>
            <delta_ts>2016-02-22 18:48:39 -0800</delta_ts>
            <desc>Fix</desc>
            <filename>154559.1.patch</filename>
            <type>text/plain</type>
            <size>7330</size>
            <attacher name="Gavin Barraclough">barraclough</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE5Njk2MSkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDMyIEBACisyMDE2LTAyLTIyICBHYXZpbiBC
YXJyYWNsb3VnaCAgPGJhcnJhY2xvdWdoQGFwcGxlLmNvbT4KKworICAgICAgICBTb21lIHRpbWVy
IGFsaWdubWVudCBjbGVhbnVwLgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93
X2J1Zy5jZ2k/aWQ9MTU0NTU5CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgRG9jdW1lbnQgc2hvdWxkbid0IG92ZXJyaWRlIFBhZ2UncyB0aW1lciBhbGln
bm1lbnQgcG9saWN5IHdpdGggYSBsb3dlciBhbGlnbm1lbnQsCisgICAgICAgIGFuZCBtX2RvbVRp
bWVyQWxpZ25tZW50SW50ZXJ2YWwgaXNuJ3QgcmVhbGx5IGEgU2V0dGluZ3MsIGl0J3MganVzdCB0
aGUgY3VycmVudCBzdGF0ZQorICAgICAgICBmb3IgdGhlIHBhZ2UuCisKKyAgICAgICAgKiBkb20v
RG9jdW1lbnQuY3BwOgorICAgICAgICAoV2ViQ29yZTo6RG9jdW1lbnQ6OnRpbWVyQWxpZ25tZW50
SW50ZXJ2YWwpOgorICAgICAgICAgICAgLSB0YWtlIHRoZSBtYXggb2YgdGhlIHBvc3NpYmxlIGlu
dGVydmFscy4KKyAgICAgICAgKiBwYWdlL1BhZ2UuY3BwOgorICAgICAgICAoV2ViQ29yZTo6UGFn
ZTo6UGFnZSk6CisgICAgICAgIChXZWJDb3JlOjpQYWdlOjpzZXRUaW1lclRocm90dGxpbmdFbmFi
bGVkKToKKyAgICAgICAgKFdlYkNvcmU6OlBhZ2U6OnNldERPTVRpbWVyQWxpZ25tZW50SW50ZXJ2
YWwpOgorICAgICAgICAgICAgLSBtb3ZlZCBoZXJlIGZyb20gU2V0dGluZ3MuCisgICAgICAgICog
cGFnZS9QYWdlLmg6CisgICAgICAgIChXZWJDb3JlOjpQYWdlOjpkb21UaW1lckFsaWdubWVudElu
dGVydmFsKToKKyAgICAgICAgICAgIC0gbW92ZWQgaGVyZSBmcm9tIFNldHRpbmdzLgorICAgICAg
ICAqIHBhZ2UvU2V0dGluZ3MuY3BwOgorICAgICAgICAoV2ViQ29yZTo6U2V0dGluZ3M6OnNldERP
TVRpbWVyQWxpZ25tZW50SW50ZXJ2YWwpOiBEZWxldGVkLgorICAgICAgICAgICAgLSBtb3ZlZCB0
byBQYWdlLgorICAgICAgICAqIHBhZ2UvU2V0dGluZ3MuaDoKKyAgICAgICAgKFdlYkNvcmU6OlNl
dHRpbmdzOjpkb21UaW1lckFsaWdubWVudEludGVydmFsKTogRGVsZXRlZC4KKyAgICAgICAgICAg
IC0gbW92ZWQgdG8gUGFnZS4KKwogMjAxNi0wMi0yMiAgQ2hyaXMgRHVtZXogIDxjZHVtZXpAYXBw
bGUuY29tPgogCiAgICAgICAgIFJFR1JFU1NJT04gKHIxOTY1NjMpOiBJbWFnZXMgbm90IGxvYWRp
bmcgb24gaHR0cHM6Ly9rbGltLmNvLm56L2Jsb2cvcGF5cGFsLXNhbnMtZGVzaWduLWluZm9ybWF0
aW9uLwpJbmRleDogU291cmNlL1dlYkNvcmUvZG9tL0RvY3VtZW50LmNwcAo9PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0t
LSBTb3VyY2UvV2ViQ29yZS9kb20vRG9jdW1lbnQuY3BwCShyZXZpc2lvbiAxOTY4NjkpCisrKyBT
b3VyY2UvV2ViQ29yZS9kb20vRG9jdW1lbnQuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0yOTI1LDE0
ICsyOTI1LDE2IEBAIHZvaWQgRG9jdW1lbnQ6OnNldFRpbWVyVGhyb3R0bGluZ0VuYWJsZWQKIAog
ZG91YmxlIERvY3VtZW50Ojp0aW1lckFsaWdubWVudEludGVydmFsKGJvb2wgaGFzUmVhY2hlZE1h
eE5lc3RpbmdMZXZlbCkgY29uc3QKIHsKKyAgICBkb3VibGUgYWxpZ25tZW50SW50ZXJ2YWwgPSBT
Y3JpcHRFeGVjdXRpb25Db250ZXh0Ojp0aW1lckFsaWdubWVudEludGVydmFsKGhhc1JlYWNoZWRN
YXhOZXN0aW5nTGV2ZWwpOworCiAgICAgLy8gQXBwbHkgRG9jdW1lbnQtbGV2ZWwgRE9NVGltZXIg
dGhyb3R0bGluZyBvbmx5IGlmIHRpbWVycyBoYXZlIHJlYWNoZWQgdGhlaXIgbWF4aW11bSBuZXN0
aW5nIGxldmVsIGFzIHRoZSBQYWdlIG1heSBzdGlsbCBiZSB2aXNpYmxlLgogICAgIGlmIChtX2lz
VGltZXJUaHJvdHRsaW5nRW5hYmxlZCAmJiBoYXNSZWFjaGVkTWF4TmVzdGluZ0xldmVsKQotICAg
ICAgICByZXR1cm4gRE9NVGltZXI6OmhpZGRlblBhZ2VBbGlnbm1lbnRJbnRlcnZhbCgpOworICAg
ICAgICBhbGlnbm1lbnRJbnRlcnZhbCA9IHN0ZDo6bWF4KGFsaWdubWVudEludGVydmFsLCBET01U
aW1lcjo6aGlkZGVuUGFnZUFsaWdubWVudEludGVydmFsKCkpOwogCi0gICAgUGFnZSogcGFnZSA9
IHRoaXMtPnBhZ2UoKTsKLSAgICBpZiAoIXBhZ2UpCi0gICAgICAgIHJldHVybiBTY3JpcHRFeGVj
dXRpb25Db250ZXh0Ojp0aW1lckFsaWdubWVudEludGVydmFsKGhhc1JlYWNoZWRNYXhOZXN0aW5n
TGV2ZWwpOwotICAgIHJldHVybiBwYWdlLT5zZXR0aW5ncygpLmRvbVRpbWVyQWxpZ25tZW50SW50
ZXJ2YWwoKTsKKyAgICBpZiAoUGFnZSogcGFnZSA9IHRoaXMtPnBhZ2UoKSkKKyAgICAgICAgYWxp
Z25tZW50SW50ZXJ2YWwgPSBzdGQ6Om1heChhbGlnbm1lbnRJbnRlcnZhbCwgcGFnZS0+ZG9tVGlt
ZXJBbGlnbm1lbnRJbnRlcnZhbCgpKTsKKworICAgIHJldHVybiBhbGlnbm1lbnRJbnRlcnZhbDsK
IH0KIAogRXZlbnRUYXJnZXQqIERvY3VtZW50OjplcnJvckV2ZW50VGFyZ2V0KCkKSW5kZXg6IFNv
dXJjZS9XZWJDb3JlL3BhZ2UvUGFnZS5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUv
cGFnZS9QYWdlLmNwcAkocmV2aXNpb24gMTk2ODY5KQorKysgU291cmNlL1dlYkNvcmUvcGFnZS9Q
YWdlLmNwcAkod29ya2luZyBjb3B5KQpAQCAtMjA1LDcgKzIwNSw4IEBAIFBhZ2U6OlBhZ2UoUGFn
ZUNvbmZpZ3VyYXRpb24mIHBhZ2VDb25maWcKICNpZiBFTkFCTEUoVklFV19NT0RFX0NTU19NRURJ
QSkKICAgICAsIG1fdmlld01vZGUoVmlld01vZGVXaW5kb3dlZCkKICNlbmRpZiAvLyBFTkFCTEUo
VklFV19NT0RFX0NTU19NRURJQSkKLSAgICAsIG1fdGltZXJUaHJvdHRsaW5nRW5hYmxlZChmYWxz
ZSkKKyAgICAsIG1fdGltZXJUaHJvdHRsaW5nRW5hYmxlZFRpbWUoZmFsc2UpCisgICAgLCBtX3Rp
bWVyQWxpZ25tZW50SW50ZXJ2YWwoRE9NVGltZXI6OmRlZmF1bHRBbGlnbm1lbnRJbnRlcnZhbCgp
KQogICAgICwgbV9pc0VkaXRhYmxlKGZhbHNlKQogICAgICwgbV9pc1ByZXJlbmRlcihmYWxzZSkK
ICAgICAsIG1fdmlld1N0YXRlKFBhZ2VJbml0aWFsVmlld1N0YXRlKQpAQCAtMTE2OCwxMSArMTE2
OSwxNiBAQCB2b2lkIFBhZ2U6OnNldFRpbWVyVGhyb3R0bGluZ0VuYWJsZWQoYm9vCiAgICAgICAg
IGVuYWJsZWQgPSBmYWxzZTsKICNlbmRpZgogCi0gICAgaWYgKGVuYWJsZWQgPT0gbV90aW1lclRo
cm90dGxpbmdFbmFibGVkKQorICAgIGlmIChlbmFibGVkID09IG1fdGltZXJUaHJvdHRsaW5nRW5h
YmxlZFRpbWUpCiAgICAgICAgIHJldHVybjsKIAotICAgIG1fdGltZXJUaHJvdHRsaW5nRW5hYmxl
ZCA9IGVuYWJsZWQ7Ci0gICAgbV9zZXR0aW5ncy0+c2V0RE9NVGltZXJBbGlnbm1lbnRJbnRlcnZh
bChlbmFibGVkID8gRE9NVGltZXI6OmhpZGRlblBhZ2VBbGlnbm1lbnRJbnRlcnZhbCgpIDogRE9N
VGltZXI6OmRlZmF1bHRBbGlnbm1lbnRJbnRlcnZhbCgpKTsKKyAgICBtX3RpbWVyVGhyb3R0bGlu
Z0VuYWJsZWRUaW1lID0gZW5hYmxlZDsKKyAgICBzZXRET01UaW1lckFsaWdubWVudEludGVydmFs
KGVuYWJsZWQgPyBET01UaW1lcjo6aGlkZGVuUGFnZUFsaWdubWVudEludGVydmFsKCkgOiBET01U
aW1lcjo6ZGVmYXVsdEFsaWdubWVudEludGVydmFsKCkpOworfQorCit2b2lkIFBhZ2U6OnNldERP
TVRpbWVyQWxpZ25tZW50SW50ZXJ2YWwoZG91YmxlIGFsaWdubWVudEludGVydmFsKQoreworICAg
IG1fdGltZXJBbGlnbm1lbnRJbnRlcnZhbCA9IGFsaWdubWVudEludGVydmFsOwogICAgIAogICAg
IGZvciAoRnJhbWUqIGZyYW1lID0gJm1haW5GcmFtZSgpOyBmcmFtZTsgZnJhbWUgPSBmcmFtZS0+
dHJlZSgpLnRyYXZlcnNlTmV4dCgpKSB7CiAgICAgICAgIGlmIChmcmFtZS0+ZG9jdW1lbnQoKSkK
SW5kZXg6IFNvdXJjZS9XZWJDb3JlL3BhZ2UvUGFnZS5oCj09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9X
ZWJDb3JlL3BhZ2UvUGFnZS5oCShyZXZpc2lvbiAxOTY4NjkpCisrKyBTb3VyY2UvV2ViQ29yZS9w
YWdlL1BhZ2UuaAkod29ya2luZyBjb3B5KQpAQCAtMjE5LDYgKzIxOSw4IEBAIHB1YmxpYzoKICAg
ICBQcm9ncmVzc1RyYWNrZXImIHByb2dyZXNzKCkgY29uc3QgeyByZXR1cm4gKm1fcHJvZ3Jlc3M7
IH0KICAgICBCYWNrRm9yd2FyZENvbnRyb2xsZXImIGJhY2tGb3J3YXJkKCkgY29uc3QgeyByZXR1
cm4gKm1fYmFja0ZvcndhcmRDb250cm9sbGVyOyB9CiAKKyAgICBkb3VibGUgZG9tVGltZXJBbGln
bm1lbnRJbnRlcnZhbCgpIGNvbnN0IHsgcmV0dXJuIG1fdGltZXJBbGlnbm1lbnRJbnRlcnZhbDsg
fQorCiAjaWYgRU5BQkxFKFZJRVdfTU9ERV9DU1NfTUVESUEpCiAgICAgZW51bSBWaWV3TW9kZSB7
CiAgICAgICAgIFZpZXdNb2RlSW52YWxpZCwKQEAgLTUyNCw2ICs1MjYsOCBAQCBwcml2YXRlOgog
CiAgICAgdm9pZCBoaWRkZW5QYWdlRE9NVGltZXJUaHJvdHRsaW5nU3RhdGVDaGFuZ2VkKCk7CiAg
ICAgdm9pZCBzZXRUaW1lclRocm90dGxpbmdFbmFibGVkKGJvb2wpOworICAgIHZvaWQgc2V0RE9N
VGltZXJBbGlnbm1lbnRJbnRlcnZhbChkb3VibGUpOworICAgIHZvaWQgdGltZXJBbGlnbm1lbnRJ
bnRlcnZhbFRpbWVyRmlyZWQoKTsKICAgICBib29sIGNhblRhYlN1c3BlbmQoKTsKICAgICB2b2lk
IHVwZGF0ZVRhYlN1c3BlbnNpb25TdGF0ZSgpOwogICAgIHZvaWQgdGFiU3VzcGVuc2lvblRpbWVy
RmlyZWQoKTsKQEAgLTYxMiw3ICs2MTYsOCBAQCBwcml2YXRlOgogICAgIFZpZXdNb2RlIG1fdmll
d01vZGU7CiAjZW5kaWYgLy8gRU5BQkxFKFZJRVdfTU9ERV9DU1NfTUVESUEpCiAKLSAgICBib29s
IG1fdGltZXJUaHJvdHRsaW5nRW5hYmxlZDsKKyAgICBib29sIG1fdGltZXJUaHJvdHRsaW5nRW5h
YmxlZFRpbWU7CisgICAgZG91YmxlIG1fdGltZXJBbGlnbm1lbnRJbnRlcnZhbDsKIAogICAgIGJv
b2wgbV9pc0VkaXRhYmxlOwogICAgIGJvb2wgbV9pc1ByZXJlbmRlcjsKSW5kZXg6IFNvdXJjZS9X
ZWJDb3JlL3BhZ2UvU2V0dGluZ3MuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL3Bh
Z2UvU2V0dGluZ3MuY3BwCShyZXZpc2lvbiAxOTY4NjkpCisrKyBTb3VyY2UvV2ViQ29yZS9wYWdl
L1NldHRpbmdzLmNwcAkod29ya2luZyBjb3B5KQpAQCAtMTgwLDcgKzE4MCw2IEBAIFNldHRpbmdz
OjpTZXR0aW5ncyhQYWdlKiBwYWdlKQogICAgICwgbV9zdG9yYWdlQmxvY2tpbmdQb2xpY3koU2Vj
dXJpdHlPcmlnaW46OkFsbG93QWxsU3RvcmFnZSkKICAgICAsIG1fbGF5b3V0SW50ZXJ2YWwobGF5
b3V0U2NoZWR1bGVUaHJlc2hvbGQpCiAgICAgLCBtX21pbmltdW1ET01UaW1lckludGVydmFsKERP
TVRpbWVyOjpkZWZhdWx0TWluaW11bUludGVydmFsKCkpCi0gICAgLCBtX2RvbVRpbWVyQWxpZ25t
ZW50SW50ZXJ2YWwoRE9NVGltZXI6OmRlZmF1bHRBbGlnbm1lbnRJbnRlcnZhbCgpKQogI2lmIEVO
QUJMRShURVhUX0FVVE9TSVpJTkcpCiAgICAgLCBtX3RleHRBdXRvc2l6aW5nRm9udFNjYWxlRmFj
dG9yKDEpCiAjaWYgSEFDS19GT1JDRV9URVhUX0FVVE9TSVpJTkdfT05fREVTS1RPUApAQCAtNDg1
LDExICs0ODQsNiBAQCB2b2lkIFNldHRpbmdzOjpzZXRNaW5pbXVtRE9NVGltZXJJbnRlcnZhCiAg
ICAgfQogfQogCi12b2lkIFNldHRpbmdzOjpzZXRET01UaW1lckFsaWdubWVudEludGVydmFsKGRv
dWJsZSBhbGlnbm1lbnRJbnRlcnZhbCkKLXsKLSAgICBtX2RvbVRpbWVyQWxpZ25tZW50SW50ZXJ2
YWwgPSBhbGlnbm1lbnRJbnRlcnZhbDsKLX0KLQogdm9pZCBTZXR0aW5nczo6c2V0TGF5b3V0SW50
ZXJ2YWwoc3RkOjpjaHJvbm86Om1pbGxpc2Vjb25kcyBsYXlvdXRJbnRlcnZhbCkKIHsKICAgICAv
LyBGSVhNRTogSXQgc2VlbXMgd2VpcmQgdGhhdCB0aGlzIGZ1bmN0aW9uIG1heSBkaXNyZWdhcmQg
dGhlIHNwZWNpZmllZCBsYXlvdXQgaW50ZXJ2YWwuCkluZGV4OiBTb3VyY2UvV2ViQ29yZS9wYWdl
L1NldHRpbmdzLmgKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvcGFnZS9TZXR0aW5ncy5o
CShyZXZpc2lvbiAxOTY4NjkpCisrKyBTb3VyY2UvV2ViQ29yZS9wYWdlL1NldHRpbmdzLmgJKHdv
cmtpbmcgY29weSkKQEAgLTE1OCw5ICsxNTgsNiBAQCBwdWJsaWM6CiAgICAgV0VCQ09SRV9FWFBP
UlQgdm9pZCBzZXRNaW5pbXVtRE9NVGltZXJJbnRlcnZhbChkb3VibGUpOyAvLyBJbml0aWFsaXpl
ZCB0byBET01UaW1lcjo6ZGVmYXVsdE1pbmltdW1JbnRlcnZhbCgpLgogICAgIGRvdWJsZSBtaW5p
bXVtRE9NVGltZXJJbnRlcnZhbCgpIGNvbnN0IHsgcmV0dXJuIG1fbWluaW11bURPTVRpbWVySW50
ZXJ2YWw7IH0KIAotICAgIHZvaWQgc2V0RE9NVGltZXJBbGlnbm1lbnRJbnRlcnZhbChkb3VibGUp
OwotICAgIGRvdWJsZSBkb21UaW1lckFsaWdubWVudEludGVydmFsKCkgY29uc3QgeyByZXR1cm4g
bV9kb21UaW1lckFsaWdubWVudEludGVydmFsOyB9Ci0KICAgICBXRUJDT1JFX0VYUE9SVCB2b2lk
IHNldExheW91dEludGVydmFsKHN0ZDo6Y2hyb25vOjptaWxsaXNlY29uZHMpOwogICAgIHN0ZDo6
Y2hyb25vOjptaWxsaXNlY29uZHMgbGF5b3V0SW50ZXJ2YWwoKSBjb25zdCB7IHJldHVybiBtX2xh
eW91dEludGVydmFsOyB9CiAKQEAgLTMwMyw3ICszMDAsNiBAQCBwcml2YXRlOgogICAgIFNlY3Vy
aXR5T3JpZ2luOjpTdG9yYWdlQmxvY2tpbmdQb2xpY3kgbV9zdG9yYWdlQmxvY2tpbmdQb2xpY3k7
CiAgICAgc3RkOjpjaHJvbm86Om1pbGxpc2Vjb25kcyBtX2xheW91dEludGVydmFsOwogICAgIGRv
dWJsZSBtX21pbmltdW1ET01UaW1lckludGVydmFsOwotICAgIGRvdWJsZSBtX2RvbVRpbWVyQWxp
Z25tZW50SW50ZXJ2YWw7CiAKICNpZiBFTkFCTEUoVEVYVF9BVVRPU0laSU5HKQogICAgIGZsb2F0
IG1fdGV4dEF1dG9zaXppbmdGb250U2NhbGVGYWN0b3I7Cg==
</data>
<flag name="review"
          id="296788"
          type_id="1"
          status="+"
          setter="cdumez"
    />
          </attachment>
      

    </bug>

</bugzilla>