<?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>97120</bug_id>
          
          <creation_ts>2012-09-19 10:41:47 -0700</creation_ts>
          <short_desc>Web Inspector: yellow on-hover pop-up won&apos;t go if another pane asynchronously opens</short_desc>
          <delta_ts>2012-09-25 09:22:41 -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>Web Inspector (Deprecated)</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</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="Peter Rybin">prybin</reporter>
          <assigned_to name="Mirela">mbudaes</assigned_to>
          <cc>apavlov</cc>
    
    <cc>bweinstein</cc>
    
    <cc>joepeck</cc>
    
    <cc>keishi</cc>
    
    <cc>loislo</cc>
    
    <cc>mbudaes</cc>
    
    <cc>mibalan</cc>
    
    <cc>pfeldman</cc>
    
    <cc>pmuellr</cc>
    
    <cc>rik</cc>
    
    <cc>timothy</cc>
    
    <cc>webkit.review.bot</cc>
    
    <cc>yurys</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>723650</commentid>
    <comment_count>0</comment_count>
    <who name="Peter Rybin">prybin</who>
    <bug_when>2012-09-19 10:41:47 -0700</bug_when>
    <thetext>General steps:
- in Sources tab provoke yellow inspect pop-up window by hovering over some variable in text
- in previous step learn the timing for hover to trigger evaluation
- manage to provoke inspect evaluation, but open ||&gt; drawer (sources list) before the pop-up actually appears
- yellow pop-up appears above ||&gt; drawer

Expected:
pointing away from pop-up makes it go

Actual:
the pop-up remains fixed as long as you move mouse inside the opened ||&gt; drawer. User have to learn that not only he has to point away from the pop-up, but also point to the original half-visible pane of source text. This is pretty much unintuitive.

Hints for reproducing
Enable pause on exceptions. Open http://design.ru Execution will stop on widgets.js with TypeError. A &quot;window&quot; variable is references in left-top corner of the text, which is handy for starting heavy evaluate and making it to open ||&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>724301</commentid>
    <comment_count>1</comment_count>
    <who name="Mirela">mbudaes</who>
    <bug_when>2012-09-20 02:00:22 -0700</bug_when>
    <thetext>I will try looking into this bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>726690</commentid>
    <comment_count>2</comment_count>
      <attachid>165388</attachid>
    <who name="Mirela">mbudaes</who>
    <bug_when>2012-09-24 08:29:07 -0700</bug_when>
    <thetext>Created attachment 165388
Add mouseout event listener on popover element.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>727395</commentid>
    <comment_count>3</comment_count>
      <attachid>165388</attachid>
    <who name="Alexander Pavlov (apavlov)">apavlov</who>
    <bug_when>2012-09-25 01:16:52 -0700</bug_when>
    <thetext>Comment on attachment 165388
Add mouseout event listener on popover element.

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

&gt; Source/WebCore/ChangeLog:3
&gt; +        Fix for bug 97210

Please remove this auto-added line

