<?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>150836</bug_id>
          
          <creation_ts>2015-11-03 04:19:05 -0800</creation_ts>
          <short_desc>[GStreamer] Use RunLoop::Timer for ready state timer in MediaPlayerPrivateGStreamer</short_desc>
          <delta_ts>2015-11-04 02:14:28 -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>Platform</component>
          <version>WebKit Local 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>Gtk</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Carlos Garcia Campos">cgarcia</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>pnormand</cc>
    
    <cc>zan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1138910</commentid>
    <comment_count>0</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2015-11-03 04:19:05 -0800</bug_when>
    <thetext>We don&apos;t really need a GThreadSafeMainLoopSource for this simple timer.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1138911</commentid>
    <comment_count>1</comment_count>
      <attachid>264690</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2015-11-03 04:20:35 -0800</bug_when>
    <thetext>Created attachment 264690
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1138912</commentid>
    <comment_count>2</comment_count>
      <attachid>264690</attachid>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2015-11-03 04:35:10 -0800</bug_when>
    <thetext>Comment on attachment 264690
Patch

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

&gt; Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:208
&gt; +#if PLATFORM(GTK)

Why not USE(GLIB) ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1138914</commentid>
    <comment_count>3</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2015-11-03 05:12:55 -0800</bug_when>
    <thetext>(In reply to comment #2)
&gt; Comment on attachment 264690 [details]
&gt; Patch
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=264690&amp;action=review
&gt; 
&gt; &gt; Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:208
&gt; &gt; +#if PLATFORM(GTK)
&gt; 
&gt; Why not USE(GLIB) ?

Because EFL doesn&apos;t support it</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1138917</commentid>
    <comment_count>4</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2015-11-03 06:04:33 -0800</bug_when>
    <thetext>(In reply to comment #3)
&gt; (In reply to comment #2)
&gt; &gt; Comment on attachment 264690 [details]
&gt; &gt; Patch
&gt; &gt; 
&gt; &gt; View in context:
&gt; &gt; https://bugs.webkit.org/attachment.cgi?id=264690&amp;action=review
&gt; &gt; 
&gt; &gt; &gt; Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:208
&gt; &gt; &gt; +#if PLATFORM(GTK)
&gt; &gt; 
&gt; &gt; Why not USE(GLIB) ?
&gt; 
&gt; Because EFL doesn&apos;t support it

But aren&apos;t they using Glib? If not it&apos;s quite silly because quite a few deps like GStreamer and libsoup have an implicit dependency on it already.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1139253</commentid>
    <comment_count>5</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2015-11-03 22:28:10 -0800</bug_when>
    <thetext>(In reply to comment #4)
&gt; (In reply to comment #3)
&gt; &gt; (In reply to comment #2)
&gt; &gt; &gt; Comment on attachment 264690 [details]
&gt; &gt; &gt; Patch
&gt; &gt; &gt; 
&gt; &gt; &gt; View in context:
&gt; &gt; &gt; https://bugs.webkit.org/attachment.cgi?id=264690&amp;action=review
&gt; &gt; &gt; 
&gt; &gt; &gt; &gt; Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:208
&gt; &gt; &gt; &gt; +#if PLATFORM(GTK)
&gt; &gt; &gt; 
&gt; &gt; &gt; Why not USE(GLIB) ?
&gt; &gt; 
&gt; &gt; Because EFL doesn&apos;t support it
&gt; 
&gt; But aren&apos;t they using Glib? If not it&apos;s quite silly because quite a few deps
&gt; like GStreamer and libsoup have an implicit dependency on it already.

They don&apos;t use glib sources in the RunLoop implementation and win cairo either. See how setPriority is defined in RunLoop.h

#if USE(GLIB) &amp;&amp; !PLATFORM(EFL)
        void setPriority(int);
#endif

EFL has its own implementation of RunLoop and RunLoop::Timer using ecore.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1139282</commentid>
    <comment_count>6</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2015-11-04 00:12:42 -0800</bug_when>
    <thetext>(In reply to comment #5)
&gt; (In reply to comment #4)
&gt; &gt; (In reply to comment #3)
&gt; &gt; &gt; (In reply to comment #2)
&gt; &gt; &gt; &gt; Comment on attachment 264690 [details]
&gt; &gt; &gt; &gt; Patch
&gt; &gt; &gt; &gt; 
&gt; &gt; &gt; &gt; View in context:
&gt; &gt; &gt; &gt; https://bugs.webkit.org/attachment.cgi?id=264690&amp;action=review
&gt; &gt; &gt; &gt; 
&gt; &gt; &gt; &gt; &gt; Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:208
&gt; &gt; &gt; &gt; &gt; +#if PLATFORM(GTK)
&gt; &gt; &gt; &gt; 
&gt; &gt; &gt; &gt; Why not USE(GLIB) ?
&gt; &gt; &gt; 
&gt; &gt; &gt; Because EFL doesn&apos;t support it
&gt; &gt; 
&gt; &gt; But aren&apos;t they using Glib? If not it&apos;s quite silly because quite a few deps
&gt; &gt; like GStreamer and libsoup have an implicit dependency on it already.
&gt; 
&gt; They don&apos;t use glib sources in the RunLoop implementation and win cairo
&gt; either. See how setPriority is defined in RunLoop.h
&gt; 
&gt; #if USE(GLIB) &amp;&amp; !PLATFORM(EFL)
&gt;         void setPriority(int);
&gt; #endif
&gt; 
&gt; EFL has its own implementation of RunLoop and RunLoop::Timer using ecore.

Alright :) So let&apos;s use the same ifdef as above for consistency then?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1139283</commentid>
    <comment_count>7</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2015-11-04 00:15:26 -0800</bug_when>
    <thetext>(In reply to comment #6)
&gt; (In reply to comment #5)
&gt; &gt; (In reply to comment #4)
&gt; &gt; &gt; (In reply to comment #3)
&gt; &gt; &gt; &gt; (In reply to comment #2)
&gt; &gt; &gt; &gt; &gt; Comment on attachment 264690 [details]
&gt; &gt; &gt; &gt; &gt; Patch
&gt; &gt; &gt; &gt; &gt; 
&gt; &gt; &gt; &gt; &gt; View in context:
&gt; &gt; &gt; &gt; &gt; https://bugs.webkit.org/attachment.cgi?id=264690&amp;action=review
&gt; &gt; &gt; &gt; &gt; 
&gt; &gt; &gt; &gt; &gt; &gt; Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:208
&gt; &gt; &gt; &gt; &gt; &gt; +#if PLATFORM(GTK)
&gt; &gt; &gt; &gt; &gt; 
&gt; &gt; &gt; &gt; &gt; Why not USE(GLIB) ?
&gt; &gt; &gt; &gt; 
&gt; &gt; &gt; &gt; Because EFL doesn&apos;t support it
&gt; &gt; &gt; 
&gt; &gt; &gt; But aren&apos;t they using Glib? If not it&apos;s quite silly because quite a few deps
&gt; &gt; &gt; like GStreamer and libsoup have an implicit dependency on it already.
&gt; &gt; 
&gt; &gt; They don&apos;t use glib sources in the RunLoop implementation and win cairo
&gt; &gt; either. See how setPriority is defined in RunLoop.h
&gt; &gt; 
&gt; &gt; #if USE(GLIB) &amp;&amp; !PLATFORM(EFL)
&gt; &gt;         void setPriority(int);
&gt; &gt; #endif
&gt; &gt; 
&gt; &gt; EFL has its own implementation of RunLoop and RunLoop::Timer using ecore.
&gt; 
&gt; Alright :) So let&apos;s use the same ifdef as above for consistency then?

And do you want me to submit an updated patch just for this?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1139285</commentid>
    <comment_count>8</comment_count>
      <attachid>264690</attachid>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2015-11-04 00:24:38 -0800</bug_when>
    <thetext>Comment on attachment 264690
Patch

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

&gt;&gt;&gt;&gt;&gt;&gt;&gt; Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:208
&gt;&gt;&gt;&gt;&gt;&gt;&gt; +#if PLATFORM(GTK)
&gt;&gt;&gt;&gt;&gt;&gt; 
&gt;&gt;&gt;&gt;&gt;&gt; Why not USE(GLIB) ?
&gt;&gt;&gt;&gt;&gt; 
&gt;&gt;&gt;&gt;&gt; Because EFL doesn&apos;t support it
&gt;&gt;&gt;&gt; 
&gt;&gt;&gt;&gt; But aren&apos;t they using Glib? If not it&apos;s quite silly because quite a few deps like GStreamer and libsoup have an implicit dependency on it already.
&gt;&gt;&gt; 
&gt;&gt;&gt; They don&apos;t use glib sources in the RunLoop implementation and win cairo either. See how setPriority is defined in RunLoop.h
&gt;&gt;&gt; 
&gt;&gt;&gt; #if USE(GLIB) &amp;&amp; !PLATFORM(EFL)
&gt;&gt;&gt;         void setPriority(int);
&gt;&gt;&gt; #endif
&gt;&gt;&gt; 
&gt;&gt;&gt; EFL has its own implementation of RunLoop and RunLoop::Timer using ecore.
&gt;&gt; 
&gt;&gt; Alright :) So let&apos;s use the same ifdef as above for consistency then?
&gt; 
&gt; And do you want me to submit an updated patch just for this?

No.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1139300</commentid>
    <comment_count>9</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2015-11-04 02:14:28 -0800</bug_when>
    <thetext>Committed r192019: &lt;http://trac.webkit.org/changeset/192019&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>264690</attachid>
            <date>2015-11-03 04:20:35 -0800</date>
            <delta_ts>2015-11-04 00:24:38 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>wcore-gst-ready-timer.diff</filename>
            <type>text/plain</type>
            <size>6134</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCBhMDE1NGQ2Li5mMWU3MTFmIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMjIg
QEAKIDIwMTUtMTEtMDMgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t
PgogCisgICAgICAgIFtHU3RyZWFtZXJdIFVzZSBSdW5Mb29wOjpUaW1lciBmb3IgcmVhZHkgc3Rh
dGUgdGltZXIgaW4gTWVkaWFQbGF5ZXJQcml2YXRlR1N0cmVhbWVyCisgICAgICAgIGh0dHBzOi8v
YnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNTA4MzYKKworICAgICAgICBSZXZpZXdl
ZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBXZSBkb24ndCByZWFsbHkgbmVlZCBhIEdU
aHJlYWRTYWZlTWFpbkxvb3BTb3VyY2UgZm9yIHRoaXMgc2ltcGxlIHRpbWVyLgorCisgICAgICAg
ICogcGxhdGZvcm0vZ3JhcGhpY3MvZ3N0cmVhbWVyL01lZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1l
ci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpNZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXI6Ok1l
ZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1lcik6CisgICAgICAgIChXZWJDb3JlOjpNZWRpYVBsYXll
clByaXZhdGVHU3RyZWFtZXI6On5NZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXIpOgorICAgICAg
ICAoV2ViQ29yZTo6TWVkaWFQbGF5ZXJQcml2YXRlR1N0cmVhbWVyOjpyZWFkeVRpbWVyRmlyZWQp
OgorICAgICAgICAoV2ViQ29yZTo6TWVkaWFQbGF5ZXJQcml2YXRlR1N0cmVhbWVyOjpjaGFuZ2VQ
aXBlbGluZVN0YXRlKToKKyAgICAgICAgKFdlYkNvcmU6Ok1lZGlhUGxheWVyUHJpdmF0ZUdTdHJl
YW1lcjo6bG9hZGluZ0ZhaWxlZCk6CisgICAgICAgICogcGxhdGZvcm0vZ3JhcGhpY3MvZ3N0cmVh
bWVyL01lZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1lci5oOgorCisyMDE1LTExLTAzICBDYXJsb3Mg
R2FyY2lhIENhbXBvcyAgPGNnYXJjaWFAaWdhbGlhLmNvbT4KKwogICAgICAgICBbR1N0cmVhbWVy
XSBVc2UgUnVuTG9vcDo6VGltZXIgaW5zdGVhZCBvZiBHTWFpbkxvb3BTb3VyY2UgaW4gdmlkZW8g
c2luawogICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTUw
ODA3CiAKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2dzdHJl
YW1lci9NZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXIuY3BwIGIvU291cmNlL1dlYkNvcmUvcGxh
dGZvcm0vZ3JhcGhpY3MvZ3N0cmVhbWVyL01lZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1lci5jcHAK
aW5kZXggMGE1NjcwYy4uNmUwN2ZjNiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcGxhdGZv
cm0vZ3JhcGhpY3MvZ3N0cmVhbWVyL01lZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1lci5jcHAKKysr
IGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvZ3N0cmVhbWVyL01lZGlhUGxheWVy
UHJpdmF0ZUdTdHJlYW1lci5jcHAKQEAgLTQxLDcgKzQxLDYgQEAKICNpbmNsdWRlIDxsaW1pdHM+
CiAjaW5jbHVkZSA8d3RmL0hleE51bWJlci5oPgogI2luY2x1ZGUgPHd0Zi9NZWRpYVRpbWUuaD4K
LSNpbmNsdWRlIDx3dGYvUnVuTG9vcC5oPgogI2luY2x1ZGUgPHd0Zi9nbGliL0dVbmlxdWVQdHIu
aD4KICNpbmNsdWRlIDx3dGYvdGV4dC9DU3RyaW5nLmg+CiAKQEAgLTcwLDEwICs2OSw2IEBACiAj
aW5jbHVkZSAiQXVkaW9Tb3VyY2VQcm92aWRlckdTdHJlYW1lci5oIgogI2VuZGlmCiAKLS8vIE1h
eCBpbnRlcnZhbCBpbiBzZWNvbmRzIHRvIHN0YXkgaW4gdGhlIFJFQURZIHN0YXRlIG9uIG1hbnVh
bAotLy8gc3RhdGUgY2hhbmdlIHJlcXVlc3RzLgotc3RhdGljIGNvbnN0IHVuc2lnbmVkIGdSZWFk
eVN0YXRlVGltZXJJbnRlcnZhbCA9IDYwOwotCiBHU1RfREVCVUdfQ0FURUdPUllfRVhURVJOKHdl
YmtpdF9tZWRpYV9wbGF5ZXJfZGVidWcpOwogI2RlZmluZSBHU1RfQ0FUX0RFRkFVTFQgd2Via2l0
X21lZGlhX3BsYXllcl9kZWJ1ZwogCkBAIC0yMDIsNiArMTk3LDcgQEAgTWVkaWFQbGF5ZXJQcml2
YXRlR1N0cmVhbWVyOjpNZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXIoTWVkaWFQbGF5ZXIqIHBs
YXllcikKICAgICAsIG1fdm9sdW1lQW5kTXV0ZUluaXRpYWxpemVkKGZhbHNlKQogICAgICwgbV9o
YXNWaWRlbyhmYWxzZSkKICAgICAsIG1faGFzQXVkaW8oZmFsc2UpCisgICAgLCBtX3JlYWR5VGlt
ZXJIYW5kbGVyKFJ1bkxvb3A6Om1haW4oKSwgdGhpcywgJk1lZGlhUGxheWVyUHJpdmF0ZUdTdHJl
YW1lcjo6cmVhZHlUaW1lckZpcmVkKQogICAgICwgbV90b3RhbEJ5dGVzKDApCiAgICAgLCBtX3By
ZXNlcnZlc1BpdGNoKGZhbHNlKQogI2lmIEVOQUJMRShXRUJfQVVESU8pCkBAIC0yMDksNiArMjA1
LDkgQEAgTWVkaWFQbGF5ZXJQcml2YXRlR1N0cmVhbWVyOjpNZWRpYVBsYXllclByaXZhdGVHU3Ry
ZWFtZXIoTWVkaWFQbGF5ZXIqIHBsYXllcikKICNlbmRpZgogICAgICwgbV9yZXF1ZXN0ZWRTdGF0
ZShHU1RfU1RBVEVfVk9JRF9QRU5ESU5HKQogeworI2lmIFBMQVRGT1JNKEdUSykKKyAgICBtX3Jl
YWR5VGltZXJIYW5kbGVyLnNldFByaW9yaXR5KEdfUFJJT1JJVFlfREVGQVVMVF9JRExFKTsKKyNl
bmRpZgogfQogCiBNZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXI6On5NZWRpYVBsYXllclByaXZh
dGVHU3RyZWFtZXIoKQpAQCAtMjM1LDcgKzIzNCw3IEBAIE1lZGlhUGxheWVyUHJpdmF0ZUdTdHJl
YW1lcjo6fk1lZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1lcigpCiAgICAgICAgIGdfc2lnbmFsX2hh
bmRsZXJzX2Rpc2Nvbm5lY3RfYnlfZnVuYyhHX09CSkVDVChtX2F1dG9BdWRpb1NpbmsuZ2V0KCkp
LAogICAgICAgICAgICAgcmVpbnRlcnByZXRfY2FzdDxncG9pbnRlcj4oc2V0QXVkaW9TdHJlYW1Q
cm9wZXJ0aWVzQ2FsbGJhY2spLCB0aGlzKTsKIAotICAgIG1fcmVhZHlUaW1lckhhbmRsZXIuY2Fu
Y2VsKCk7CisgICAgbV9yZWFkeVRpbWVySGFuZGxlci5zdG9wKCk7CiAgICAgaWYgKG1fbWlzc2lu
Z1BsdWdpbnNDYWxsYmFjaykgewogICAgICAgICBtX21pc3NpbmdQbHVnaW5zQ2FsbGJhY2stPmlu
dmFsaWRhdGUoKTsKICAgICAgICAgbV9taXNzaW5nUGx1Z2luc0NhbGxiYWNrID0gbnVsbHB0cjsK
QEAgLTM2Niw2ICszNjUsMTEgQEAgZmxvYXQgTWVkaWFQbGF5ZXJQcml2YXRlR1N0cmVhbWVyOjpw
bGF5YmFja1Bvc2l0aW9uKCkgY29uc3QKICAgICByZXR1cm4gcmVzdWx0OwogfQogCit2b2lkIE1l
ZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1lcjo6cmVhZHlUaW1lckZpcmVkKCkKK3sKKyAgICBjaGFu
Z2VQaXBlbGluZVN0YXRlKEdTVF9TVEFURV9OVUxMKTsKK30KKwogYm9vbCBNZWRpYVBsYXllclBy
aXZhdGVHU3RyZWFtZXI6OmNoYW5nZVBpcGVsaW5lU3RhdGUoR3N0U3RhdGUgbmV3U3RhdGUpCiB7
CiAgICAgQVNTRVJUKG1fcGlwZWxpbmUpOwpAQCAtMzkzLDEyICszOTcsMTMgQEAgYm9vbCBNZWRp
YVBsYXllclByaXZhdGVHU3RyZWFtZXI6OmNoYW5nZVBpcGVsaW5lU3RhdGUoR3N0U3RhdGUgbmV3
U3RhdGUpCiAgICAgLy8gaWYgd2Ugc3RheSBmb3IgdG9vIGxvbmcgb24gUkVBRFkuCiAgICAgLy8g
QWxzbyBsZXRzIHJlbW92ZSB0aGUgdGltZXIgaWYgd2UgcmVxdWVzdCBhIHN0YXRlIGNoYW5nZSBm
b3IgYW55IHN0YXRlIG90aGVyIHRoYW4gUkVBRFkuCiAgICAgLy8gU2VlIGFsc28gaHR0cHM6Ly9i
dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTExNzM1NAotICAgIGlmIChuZXdTdGF0ZSA9
PSBHU1RfU1RBVEVfUkVBRFkgJiYgIW1fcmVhZHlUaW1lckhhbmRsZXIuaXNTY2hlZHVsZWQoKSkg
ewotICAgICAgICBtX3JlYWR5VGltZXJIYW5kbGVyLnNjaGVkdWxlQWZ0ZXJEZWxheSgiW1dlYktp
dF0gbWVkaWFQbGF5ZXJQcml2YXRlUmVhZHlTdGF0ZVRpbWVvdXRDYWxsYmFjayIsIHN0ZDo6ZnVu
Y3Rpb248dm9pZCgpPihbdGhpc10geyBjaGFuZ2VQaXBlbGluZVN0YXRlKEdTVF9TVEFURV9OVUxM
KTsgfSksCi0gICAgICAgICAgICBzdGQ6OmNocm9ubzo6c2Vjb25kcyhnUmVhZHlTdGF0ZVRpbWVy
SW50ZXJ2YWwpKTsKLSAgICB9IGVsc2UgaWYgKG5ld1N0YXRlICE9IEdTVF9TVEFURV9SRUFEWSAm
JiBtX3JlYWR5VGltZXJIYW5kbGVyLmlzU2NoZWR1bGVkKCkpIHsKLSAgICAgICAgbV9yZWFkeVRp
bWVySGFuZGxlci5jYW5jZWwoKTsKLSAgICB9CisgICAgaWYgKG5ld1N0YXRlID09IEdTVF9TVEFU
RV9SRUFEWSAmJiAhbV9yZWFkeVRpbWVySGFuZGxlci5pc0FjdGl2ZSgpKSB7CisgICAgICAgIC8v
IE1heCBpbnRlcnZhbCBpbiBzZWNvbmRzIHRvIHN0YXkgaW4gdGhlIFJFQURZIHN0YXRlIG9uIG1h
bnVhbAorICAgICAgICAvLyBzdGF0ZSBjaGFuZ2UgcmVxdWVzdHMuCisgICAgICAgIHN0YXRpYyBj
b25zdCBkb3VibGUgcmVhZHlTdGF0ZVRpbWVyRGVsYXkgPSA2MDsKKyAgICAgICAgbV9yZWFkeVRp
bWVySGFuZGxlci5zdGFydE9uZVNob3QocmVhZHlTdGF0ZVRpbWVyRGVsYXkpOworICAgIH0gZWxz
ZSBpZiAobmV3U3RhdGUgIT0gR1NUX1NUQVRFX1JFQURZKQorICAgICAgICBtX3JlYWR5VGltZXJI
YW5kbGVyLnN0b3AoKTsKIAogICAgIHJldHVybiB0cnVlOwogfQpAQCAtMTY4Nyw3ICsxNjkyLDcg
QEAgdm9pZCBNZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXI6OmxvYWRpbmdGYWlsZWQoTWVkaWFQ
bGF5ZXI6Ok5ldHdvcmtTdGF0ZSBlcnJvcikKICAgICB9CiAKICAgICAvLyBMb2FkaW5nIGZhaWxl
ZCwgcmVtb3ZlIHJlYWR5IHRpbWVyLgotICAgIG1fcmVhZHlUaW1lckhhbmRsZXIuY2FuY2VsKCk7
CisgICAgbV9yZWFkeVRpbWVySGFuZGxlci5zdG9wKCk7CiB9CiAKIHN0YXRpYyBIYXNoU2V0PFN0
cmluZz4gbWltZVR5cGVDYWNoZSgpCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9y
bS9ncmFwaGljcy9nc3RyZWFtZXIvTWVkaWFQbGF5ZXJQcml2YXRlR1N0cmVhbWVyLmggYi9Tb3Vy
Y2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9nc3RyZWFtZXIvTWVkaWFQbGF5ZXJQcml2YXRl
R1N0cmVhbWVyLmgKaW5kZXggMTc5MTNkZi4uYWU5YWJiNyAxMDA2NDQKLS0tIGEvU291cmNlL1dl
YkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvZ3N0cmVhbWVyL01lZGlhUGxheWVyUHJpdmF0ZUdTdHJl
YW1lci5oCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2dzdHJlYW1lci9N
ZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXIuaApAQCAtMzMsNiArMzMsNyBAQAogI2luY2x1ZGUg
PGdzdC9nc3QuaD4KICNpbmNsdWRlIDxnc3QvcGJ1dGlscy9pbnN0YWxsLXBsdWdpbnMuaD4KICNp
bmNsdWRlIDx3dGYvRm9yd2FyZC5oPgorI2luY2x1ZGUgPHd0Zi9SdW5Mb29wLmg+CiAjaW5jbHVk
ZSA8d3RmL1dlYWtQdHIuaD4KICNpbmNsdWRlIDx3dGYvZ2xpYi9HVGhyZWFkU2FmZU1haW5Mb29w
U291cmNlLmg+CiAKQEAgLTE4OCw2ICsxODksOCBAQCBwcml2YXRlOgogICAgIE1lZGlhVGltZSB0
b3RhbEZyYW1lRGVsYXkoKSBvdmVycmlkZSB7IHJldHVybiBNZWRpYVRpbWU6Onplcm9UaW1lKCk7
IH0KICNlbmRpZgogCisgICAgdm9pZCByZWFkeVRpbWVyRmlyZWQoKTsKKwogICAgIFdlYWtQdHJG
YWN0b3J5PE1lZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1lcj4gbV93ZWFrUHRyRmFjdG9yeTsKIAog
ICAgIEdSZWZQdHI8R3N0RWxlbWVudD4gbV9zb3VyY2U7CkBAIC0yMjksNyArMjMyLDcgQEAgcHJp
dmF0ZToKICAgICBHVGhyZWFkU2FmZU1haW5Mb29wU291cmNlIG1fdGV4dFRpbWVySGFuZGxlcjsK
ICAgICBHVGhyZWFkU2FmZU1haW5Mb29wU291cmNlIG1fdmlkZW9UaW1lckhhbmRsZXI7CiAgICAg
R1RocmVhZFNhZmVNYWluTG9vcFNvdXJjZSBtX3ZpZGVvQ2Fwc1RpbWVySGFuZGxlcjsKLSAgICBH
VGhyZWFkU2FmZU1haW5Mb29wU291cmNlIG1fcmVhZHlUaW1lckhhbmRsZXI7CisgICAgUnVuTG9v
cDo6VGltZXI8TWVkaWFQbGF5ZXJQcml2YXRlR1N0cmVhbWVyPiBtX3JlYWR5VGltZXJIYW5kbGVy
OwogICAgIG11dGFibGUgdW5zaWduZWQgbG9uZyBsb25nIG1fdG90YWxCeXRlczsKICAgICBVUkwg
bV91cmw7CiAgICAgYm9vbCBtX3ByZXNlcnZlc1BpdGNoOwo=
</data>
<flag name="review"
          id="289772"
          type_id="1"
          status="+"
          setter="pnormand"
    />
          </attachment>
      

    </bug>

</bugzilla>