&gt; Source/WebCore/inspector/front-end/Popover.js:242
&gt; +        if (event.target === this._hoverElement || 

WebKit has no limit for the line length, so the entire condition can be placed on a single line. And we definitely do not observe closing parentheses on a separate line.

&gt; Source/WebCore/inspector/front-end/Popover.js:243
&gt; +            (this.isPopoverVisible() &amp;&amp; !event.toElement.isSelfOrDescendant(this._popover._contentDiv))

IIRC, event.toElement can be null/undefined if you are moving the pointer outside the browser window quickly (which is confirmed by http://trac.webkit.org/browser/trunk/Source/WebCore/dom/MouseEvent.cpp#L137). Also, &quot;toElement&quot; is an MSIE extension. The spec&apos;ed way is &quot;event.relatedTarget&quot;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>727413</commentid>
    <comment_count>4</comment_count>
      <attachid>165556</attachid>
    <who name="Mirela">mbudaes</who>
    <bug_when>2012-09-25 01:46:42 -0700</bug_when>
    <thetext>Created attachment 165556
Update the patch based on the review.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>727502</commentid>
    <comment_count>5</comment_count>
      <attachid>165556</attachid>
    <who name="Alexander Pavlov (apavlov)">apavlov</who>
    <bug_when>2012-09-25 04:28:17 -0700</bug_when>
    <thetext>Comment on attachment 165556
Update the patch based on the review.

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

&gt; Source/WebCore/ChangeLog:3
&gt; +        Fix for bug 97210

Hmm, please remove this auto-generated line...

&gt; Source/WebCore/inspector/front-end/Popover.js:242
&gt; +        if (event.target === this._hoverElement || (this.isPopoverVisible() &amp;&amp; !event.relatedTarget.isSelfOrDescendant(this._popover._contentDiv)))

Please check event.relatedTarget for null-ity to avoid throwing a JS exception.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>727677</commentid>
    <comment_count>6</comment_count>
      <attachid>165619</attachid>
    <who name="Mirela">mbudaes</who>
    <bug_when>2012-09-25 08:44:40 -0700</bug_when>
    <thetext>Created attachment 165619
New patch update</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>727682</commentid>
    <comment_count>7</comment_count>
      <attachid>165619</attachid>
    <who name="Alexander Pavlov (apavlov)">apavlov</who>
    <bug_when>2012-09-25 08:53:11 -0700</bug_when>
    <thetext>Comment on attachment 165619
New patch update

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

Two small nits to fix, and the patch will be ready to land

&gt; Source/WebCore/inspector/front-end/Popover.js:244
&gt; +            &amp;&amp; event.relatedTarget !== null

WebKit coding style guidelines require that null/0/undefined/whatever checks be performed by casting to a bool (e.g. ... &amp;&amp; event.relatedTarget &amp;&amp; ...) unless we need to disambiguate the actual value (e.g. null vs. undefined vs. &quot;&quot; vs. 0) - see http://www.webkit.org/coding/coding-style.html#zero-comparison

&gt; Source/WebCore/inspector/front-end/Popover.js:246
&gt; +        if (event.target === this._hoverElement || isPopoverMouseOut )

Extraneous whitespace before &apos;)&apos;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>727686</commentid>
    <comment_count>8</comment_count>
      <attachid>165621</attachid>
    <who name="Mirela">mbudaes</who>
    <bug_when>2012-09-25 09:01:48 -0700</bug_when>
    <thetext>Created attachment 165621
Address review comments</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>727688</commentid>
    <comment_count>9</comment_count>
      <attachid>165621</attachid>
    <who name="Alexander Pavlov (apavlov)">apavlov</who>
    <bug_when>2012-09-25 09:03:18 -0700</bug_when>
    <thetext>Comment on attachment 165621
Address review comments

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>727702</commentid>
    <comment_count>10</comment_count>
      <attachid>165621</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-09-25 09:22:37 -0700</bug_when>
    <thetext>Comment on attachment 165621
Address review comments

Clearing flags on attachment: 165621

Committed r129516: &lt;http://trac.webkit.org/changeset/129516&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>727703</commentid>
    <comment_count>11</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-09-25 09:22:41 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>165388</attachid>
            <date>2012-09-24 08:29:07 -0700</date>
            <delta_ts>2012-09-25 01:16:52 -0700</delta_ts>
            <desc>Add mouseout event listener on popover element.</desc>
            <filename>b97120.patch</filename>
            <type>text/plain</type>
            <size>2035</size>
            <attacher name="Mirela">mbudaes</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCBjOTBiOWRlLi41N2YzNDlmIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjAg
QEAKKzIwMTItMDktMjQgIG1pcmUgIDxtYnVkYWVzQGFkb2JlLmNvbT4KKworICAgICAgICBGaXgg
Zm9yIGJ1ZyA5NzIxMAorCisgICAgICAgIFdlYiBJbnNwZWN0b3I6IHllbGxvdyBvbi1ob3ZlciBw
b3AtdXAgd29uJ3QgZ28gaWYgYW5vdGhlciBwYW5lIGFzeW5jaHJvbm91c2x5IG9wZW5zCisgICAg
ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD05NzEyMAorCisgICAg
ICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEFkZCBtb3VzZW91dCBl
dmVudCBsaXN0ZW5lciB0byBwb3BvdmVyIGVsZW1lbnQgc28gdGhhdCB0aGUgcG9wLXVwIHdpbGwg
Z28gd2hlbiBtb3ZpbmcgdG8gYW5vdGhlciBwYW5lbC4KKworICAgICAgICBObyBuZXcgdGVzdHMu
CisKKyAgICAgICAgKiBpbnNwZWN0b3IvZnJvbnQtZW5kL1BvcG92ZXIuanM6CisgICAgICAgIChX
ZWJJbnNwZWN0b3IuUG9wb3Zlci5wcm90b3R5cGUuc2hvdyk6CisgICAgICAgIChXZWJJbnNwZWN0
b3IuUG9wb3ZlckhlbHBlci5wcm90b3R5cGUuX21vdXNlT3V0KToKKwogMjAxMi0wOS0yNCAgTmlj
byBXZWJlciAgPHRoYWtpc0BjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgW2Nocm9taXVtXSBSZW1v
dmUgQml0TG9ja2VyU2tpYS4gSXQncyBhcHBhcmVudGx5IG5vdCB1c2VkLgpkaWZmIC0tZ2l0IGEv
U291cmNlL1dlYkNvcmUvaW5zcGVjdG9yL2Zyb250LWVuZC9Qb3BvdmVyLmpzIGIvU291cmNlL1dl
YkNvcmUvaW5zcGVjdG9yL2Zyb250LWVuZC9Qb3BvdmVyLmpzCmluZGV4IDVkZmIzM2YuLjFiYzNh
MjcgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvUG9wb3Zl
ci5qcworKysgYi9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL1BvcG92ZXIuanMK
QEAgLTc0LDggKzc0LDEwIEBAIFdlYkluc3BlY3Rvci5Qb3BvdmVyLnByb3RvdHlwZSA9IHsKICAg
ICAgICAgZG9jdW1lbnQuYm9keS5hcHBlbmRDaGlsZCh0aGlzLmVsZW1lbnQpOwogICAgICAgICB0
aGlzLl9wb3NpdGlvbkVsZW1lbnQoYW5jaG9yLCBwcmVmZXJyZWRXaWR0aCwgcHJlZmVycmVkSGVp
Z2h0KTsKICAgICAgICAgdGhpcy5fdmlzaWJsZSA9IHRydWU7Ci0gICAgICAgIGlmICh0aGlzLl9w
b3BvdmVySGVscGVyKQorICAgICAgICBpZiAodGhpcy5fcG9wb3ZlckhlbHBlcikgewogICAgICAg
ICAgICAgY29udGVudEVsZW1lbnQuYWRkRXZlbnRMaXN0ZW5lcigibW91c2Vtb3ZlIiwgdGhpcy5f
cG9wb3ZlckhlbHBlci5fa2lsbEhpZGVQb3BvdmVyVGltZXIuYmluZCh0aGlzLl9wb3BvdmVySGVs
cGVyKSwgdHJ1ZSk7CisgICAgICAgICAgICB0aGlzLmVsZW1lbnQuYWRkRXZlbnRMaXN0ZW5lcigi
bW91c2VvdXQiLCB0aGlzLl9wb3BvdmVySGVscGVyLl9tb3VzZU91dC5iaW5kKHRoaXMuX3BvcG92
ZXJIZWxwZXIpLCB0cnVlKTsKKyAgICAgICAgfQogICAgIH0sCiAKICAgICBoaWRlOiBmdW5jdGlv
bigpCkBAIC0yMzcsNyArMjM5LDkgQEAgV2ViSW5zcGVjdG9yLlBvcG92ZXJIZWxwZXIucHJvdG90
eXBlID0gewogCiAgICAgX21vdXNlT3V0OiBmdW5jdGlvbihldmVudCkKICAgICB7Ci0gICAgICAg
IGlmIChldmVudC50YXJnZXQgPT09IHRoaXMuX2hvdmVyRWxlbWVudCkKKyAgICAgICAgaWYgKGV2
ZW50LnRhcmdldCA9PT0gdGhpcy5faG92ZXJFbGVtZW50IHx8IAorICAgICAgICAgICAgKHRoaXMu
aXNQb3BvdmVyVmlzaWJsZSgpICYmICFldmVudC50b0VsZW1lbnQuaXNTZWxmT3JEZXNjZW5kYW50
KHRoaXMuX3BvcG92ZXIuX2NvbnRlbnREaXYpKQorICAgICAgICAgICApCiAgICAgICAgICAgICB0
aGlzLl9zdGFydEhpZGVQb3BvdmVyVGltZXIoKTsKICAgICB9LAogCg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>165556</attachid>
            <date>2012-09-25 01:46:42 -0700</date>
            <delta_ts>2012-09-25 04:28:17 -0700</delta_ts>
            <desc>Update the patch based on the review.</desc>
            <filename>b97120.patch</filename>
            <type>text/plain</type>
            <size>2012</size>
            <attacher name="Mirela">mbudaes</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCBjOTBiOWRlLi41N2YzNDlmIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjAg
QEAKKzIwMTItMDktMjQgIG1pcmUgIDxtYnVkYWVzQGFkb2JlLmNvbT4KKworICAgICAgICBGaXgg
Zm9yIGJ1ZyA5NzIxMAorCisgICAgICAgIFdlYiBJbnNwZWN0b3I6IHllbGxvdyBvbi1ob3ZlciBw
b3AtdXAgd29uJ3QgZ28gaWYgYW5vdGhlciBwYW5lIGFzeW5jaHJvbm91c2x5IG9wZW5zCisgICAg
ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD05NzEyMAorCisgICAg
ICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEFkZCBtb3VzZW91dCBl
dmVudCBsaXN0ZW5lciB0byBwb3BvdmVyIGVsZW1lbnQgc28gdGhhdCB0aGUgcG9wLXVwIHdpbGwg
Z28gd2hlbiBtb3ZpbmcgdG8gYW5vdGhlciBwYW5lbC4KKworICAgICAgICBObyBuZXcgdGVzdHMu
CisKKyAgICAgICAgKiBpbnNwZWN0b3IvZnJvbnQtZW5kL1BvcG92ZXIuanM6CisgICAgICAgIChX
ZWJJbnNwZWN0b3IuUG9wb3Zlci5wcm90b3R5cGUuc2hvdyk6CisgICAgICAgIChXZWJJbnNwZWN0
b3IuUG9wb3ZlckhlbHBlci5wcm90b3R5cGUuX21vdXNlT3V0KToKKwogMjAxMi0wOS0yNCAgTmlj
byBXZWJlciAgPHRoYWtpc0BjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgW2Nocm9taXVtXSBSZW1v
dmUgQml0TG9ja2VyU2tpYS4gSXQncyBhcHBhcmVudGx5IG5vdCB1c2VkLgpkaWZmIC0tZ2l0IGEv
U291cmNlL1dlYkNvcmUvaW5zcGVjdG9yL2Zyb250LWVuZC9Qb3BvdmVyLmpzIGIvU291cmNlL1dl
YkNvcmUvaW5zcGVjdG9yL2Zyb250LWVuZC9Qb3BvdmVyLmpzCmluZGV4IDVkZmIzM2YuLmE0NDU3
OWMgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvUG9wb3Zl
ci5qcworKysgYi9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL1BvcG92ZXIuanMK
QEAgLTc0LDggKzc0LDEwIEBAIFdlYkluc3BlY3Rvci5Qb3BvdmVyLnByb3RvdHlwZSA9IHsKICAg
ICAgICAgZG9jdW1lbnQuYm9keS5hcHBlbmRDaGlsZCh0aGlzLmVsZW1lbnQpOwogICAgICAgICB0
aGlzLl9wb3NpdGlvbkVsZW1lbnQoYW5jaG9yLCBwcmVmZXJyZWRXaWR0aCwgcHJlZmVycmVkSGVp
Z2h0KTsKICAgICAgICAgdGhpcy5fdmlzaWJsZSA9IHRydWU7Ci0gICAgICAgIGlmICh0aGlzLl9w
b3BvdmVySGVscGVyKQorICAgICAgICBpZiAodGhpcy5fcG9wb3ZlckhlbHBlcikgewogICAgICAg
ICAgICAgY29udGVudEVsZW1lbnQuYWRkRXZlbnRMaXN0ZW5lcigibW91c2Vtb3ZlIiwgdGhpcy5f
cG9wb3ZlckhlbHBlci5fa2lsbEhpZGVQb3BvdmVyVGltZXIuYmluZCh0aGlzLl9wb3BvdmVySGVs
cGVyKSwgdHJ1ZSk7CisgICAgICAgICAgICB0aGlzLmVsZW1lbnQuYWRkRXZlbnRMaXN0ZW5lcigi
bW91c2VvdXQiLCB0aGlzLl9wb3BvdmVySGVscGVyLl9tb3VzZU91dC5iaW5kKHRoaXMuX3BvcG92
ZXJIZWxwZXIpLCB0cnVlKTsKKyAgICAgICAgfQogICAgIH0sCiAKICAgICBoaWRlOiBmdW5jdGlv
bigpCkBAIC0yMzcsNyArMjM5LDcgQEAgV2ViSW5zcGVjdG9yLlBvcG92ZXJIZWxwZXIucHJvdG90
eXBlID0gewogCiAgICAgX21vdXNlT3V0OiBmdW5jdGlvbihldmVudCkKICAgICB7Ci0gICAgICAg
IGlmIChldmVudC50YXJnZXQgPT09IHRoaXMuX2hvdmVyRWxlbWVudCkKKyAgICAgICAgaWYgKGV2
ZW50LnRhcmdldCA9PT0gdGhpcy5faG92ZXJFbGVtZW50IHx8ICh0aGlzLmlzUG9wb3ZlclZpc2li
bGUoKSAmJiAhZXZlbnQucmVsYXRlZFRhcmdldC5pc1NlbGZPckRlc2NlbmRhbnQodGhpcy5fcG9w
b3Zlci5fY29udGVudERpdikpKQogICAgICAgICAgICAgdGhpcy5fc3RhcnRIaWRlUG9wb3ZlclRp
bWVyKCk7CiAgICAgfSwKIAo=
</data>
<flag name="review"
          id="177623"
          type_id="1"
          status="-"
          setter="apavlov"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>165619</attachid>
            <date>2012-09-25 08:44:40 -0700</date>
            <delta_ts>2012-09-25 08:53:11 -0700</delta_ts>
            <desc>New patch update</desc>
            <filename>b97120.patch</filename>
            <type>text/plain</type>
            <size>2173</size>
            <attacher name="Mirela">mbudaes</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCBjOTBiOWRlLi5kZWEwYWVkIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTgg
QEAKKzIwMTItMDktMjQgIG1pcmUgIDxtYnVkYWVzQGFkb2JlLmNvbT4KKworICAgICAgICBXZWIg
SW5zcGVjdG9yOiB5ZWxsb3cgb24taG92ZXIgcG9wLXVwIHdvbid0IGdvIGlmIGFub3RoZXIgcGFu
ZSBhc3luY2hyb25vdXNseSBvcGVucworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z
aG93X2J1Zy5jZ2k/aWQ9OTcxMjAKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMh
KS4KKworICAgICAgICBBZGQgbW91c2VvdXQgZXZlbnQgbGlzdGVuZXIgdG8gcG9wb3ZlciBlbGVt
ZW50IHNvIHRoYXQgdGhlIHBvcC11cCB3aWxsIGdvIHdoZW4gbW92aW5nIHRvIGFub3RoZXIgcGFu
ZWwuCisKKyAgICAgICAgTm8gbmV3IHRlc3RzLgorCisgICAgICAgICogaW5zcGVjdG9yL2Zyb250
LWVuZC9Qb3BvdmVyLmpzOgorICAgICAgICAoV2ViSW5zcGVjdG9yLlBvcG92ZXIucHJvdG90eXBl
LnNob3cpOgorICAgICAgICAoV2ViSW5zcGVjdG9yLlBvcG92ZXJIZWxwZXIucHJvdG90eXBlLl9t
b3VzZU91dCk6CisKIDIwMTItMDktMjQgIE5pY28gV2ViZXIgIDx0aGFraXNAY2hyb21pdW0ub3Jn
PgogCiAgICAgICAgIFtjaHJvbWl1bV0gUmVtb3ZlIEJpdExvY2tlclNraWEuIEl0J3MgYXBwYXJl
bnRseSBub3QgdXNlZC4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9mcm9u
dC1lbmQvUG9wb3Zlci5qcyBiL1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvUG9w
b3Zlci5qcwppbmRleCA1ZGZiMzNmLi5iY2I1YzRkIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9pbnNwZWN0b3IvZnJvbnQtZW5kL1BvcG92ZXIuanMKKysrIGIvU291cmNlL1dlYkNvcmUvaW5z
cGVjdG9yL2Zyb250LWVuZC9Qb3BvdmVyLmpzCkBAIC03NCw4ICs3NCwxMCBAQCBXZWJJbnNwZWN0
b3IuUG9wb3Zlci5wcm90b3R5cGUgPSB7CiAgICAgICAgIGRvY3VtZW50LmJvZHkuYXBwZW5kQ2hp
bGQodGhpcy5lbGVtZW50KTsKICAgICAgICAgdGhpcy5fcG9zaXRpb25FbGVtZW50KGFuY2hvciwg
cHJlZmVycmVkV2lkdGgsIHByZWZlcnJlZEhlaWdodCk7CiAgICAgICAgIHRoaXMuX3Zpc2libGUg
PSB0cnVlOwotICAgICAgICBpZiAodGhpcy5fcG9wb3ZlckhlbHBlcikKKyAgICAgICAgaWYgKHRo
aXMuX3BvcG92ZXJIZWxwZXIpIHsKICAgICAgICAgICAgIGNvbnRlbnRFbGVtZW50LmFkZEV2ZW50
TGlzdGVuZXIoIm1vdXNlbW92ZSIsIHRoaXMuX3BvcG92ZXJIZWxwZXIuX2tpbGxIaWRlUG9wb3Zl
clRpbWVyLmJpbmQodGhpcy5fcG9wb3ZlckhlbHBlciksIHRydWUpOworICAgICAgICAgICAgdGhp
cy5lbGVtZW50LmFkZEV2ZW50TGlzdGVuZXIoIm1vdXNlb3V0IiwgdGhpcy5fcG9wb3ZlckhlbHBl
ci5fbW91c2VPdXQuYmluZCh0aGlzLl9wb3BvdmVySGVscGVyKSwgdHJ1ZSk7CisgICAgICAgIH0K
ICAgICB9LAogCiAgICAgaGlkZTogZnVuY3Rpb24oKQpAQCAtMjM3LDcgKzIzOSwxMSBAQCBXZWJJ
bnNwZWN0b3IuUG9wb3ZlckhlbHBlci5wcm90b3R5cGUgPSB7CiAKICAgICBfbW91c2VPdXQ6IGZ1
bmN0aW9uKGV2ZW50KQogICAgIHsKLSAgICAgICAgaWYgKGV2ZW50LnRhcmdldCA9PT0gdGhpcy5f
aG92ZXJFbGVtZW50KQorICAgICAgICAvLyBpc1BvcG92ZXJNb3VzZU91dCAtIGNoZWNrIHRoZSBt
b3VzZSBpcyBvdXQgb2YgdGhlIHBvcG92ZXIgd2luZG93ICAgCisgICAgICAgIHZhciBpc1BvcG92
ZXJNb3VzZU91dCA9IHRoaXMuaXNQb3BvdmVyVmlzaWJsZSgpCisgICAgICAgICAgICAmJiBldmVu
dC5yZWxhdGVkVGFyZ2V0ICE9PSBudWxsCisgICAgICAgICAgICAmJiAhZXZlbnQucmVsYXRlZFRh
cmdldC5pc1NlbGZPckRlc2NlbmRhbnQodGhpcy5fcG9wb3Zlci5fY29udGVudERpdik7CisgICAg
ICAgIGlmIChldmVudC50YXJnZXQgPT09IHRoaXMuX2hvdmVyRWxlbWVudCB8fCBpc1BvcG92ZXJN
b3VzZU91dCApCiAgICAgICAgICAgICB0aGlzLl9zdGFydEhpZGVQb3BvdmVyVGltZXIoKTsKICAg
ICB9LAogCg==
</data>
<flag name="review"
          id="177727"
          type_id="1"
          status="-"
          setter="apavlov"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>165621</attachid>
            <date>2012-09-25 09:01:48 -0700</date>
            <delta_ts>2012-09-25 09:22:37 -0700</delta_ts>
            <desc>Address review comments</desc>
            <filename>b97120.patch</filename>
            <type>text/plain</type>
            <size>2163</size>
            <attacher name="Mirela">mbudaes</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCBjOTBiOWRlLi5kZWEwYWVkIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTgg
QEAKKzIwMTItMDktMjQgIG1pcmUgIDxtYnVkYWVzQGFkb2JlLmNvbT4KKworICAgICAgICBXZWIg
SW5zcGVjdG9yOiB5ZWxsb3cgb24taG92ZXIgcG9wLXVwIHdvbid0IGdvIGlmIGFub3RoZXIgcGFu
ZSBhc3luY2hyb25vdXNseSBvcGVucworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z
aG93X2J1Zy5jZ2k/aWQ9OTcxMjAKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMh
KS4KKworICAgICAgICBBZGQgbW91c2VvdXQgZXZlbnQgbGlzdGVuZXIgdG8gcG9wb3ZlciBlbGVt
ZW50IHNvIHRoYXQgdGhlIHBvcC11cCB3aWxsIGdvIHdoZW4gbW92aW5nIHRvIGFub3RoZXIgcGFu
ZWwuCisKKyAgICAgICAgTm8gbmV3IHRlc3RzLgorCisgICAgICAgICogaW5zcGVjdG9yL2Zyb250
LWVuZC9Qb3BvdmVyLmpzOgorICAgICAgICAoV2ViSW5zcGVjdG9yLlBvcG92ZXIucHJvdG90eXBl
LnNob3cpOgorICAgICAgICAoV2ViSW5zcGVjdG9yLlBvcG92ZXJIZWxwZXIucHJvdG90eXBlLl9t
b3VzZU91dCk6CisKIDIwMTItMDktMjQgIE5pY28gV2ViZXIgIDx0aGFraXNAY2hyb21pdW0ub3Jn
PgogCiAgICAgICAgIFtjaHJvbWl1bV0gUmVtb3ZlIEJpdExvY2tlclNraWEuIEl0J3MgYXBwYXJl
bnRseSBub3QgdXNlZC4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9mcm9u
dC1lbmQvUG9wb3Zlci5qcyBiL1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvUG9w
b3Zlci5qcwppbmRleCA1ZGZiMzNmLi5jZTFlMTBhIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9pbnNwZWN0b3IvZnJvbnQtZW5kL1BvcG92ZXIuanMKKysrIGIvU291cmNlL1dlYkNvcmUvaW5z
cGVjdG9yL2Zyb250LWVuZC9Qb3BvdmVyLmpzCkBAIC03NCw4ICs3NCwxMCBAQCBXZWJJbnNwZWN0
b3IuUG9wb3Zlci5wcm90b3R5cGUgPSB7CiAgICAgICAgIGRvY3VtZW50LmJvZHkuYXBwZW5kQ2hp
bGQodGhpcy5lbGVtZW50KTsKICAgICAgICAgdGhpcy5fcG9zaXRpb25FbGVtZW50KGFuY2hvciwg
cHJlZmVycmVkV2lkdGgsIHByZWZlcnJlZEhlaWdodCk7CiAgICAgICAgIHRoaXMuX3Zpc2libGUg
PSB0cnVlOwotICAgICAgICBpZiAodGhpcy5fcG9wb3ZlckhlbHBlcikKKyAgICAgICAgaWYgKHRo
aXMuX3BvcG92ZXJIZWxwZXIpIHsKICAgICAgICAgICAgIGNvbnRlbnRFbGVtZW50LmFkZEV2ZW50
TGlzdGVuZXIoIm1vdXNlbW92ZSIsIHRoaXMuX3BvcG92ZXJIZWxwZXIuX2tpbGxIaWRlUG9wb3Zl
clRpbWVyLmJpbmQodGhpcy5fcG9wb3ZlckhlbHBlciksIHRydWUpOworICAgICAgICAgICAgdGhp
cy5lbGVtZW50LmFkZEV2ZW50TGlzdGVuZXIoIm1vdXNlb3V0IiwgdGhpcy5fcG9wb3ZlckhlbHBl
ci5fbW91c2VPdXQuYmluZCh0aGlzLl9wb3BvdmVySGVscGVyKSwgdHJ1ZSk7CisgICAgICAgIH0K
ICAgICB9LAogCiAgICAgaGlkZTogZnVuY3Rpb24oKQpAQCAtMjM3LDcgKzIzOSwxMSBAQCBXZWJJ
bnNwZWN0b3IuUG9wb3ZlckhlbHBlci5wcm90b3R5cGUgPSB7CiAKICAgICBfbW91c2VPdXQ6IGZ1
bmN0aW9uKGV2ZW50KQogICAgIHsKLSAgICAgICAgaWYgKGV2ZW50LnRhcmdldCA9PT0gdGhpcy5f
aG92ZXJFbGVtZW50KQorICAgICAgICAvLyBpc1BvcG92ZXJNb3VzZU91dCAtIGNoZWNrIHRoZSBt
b3VzZSBpcyBvdXQgb2YgdGhlIHBvcG92ZXIgd2luZG93ICAgCisgICAgICAgIHZhciBpc1BvcG92
ZXJNb3VzZU91dCA9IHRoaXMuaXNQb3BvdmVyVmlzaWJsZSgpCisgICAgICAgICAgICAmJiBldmVu
dC5yZWxhdGVkVGFyZ2V0CisgICAgICAgICAgICAmJiAhZXZlbnQucmVsYXRlZFRhcmdldC5pc1Nl
bGZPckRlc2NlbmRhbnQodGhpcy5fcG9wb3Zlci5fY29udGVudERpdik7CisgICAgICAgIGlmIChl
dmVudC50YXJnZXQgPT09IHRoaXMuX2hvdmVyRWxlbWVudCB8fCBpc1BvcG92ZXJNb3VzZU91dCkK
ICAgICAgICAgICAgIHRoaXMuX3N0YXJ0SGlkZVBvcG92ZXJUaW1lcigpOwogICAgIH0sCiAK
</data>

          </attachment>
      

    </bug>

</bugzilla>