<?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>81143</bug_id>
          
          <creation_ts>2012-03-14 12:34:00 -0700</creation_ts>
          <short_desc>[Qt][WK2] Implement PageClient::isViewWindowActive()</short_desc>
          <delta_ts>2014-02-03 03:20:22 -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 Qt</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>INVALID</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Qt, QtTriaged</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>82053</dependson>
    
    <dependson>82167</dependson>
    
    <dependson>82184</dependson>
    
    <dependson>83887</dependson>
          <blocked>70236</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Jesus Sanchez-Palencia">jesus</reporter>
          <assigned_to name="Jesus Sanchez-Palencia">jesus</assigned_to>
          <cc>hausmann</cc>
    
    <cc>kenneth</cc>
    
    <cc>menard</cc>
    
    <cc>ossy</cc>
    
    <cc>webkit.review.bot</cc>
    
    <cc>zoltan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>578653</commentid>
    <comment_count>0</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-03-14 12:34:00 -0700</bug_when>
    <thetext>Since now QQuickCanvas provides API for telling us whether it is active or not, we can implement PageClient::isViewWindowActive.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>578863</commentid>
    <comment_count>1</comment_count>
      <attachid>131942</attachid>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-03-14 15:41:53 -0700</bug_when>
    <thetext>Created attachment 131942
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>578869</commentid>
    <comment_count>2</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-03-14 15:45:29 -0700</bug_when>
    <thetext>(In reply to comment #1)
&gt; Created an attachment (id=131942) [details]
&gt; Patch

This patch has a small workaround due to a segfault in Qt that happens everytime we try QQuickCanvas::isActive() on focusOutEvents. The issue was already reported and a fix for it is being reviewed. After it lands on Qt I can fix this and remove the //FIXME.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>579235</commentid>
    <comment_count>3</comment_count>
      <attachid>131942</attachid>
    <who name="Kenneth Rohde Christiansen">kenneth</who>
    <bug_when>2012-03-15 02:55:09 -0700</bug_when>
    <thetext>Comment on attachment 131942
Patch

why no url to the Qt bug?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>579317</commentid>
    <comment_count>4</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-03-15 06:28:55 -0700</bug_when>
    <thetext>(In reply to comment #3)
&gt; (From update of attachment 131942 [details])
&gt; why no url to the Qt bug?

http://codereview.qt-project.org/#change,20129

Sorry :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>579325</commentid>
    <comment_count>5</comment_count>
    <who name="Kenneth Rohde Christiansen">kenneth</who>
    <bug_when>2012-03-15 06:38:53 -0700</bug_when>
    <thetext>In the code with the FIXME</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>579333</commentid>
    <comment_count>6</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-03-15 06:45:03 -0700</bug_when>
    <thetext>(In reply to comment #5)
&gt; In the code with the FIXME

I can fix when landing. Anything else?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>579503</commentid>
    <comment_count>7</comment_count>
      <attachid>131942</attachid>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-03-15 10:43:51 -0700</bug_when>
    <thetext>Comment on attachment 131942
Patch

I will upload a new version now that the Qt Bug has been fixed. As soon as it goes through CI and we pick the new hash, I will land it. (after the r+ of course... :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>579544</commentid>
    <comment_count>8</comment_count>
      <attachid>132081</attachid>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-03-15 11:18:15 -0700</bug_when>
    <thetext>Created attachment 132081
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>579691</commentid>
    <comment_count>9</comment_count>
      <attachid>132081</attachid>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2012-03-15 13:38:22 -0700</bug_when>
    <thetext>Comment on attachment 132081
Patch

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

&gt; Source/WebKit2/UIProcess/qt/QtPageClient.cpp:259
&gt; +    return m_webView-&gt;canvas()-&gt;isActive();

What does it mean for web content if the view window is active? Should this perhaps be tied to the QQuickItem instead of the window?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>579712</commentid>
    <comment_count>10</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-03-15 14:04:12 -0700</bug_when>
    <thetext>(In reply to comment #9)
&gt; (From update of attachment 132081 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=132081&amp;action=review
&gt; 
&gt; &gt; Source/WebKit2/UIProcess/qt/QtPageClient.cpp:259
&gt; &gt; +    return m_webView-&gt;canvas()-&gt;isActive();
&gt; 
&gt; What does it mean for web content if the view window is active? Should this perhaps be tied to the QQuickItem instead of the window?

The PageClient API seems to have been thought for returning the Window state (PageClient::isViewWindowActive) for WebPageProxy (see WebPageProxy::viewStateDidChange). I can see this being tied to the Page Visibility Spec (http://www.w3.org/TR/page-visibility/), for example, in which there are states related to the state of the window itself (QQuickCanvas, in our case) and not directly to the view one.

For instance, let&apos;s say we have multiple tabs (webviews) but only one is in foreground and, therefore, has focus. This one will considered active while the others will be considered not active even if their top level window (canvas()) still is. I believe that is why we differentiate these states in WebPageProxy as ViewIsFocused and ViewWindowIsActive (besides ViewIsVisible and ViewIsInWindow).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>579714</commentid>
    <comment_count>11</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-03-15 14:05:15 -0700</bug_when>
    <thetext>(In reply to comment #10)
&gt; The PageClient API seems to have been thought for returning the Window state (PageClient::isViewWindowActive) for WebPageProxy (see WebPageProxy::viewStateDidChange). I can see this being tied to the Page Visibility Spec (http://www.w3.org/TR/page-visibility/), for example, in which there are states related to the state of the window itself (QQuickCanvas, in our case) and not directly to the view one.

By the way, such work is being tracked by:

https://bugs.webkit.org/show_bug.cgi?id=81154
https://bugs.webkit.org/show_bug.cgi?id=81164
and https://bugs.webkit.org/show_bug.cgi?id=69554</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>583484</commentid>
    <comment_count>12</comment_count>
      <attachid>132903</attachid>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-03-20 14:53:12 -0700</bug_when>
    <thetext>Created attachment 132903
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>583493</commentid>
    <comment_count>13</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-03-20 14:55:46 -0700</bug_when>
    <thetext>(In reply to comment #12)
&gt; Created an attachment (id=132903) [details]
&gt; Patch

Updated the patch due to a check that was missing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>586276</commentid>
    <comment_count>14</comment_count>
      <attachid>132903</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-03-23 06:27:22 -0700</bug_when>
    <thetext>Comment on attachment 132903
Patch

Clearing flags on attachment: 132903

Committed r111855: &lt;http://trac.webkit.org/changeset/111855&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>586277</commentid>
    <comment_count>15</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-03-23 06:27:28 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>586307</commentid>
    <comment_count>16</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-03-23 07:10:32 -0700</bug_when>
    <thetext>Reopen, because it broke 30+ tests - http://build.webkit.sed.hu/results/x86-32%20Linux%20Qt%20Release%20WebKit2/r111856%20%2821981%29/results.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>586724</commentid>
    <comment_count>17</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-03-23 14:10:57 -0700</bug_when>
    <thetext>(In reply to comment #16)
&gt; Reopen, because it broke 30+ tests - http://build.webkit.sed.hu/results/x86-32%20Linux%20Qt%20Release%20WebKit2/r111856%20%2821981%29/results.html

I&apos;ve found what caused the crashes and I&apos;m now looking for a way to tweak the WebKitTestRunner and its WrapperWindow in our favor.

I believe the problem is not with the patch itself but with some missing pieces in our PlatformWebView. I will update this bug as soon as I find out the root cause.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>586837</commentid>
    <comment_count>18</comment_count>
      <attachid>133579</attachid>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-03-23 16:03:05 -0700</bug_when>
    <thetext>Created attachment 133579
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>587095</commentid>
    <comment_count>19</comment_count>
      <attachid>133579</attachid>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-03-24 08:17:30 -0700</bug_when>
    <thetext>Comment on attachment 133579
Patch

Let&apos;s give it a try...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>587097</commentid>
    <comment_count>20</comment_count>
      <attachid>133579</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-03-24 08:24:10 -0700</bug_when>
    <thetext>Comment on attachment 133579
Patch

Clearing flags on attachment: 133579

Committed r111993: &lt;http://trac.webkit.org/changeset/111993&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>587098</commentid>
    <comment_count>21</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-03-24 08:24:16 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>587448</commentid>
    <comment_count>22</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-03-25 22:23:35 -0700</bug_when>
    <thetext>Reopen, because it broke many tests, see https://bugs.webkit.org/show_bug.cgi?id=82167 for details.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>587595</commentid>
    <comment_count>23</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-03-26 04:08:14 -0700</bug_when>
    <thetext>Rollout landed in http://trac.webkit.org/changeset/112073 (rs=darktears)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>587646</commentid>
    <comment_count>24</comment_count>
      <attachid>133579</attachid>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-03-26 05:54:27 -0700</bug_when>
    <thetext>Comment on attachment 133579
Patch

We need to wait the next round of Qt5 updates on the bots before landing this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>599237</commentid>
    <comment_count>25</comment_count>
      <attachid>133579</attachid>
    <who name="Kenneth Rohde Christiansen">kenneth</who>
    <bug_when>2012-04-10 07:03:14 -0700</bug_when>
    <thetext>Comment on attachment 133579
Patch

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

&gt; Tools/WebKitTestRunner/qt/PlatformWebViewQt.cpp:83
&gt; +    // QWindow does not create the underlying platform
&gt; +    // specific bits unless QWindow::setVisible() or QWindow::show() are called,
&gt; +    // and without it QWindow::isActive() will always return false.
&gt; +    // For these cases QWindow::create() is provided.

// QWindow delays creating of the underlying platform specific bits until ::setVisible()
// or ::show() are called. This makes ::isActive() always return false. In order to force
// the creation ::create() exists, so we use it here.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>599240</commentid>
    <comment_count>26</comment_count>
      <attachid>133579</attachid>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-04-10 07:07:32 -0700</bug_when>
    <thetext>Comment on attachment 133579
Patch

CQ has issues, I will land it manually.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>599279</commentid>
    <comment_count>27</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-04-10 08:15:40 -0700</bug_when>
    <thetext>Committed r113714: &lt;http://trac.webkit.org/changeset/113714&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>599314</commentid>
    <comment_count>28</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-04-10 08:55:12 -0700</bug_when>
    <thetext>Reopen, because it broke API tests:

FAIL!  : qmltests::DesktopWebViewLinkHovered::test_linkHovered() &apos;wait for signal linkHovered&apos; returned FALSE. ()
   Loc: [/home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/Source/WebKit2/UIProcess/API/qt/tests/qmltests/DesktopBehavior/tst_linkHovered.qml(49)]
FAIL!  : qmltests::DesktopWebViewLinkHovered::test_linkHoveredDoesntEmitRepeated() &apos;wait for signal linkHovered&apos; returned FALSE. ()
   Loc: [/home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/Source/WebKit2/UIProcess/API/qt/tests/qmltests/DesktopBehavior/tst_linkHovered.qml(68)]
FAIL!  : qmltests::DesktopWebViewLoadHtml::test_baseUrlAfterLoadHtml() &apos;wait for signal linkHovered&apos; returned FALSE. ()
   Loc: [/home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/Source/WebKit2/UIProcess/API/qt/tests/qmltests/DesktopBehavior/tst_loadHtml.qml(48)]
   

Could you check and fix it please?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>599319</commentid>
    <comment_count>29</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-04-10 09:05:22 -0700</bug_when>
    <thetext>Additionally it broke a layout test:
--- /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/fast/events/mouseout-on-window-expected.txt 
+++ /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/fast/events/mouseout-on-window-actual.txt 
@@ -1,2 +1,2 @@
 This test ensures that mouse out events are sent to the window.
-PASS: Received mouseout event.
+FAIL: No mouseout was sent.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>599329</commentid>
    <comment_count>30</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-04-10 09:18:39 -0700</bug_when>
    <thetext>And it made 24 new flakey test :(</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>599397</commentid>
    <comment_count>31</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-04-10 10:39:38 -0700</bug_when>
    <thetext>Wow! Ok, I will have a look at it now. I will roll it out if I need more time, Ossy.  Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>599475</commentid>
    <comment_count>32</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-04-10 12:13:28 -0700</bug_when>
    <thetext>(In reply to comment #28)
&gt; Reopen, because it broke API tests:
&gt;
&gt; Could you check and fix it please?

So, Ossy, I was debugging it with Alexis and we realized that the problem is happening because we run the tests in parallel. If you run &quot;run-qtwebkit-tests -j1&quot; you will see that the tests doesn&apos;t fail. 

Our belief is that running tests in parallel that depend on their window state is a potential mistake, which has now been flagged because this is exactly what this patch fixes.


(In reply to comment #29)
&gt; Additionally it broke a layout test:

This I can&apos;t reproduce this locally. =/</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>599547</commentid>
    <comment_count>33</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-04-10 12:55:48 -0700</bug_when>
    <thetext>(In reply to comment #32)
&gt; So, Ossy, I was debugging it with Alexis and we realized that the problem is happening because we run the tests in parallel. If you run &quot;run-qtwebkit-tests -j1&quot; you will see that the tests doesn&apos;t fail. 
&gt; 
&gt; Our belief is that running tests in parallel that depend on their window state is a potential mistake, which has now been flagged because this is exactly what this patch fixes.

I tried it locally, but I got same results as the bot results.
Did you run the tests in XFVB as the bots do?

xvfb-run -a --server-args=&quot;-screen 0 1024x768x24&quot; python ./Tools/Scripts/run-qtwebkit-tests --output-file=qt-unit-tests.html --do-not-open-results --timeout=120 WebKitBuild/Release/Source/WebKit2/UIProcess/API/qt/tests/

I got same results with -j1 and without -j1 ...

&gt; (In reply to comment #29)
&gt; &gt; Additionally it broke a layout test:
&gt; 
&gt; This I can&apos;t reproduce this locally. =/

It&apos;s hard to believe. :( Did you run all tests with Tools/Scripts/run-webkit-tests on the reference platform? (Ubuntu 11.10)
If you do it, fast/events/mouseout-on-window fails and there are
24 new flakey tests after your patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>599585</commentid>
    <comment_count>34</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-04-10 13:43:17 -0700</bug_when>
    <thetext>(In reply to comment #33)
&gt; I tried it locally, but I got same results as the bot results.
&gt; Did you run the tests in XFVB as the bots do?

Nope.

&gt; 
&gt; &gt; (In reply to comment #29)
&gt; &gt; &gt; Additionally it broke a layout test:
&gt; &gt; 
&gt; &gt; This I can&apos;t reproduce this locally. =/
&gt; 
&gt; It&apos;s hard to believe. :( Did you run all tests with Tools/Scripts/run-webkit-tests on the reference platform? (Ubuntu 11.10)
&gt; If you do it, fast/events/mouseout-on-window fails and there are
&gt; 24 new flakey tests after your patch.

Hard to believe?!
It was tested on two different machines (api tests with -j1 and this layout test specifically) but none of them use Ubuntu. Plus, a fast/event test that is called &quot;mouseout-on-window&quot; shouldn&apos;t have any platform-specific issues, right?

Anyway, I&apos;m having a deeper look in order to find a workaround for it somehow.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>599622</commentid>
    <comment_count>35</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-04-10 14:11:51 -0700</bug_when>
    <thetext>(In reply to comment #34)
&gt; It was tested on two different machines (api tests with -j1 and this layout test specifically) but none of them use Ubuntu. Plus, a fast/event test that is called &quot;mouseout-on-window&quot; shouldn&apos;t have any platform-specific issues, right?

By &quot;platform-specific&quot; here I meant &quot;distro-specific&quot;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>600298</commentid>
    <comment_count>36</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-04-11 08:54:04 -0700</bug_when>
    <thetext>Here is the new flakey tests after this patch:
-----------------------------------------------
  editing/pasteboard/copy-crash.html = TEXT PASS
  fast/block/positioning/absolute-appended-to-inline.html = TEXT PASS
  fast/css/hover-affects-child.html = TEXT PASS
  fast/css/pseudo-any.html = TEXT PASS
  fast/dom/shadow/shadow-boundary-crossing.html = TEXT PASS
  fast/dom/shadow/shadow-contents-event.html = TEXT PASS
  fast/events/dont-loose-last-event.html = TEXT PASS
  fast/events/event-sender-mouse-moved.html = TEXT PASS
  fast/events/iframe-onmousemove.html = TEXT PASS
  fast/events/mouse-moved-remove-frame-crash.html = TEXT PASS
  fast/events/mousemove-after-drag-over-scrollbar.html = TEXT PASS
  fast/events/mouseout-dead-node.html = TEXT PASS
  fast/events/mouseout-dead-subframe.html = TEXT PASS
  fast/events/mouseover-mouseout.html = TEXT PASS
  fast/events/mouseover-mouseout2.html = TEXT PASS
  fast/events/touch/touch-before-pressing-spin-button.html = TEXT PASS
  fast/forms/autofocus-input-css-style-change.html = TEXT PASS
  fast/forms/input-step-as-double.html = TEXT PASS
  fast/forms/number/spin-button-events.html = TEXT PASS
  fast/forms/number/spin-button-state.html = TEXT PASS
  fast/forms/number/spin-in-multi-column.html = TEXT PASS
  fast/forms/range/slider-mouse-events.html = TEXT PASS
  fast/frames/flattening/iframe-flattening-fixed-width.html = TEXT PASS
  fast/overflow/overflow-focus-ring.html = TEXT PASS
  fast/replaced/image-map-2.html = TEXT PASS
  scrollbars/overflow-custom-scrollbar-crash.html = TEXT PASS
  scrollbars/scrollbar-iframe-click-does-not-blur-content.html = TEXT PASS
  svg/custom/foreign-object-skew.svg = TEXT PASS
  svg/custom/js-update-container2.svg = TEXT PASS
  svg/custom/pointer-events-invalid-fill.svg = TEXT PASS
  svg/custom/use-instanceRoot-as-event-target.xhtml = TEXT PASS
  svg/zoom/page/zoom-foreignObject.svg = TEXT PASS
  svg/zoom/text/zoom-foreignObject.svg = TEXT PASS

And the new failing test:
--------------------------
  fast/css/hover-update.html = TEXT
  fast/css/nested-layers-with-hover.html = TEXT
  fast/events/mouseout-on-window.html = TEXT</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>600299</commentid>
    <comment_count>37</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-04-11 09:00:05 -0700</bug_when>
    <thetext>(In reply to comment #34)
&gt; &gt; &gt; This I can&apos;t reproduce this locally. =/
&gt; &gt; 
&gt; &gt; It&apos;s hard to believe. :( Did you run all tests with Tools/Scripts/run-webkit-tests on the reference platform? (Ubuntu 11.10)
&gt; &gt; If you do it, fast/events/mouseout-on-window fails and there are
&gt; &gt; 24 new flakey tests after your patch.
&gt; 
&gt; Hard to believe?!
&gt; It was tested on two different machines (api tests with -j1 and this layout test specifically) but none of them use Ubuntu. Plus, a fast/event test that is called &quot;mouseout-on-window&quot; shouldn&apos;t have any platform-specific issues, right?

I think it is the main problem. All developers should run tests on the _reference_ platform. And at least you should have at least one Ubuntu
to be able reproduce QtWebKit bugs occured on the buildbots.
 
&gt; Anyway, I&apos;m having a deeper look in order to find a workaround for it somehow.
Thanks for taking care.

To be clarify I won&apos;t roll out QtWebKit patches, obviously buggy patches, 
I don&apos;t wan&apos;t to be the grumpy bad guy who blocks the rapid developing.
But in this case our Qt5-WK2 bots aren&apos;t in a good shape. And it&apos;s hard 
to catch new flakey and new failing tests until it is fixed. :-/ But
if it&apos;s OK for you guys let&apos;s leave the bot in this bad shape.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>600302</commentid>
    <comment_count>38</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-04-11 09:02:45 -0700</bug_when>
    <thetext>I modified our master.cfg to run API tests with -j1. The runtime was 
increased from 0.5 minutes to 2.5 minutes, but tests still fail.
It&apos;s strange, because it worked for me locally in the same environment.
But layout tests might break something in the environment and it might
break API tests ... I have no idea what happens ...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>600382</commentid>
    <comment_count>39</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-04-11 10:35:36 -0700</bug_when>
    <thetext>(In reply to comment #38)
&gt; I modified our master.cfg to run API tests with -j1. The runtime was 
&gt; increased from 0.5 minutes to 2.5 minutes, but tests still fail.
&gt; It&apos;s strange, because it worked for me locally in the same environment.
&gt; But layout tests might break something in the environment and it might
&gt; break API tests ... I have no idea what happens ...

I will be investigating the issues today and I will update the bug with new comments.

Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>600678</commentid>
    <comment_count>40</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-04-11 16:10:27 -0700</bug_when>
    <thetext>(In reply to comment #39)
&gt; I will be investigating the issues today and I will update the bug with new comments.

An issue with my xvfb version and nvidia driver is preventing to run this as the bots do. I will need a bit more time to investigate and fix this. =/</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>601179</commentid>
    <comment_count>41</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-04-12 06:47:16 -0700</bug_when>
    <thetext>(In reply to comment #38)
&gt; I modified our master.cfg to run API tests with -j1. The runtime was 
&gt; increased from 0.5 minutes to 2.5 minutes, but tests still fail.
&gt; It&apos;s strange, because it worked for me locally in the same environment.
&gt; But layout tests might break something in the environment and it might
&gt; break API tests ... I have no idea what happens ...

After -j1 we got very strange results on the 32 bit Qt5-WK2 bot:
31 passed, 3 failed, 0 skipped, 3 crashed

http://build.webkit.sed.hu/builders/x86-32%20Linux%20Qt%20Release%20WebKit2/builds/22770/steps/API%20tests/logs/stdio

CRASHED: WebKitBuild/Release/Source/WebKit2/UIProcess/API/qt/tests/qmltests/tst_qmltests_WebView
CRASHED: WebKitBuild/Release/Source/WebKit2/UIProcess/API/qt/tests/qquickwebview/tst_qquickwebview
CRASHED: WebKitBuild/Release/Source/WebKit2/UIProcess/API/qt/tests/publicapi/tst_publicapi

Great! :(</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>601194</commentid>
    <comment_count>42</comment_count>
    <who name="Jesus Sanchez-Palencia">jesus</who>
    <bug_when>2012-04-12 07:05:46 -0700</bug_when>
    <thetext>(In reply to comment #41)
&gt; Great! :(

So remove the -j1 for the time being. But don&apos;t you think that not running the tests in parallel shouldn&apos;t be causing crashes?!!

As I said, Xvfb is not working properly with my nvidia card on arch linux. I&apos;m fixing this and setting up a virtual machine with the &quot;reference platform&quot; so I can debug the issues. Before that there is nothing I can do. If the patch is causing too much trouble on the bots, just roll it out as well and it will come back later when we all have understand what is going on with it. I would do it myself if I had X running right now. (and I would try -j1 without this patch just to see how the bots would behave as well)

Other than that, if you really want to help, try to understand what we have explained on irc and on this bug report and help us to debug it as well. There is something way too wrong and not only with the patch if even you said that you are getting different behavior on the bots and on your machine, which is running the &quot;reference platform&quot;.

thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>601249</commentid>
    <comment_count>43</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-04-12 08:30:27 -0700</bug_when>
    <thetext>(In reply to comment #42)
&gt; (In reply to comment #41)
&gt; &gt; Great! :(
&gt; 
&gt; So remove the -j1 for the time being. But don&apos;t you think that not running the tests in parallel shouldn&apos;t be causing crashes?!!

I removed -j1 and now the runtime is 27 sec again and there isn&apos;t any crashes. Very strange.
 
&gt; As I said, Xvfb is not working properly with my nvidia card on arch linux. I&apos;m fixing this and setting up a virtual machine with the &quot;reference platform&quot; so I can debug the issues. Before that there is nothing I can do. If the patch is causing too much trouble on the bots, just roll it out as well and it will come back later when we all have understand what is going on with it. I would do it myself if I had X running right now. (and I would try -j1 without this patch just to see how the bots would behave as well)
&gt; 
&gt; Other than that, if you really want to help, try to understand what we have explained on irc and on this bug report and help us to debug it as well. There is something way too wrong and not only with the patch if even you said that you are getting different behavior on the bots and on your machine, which is running the &quot;reference platform&quot;.

I&apos;m terrible sorry, but I don&apos;t have too much free time for debugging when
I&apos;m filing bugs about new regressions, rebase expected files instead of
authors, digging which patch broke what in 24/7. :(

I agree, something is really wrong with run-qtwebkit-tests script near -j1 option ... I&apos;ll check it soon with and without your patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>975421</commentid>
    <comment_count>44</comment_count>
    <who name="Jocelyn Turcotte">jturcotte</who>
    <bug_when>2014-02-03 03:20:22 -0800</bug_when>
    <thetext>=== Bulk closing of Qt bugs ===

If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary.

If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>131942</attachid>
            <date>2012-03-14 15:41:53 -0700</date>
            <delta_ts>2012-03-15 10:43:51 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-81143-20120314194117.patch</filename>
            <type>text/plain</type>
            <size>4105</size>
            <attacher name="Jesus Sanchez-Palencia">jesus</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTEwNzM4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggMjczNDJiZmUzYWFlYzgz
OWYzOTQwZDUxMDAzY2UyNmEzNzczOGZmOC4uM2UyNWY2ZDQ1NmVmYzI5NGIyMWI5MjhjYzgwYzY2
NTQ0MzIwZTU4YiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDIyIEBACisyMDEyLTAzLTE0ICBKZXN1
cyBTYW5jaGV6LVBhbGVuY2lhICA8amVzdXMucGFsZW5jaWFAb3BlbmJvc3NhLm9yZz4KKworICAg
ICAgICBbUXRdW1dLMl0gSW1wbGVtZW50IFBhZ2VDbGllbnQ6OmlzVmlld1dpbmRvd0FjdGl2ZSgp
CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04MTE0Mwor
CisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFRoaXMgcGF0
Y2ggaW1wbGVtZW50cyBQYWdlQ2xpZW50Ojppc1ZpZXdXaW5kb3dBY3RpdmUoKQorICAgICAgICBu
b3cgdGhhdCBRUXVpY2tDYW52YXM6OmlzQWN0aXZlKCkgaXMgYXZhaWxhYmxlIChmcm9tIFFXaW5k
b3cpLgorCisgICAgICAgICogVUlQcm9jZXNzL0FQSS9xdC9xcXVpY2t3ZWJ2aWV3LmNwcDoKKyAg
ICAgICAgKFFRdWlja1dlYlZpZXc6OmZvY3VzSW5FdmVudCk6CisgICAgICAgIChRUXVpY2tXZWJW
aWV3Ojpmb2N1c091dEV2ZW50KToKKyAgICAgICAgKiBVSVByb2Nlc3MvQVBJL3F0L3FxdWlja3dl
YnZpZXdfcF9wLmg6CisgICAgICAgIChRUXVpY2tXZWJWaWV3UHJpdmF0ZSk6CisgICAgICAgIChR
UXVpY2tXZWJWaWV3UHJpdmF0ZTo6aXNXaW5kb3dBY3RpdmUpOgorICAgICAgICAqIFVJUHJvY2Vz
cy9xdC9RdFBhZ2VDbGllbnQuY3BwOgorICAgICAgICAoUXRQYWdlQ2xpZW50Ojppc1ZpZXdXaW5k
b3dBY3RpdmUpOgorCiAyMDEyLTAzLTE0ICBBbmRlcnMgQ2FybHNzb24gIDxhbmRlcnNjYUBhcHBs
ZS5jb20+CiAKICAgICAgICAgRml4IFVJIHByb2Nlc3MgY3Jhc2ggd2hlbiBhIHBsdWctaW4gcHJv
Y2VzcyBjcmFzaGVzIHdpdGggYSBtb2RhbCBkaWFsb2cgc2hvd2luZwpkaWZmIC0tZ2l0IGEvU291
cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9xdC9xcXVpY2t3ZWJ2aWV3LmNwcCBiL1NvdXJjZS9X
ZWJLaXQyL1VJUHJvY2Vzcy9BUEkvcXQvcXF1aWNrd2Vidmlldy5jcHAKaW5kZXggZWUxYWJmNDhj
ZGM4YzBmZWQ4MjYzODAyZmNlODdiZjdiNTZmY2U1Ni4uYzU3ZGVkZWVkMDVhZWYxZWE1YjYxZTkz
MmEwZDc2YzMxYzc2M2ZlOSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQ
SS9xdC9xcXVpY2t3ZWJ2aWV3LmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJ
L3F0L3FxdWlja3dlYnZpZXcuY3BwCkBAIC0xMzYyLDEyICsxMzYyLDIwIEBAIHZvaWQgUVF1aWNr
V2ViVmlldzo6aW5wdXRNZXRob2RFdmVudChRSW5wdXRNZXRob2RFdmVudCogZXZlbnQpCiB2b2lk
IFFRdWlja1dlYlZpZXc6OmZvY3VzSW5FdmVudChRRm9jdXNFdmVudCogZXZlbnQpCiB7CiAgICAg
UV9EKFFRdWlja1dlYlZpZXcpOworICAgIC8vIEZJWE1FOiB3ZSBzaG91bGRuJ3QgbmVlZCB0byBr
ZWVwIGQtPmlzVmlld1dpbmRvd0FjdGl2ZQorICAgIC8vIGFuZCBjb3VsZCAgYmUgdXNpbmcgY2Fu
dmFzKCktPmlzQWN0aXZlKCkgaW5zdGVhZAorICAgIC8vIGJ1dCBhIGJ1ZyBpbiBRdCBpcyBwcmV2
ZW50aW5nIHVzIGZyb20gZG9pbmcgdGhhdC4KKyAgICBkLT5pc1ZpZXdXaW5kb3dBY3RpdmUgPSBj
YW52YXMoKSAmJiBjYW52YXMoKS0+aXNBY3RpdmUoKTsKICAgICBkLT5wYWdlVmlldy0+ZXZlbnRI
YW5kbGVyKCktPmhhbmRsZUZvY3VzSW5FdmVudChldmVudCk7CiB9CiAKIHZvaWQgUVF1aWNrV2Vi
Vmlldzo6Zm9jdXNPdXRFdmVudChRRm9jdXNFdmVudCogZXZlbnQpCiB7CiAgICAgUV9EKFFRdWlj
a1dlYlZpZXcpOworICAgIC8vIEZJWE1FOiB3ZSBzaG91bGRuJ3QgbmVlZCB0byBrZWVwIGQtPmlz
Vmlld1dpbmRvd0FjdGl2ZQorICAgIC8vIGFuZCBjb3VsZCAgYmUgdXNpbmcgY2FudmFzKCktPmlz
QWN0aXZlKCkgaW5zdGVhZAorICAgIC8vIGJ1dCBhIGJ1ZyBpbiBRdCBpcyBwcmV2ZW50aW5nIHVz
IGZyb20gZG9pbmcgdGhhdC4KKyAgICBkLT5pc1ZpZXdXaW5kb3dBY3RpdmUgPSBmYWxzZTsKICAg
ICBkLT5wYWdlVmlldy0+ZXZlbnRIYW5kbGVyKCktPmhhbmRsZUZvY3VzT3V0RXZlbnQoZXZlbnQp
OwogfQogCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL3F0L3FxdWlj
a3dlYnZpZXdfcF9wLmggYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL3F0L3FxdWlja3dl
YnZpZXdfcF9wLmgKaW5kZXggMzlhMTRhN2E2NzViYzVkMzY3YzIyNmIzZTQyZGE5OTk0NGVkYjA1
Mi4uOGRhMDliNGQ0MmU5ZjdmYmE4MjcxMDhlOTE2Yjg5ZWE1MmE3MDE0MyAxMDA2NDQKLS0tIGEv
U291cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9xdC9xcXVpY2t3ZWJ2aWV3X3BfcC5oCisrKyBi
L1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvcXQvcXF1aWNrd2Vidmlld19wX3AuaApAQCAt
MTMzLDYgKzEzMywxMSBAQCBwdWJsaWM6CiAgICAgUGFzc093blB0cjxEcmF3aW5nQXJlYVByb3h5
PiBjcmVhdGVEcmF3aW5nQXJlYVByb3h5KCk7CiAgICAgdm9pZCBoYW5kbGVEb3dubG9hZFJlcXVl
c3QoRG93bmxvYWRQcm94eSopOwogCisgICAgLy8gRklYTUU6IHdlIHNob3VsZG4ndCBiZSBuZWVk
aW5nIHRvIGtlZXAgaXNWaWV3V2luZG93QWN0aXZlCisgICAgLy8gYW5kIGNvdWxkICBiZSB1c2lu
ZyBjYW52YXMoKS0+aXNBY3RpdmUoKSBkaXJlY3RseSBpbnN0ZWFkCisgICAgLy8gYnV0IGEgYnVn
IGluIFF0IGlzIHByZXZlbnRpbmcgdXMgZnJvbSBkb2luZyB0aGF0LgorICAgIGJvb2wgaXNXaW5k
b3dBY3RpdmUoKSBjb25zdCB7IHJldHVybiBpc1ZpZXdXaW5kb3dBY3RpdmU7IH0KKwogcHJvdGVj
dGVkOgogICAgIFFRdWlja1dlYlZpZXdQcml2YXRlKFFRdWlja1dlYlZpZXcqIHZpZXdwb3J0KTsK
ICAgICBSZWZQdHI8UXRXZWJDb250ZXh0PiBjb250ZXh0OwpAQCAtMTY5LDYgKzE3NCw3IEBAIHBy
b3RlY3RlZDoKICAgICBib29sIG1fcmVuZGVyVG9PZmZzY3JlZW5CdWZmZXI7CiAgICAgYm9vbCBt
X2xvYWRTdGFydGVkU2lnbmFsU2VudDsKICAgICBib29sIG1fZGlhbG9nUnVubmVyQWN0aXZlOwor
ICAgIGJvb2wgaXNWaWV3V2luZG93QWN0aXZlOwogICAgIFFVcmwgbV9pY29uVVJMOwogICAgIFFV
cmwgbV9kZWZlcmVkVXJsVG9Mb2FkOwogfTsKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL1VJ
UHJvY2Vzcy9xdC9RdFBhZ2VDbGllbnQuY3BwIGIvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL3F0
L1F0UGFnZUNsaWVudC5jcHAKaW5kZXggZDcwNDljYjNiNjIwODU5YWY5MzZhZTRmNmI2ZThjNGYw
MTA1N2VhZS4uOGU0YmE1OThlYTBkYjk4NDFkZDFlNGQ1ZWEzNmI4YmE4NjAzYzdjYiAxMDA2NDQK
LS0tIGEvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL3F0L1F0UGFnZUNsaWVudC5jcHAKKysrIGIv
U291cmNlL1dlYktpdDIvVUlQcm9jZXNzL3F0L1F0UGFnZUNsaWVudC5jcHAKQEAgLTI1NSw4ICsy
NTUsOSBAQCBXZWJDb3JlOjpJbnRTaXplIFF0UGFnZUNsaWVudDo6dmlld1NpemUoKQogCiBib29s
IFF0UGFnZUNsaWVudDo6aXNWaWV3V2luZG93QWN0aXZlKCkKIHsKLSAgICAvLyBGSVhNRTogVGhl
IHNjZW5lIGdyYXBoIGRvZXMgbm90IGhhdmUgdGhlIGNvbmNlcHQgb2YgYmVpbmcgYWN0aXZlIG9y
IG5vdCB3aGVuIHRoaXMgd2FzIHdyaXR0ZW4uCi0gICAgcmV0dXJuIHRydWU7CisgICAgLy8gRklY
TUU6IHdlIHNob3VsZCByZWFkIGl0IGRpcmVjdGx5IGZyb20gbV93ZWJWaWV3LT5jYW52YXMoKS0+
aXNBY3RpdmUoKQorICAgIC8vIGJ1dCBhIGJ1ZyBpbiBRdCBpcyBwcmV2ZW50aW5nIHVzIGZyb20g
dGhhdC4KKyAgICByZXR1cm4gUVF1aWNrV2ViVmlld1ByaXZhdGU6OmdldChtX3dlYlZpZXcpLT5p
c1dpbmRvd0FjdGl2ZSgpOwogfQogCiBib29sIFF0UGFnZUNsaWVudDo6aXNWaWV3Rm9jdXNlZCgp
Cg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>132081</attachid>
            <date>2012-03-15 11:18:15 -0700</date>
            <delta_ts>2012-03-20 14:53:04 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-81143-20120315151739.patch</filename>
            <type>text/plain</type>
            <size>1708</size>
            <attacher name="Jesus Sanchez-Palencia">jesus</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTEwNzM4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggMjczNDJiZmUzYWFlYzgz
OWYzOTQwZDUxMDAzY2UyNmEzNzczOGZmOC4uMjhkYTNiY2Q4MmRjMmI5MTViNjNkNjhjOWNlZGJm
ZGQzZjNlYzAwOCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDEyLTAzLTE1ICBKZXN1
cyBTYW5jaGV6LVBhbGVuY2lhICA8amVzdXMucGFsZW5jaWFAb3BlbmJvc3NhLm9yZz4KKworICAg
ICAgICBbUXRdW1dLMl0gSW1wbGVtZW50IFBhZ2VDbGllbnQ6OmlzVmlld1dpbmRvd0FjdGl2ZSgp
CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04MTE0Mwor
CisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFRoaXMgcGF0
Y2ggaW1wbGVtZW50cyBQYWdlQ2xpZW50Ojppc1ZpZXdXaW5kb3dBY3RpdmUoKQorICAgICAgICBu
b3cgdGhhdCBRUXVpY2tDYW52YXM6OmlzQWN0aXZlKCkgaXMgYXZhaWxhYmxlIChmcm9tIFFXaW5k
b3cpLgorCisgICAgICAgICogVUlQcm9jZXNzL3F0L1F0UGFnZUNsaWVudC5jcHA6CisgICAgICAg
IChRdFBhZ2VDbGllbnQ6OmlzVmlld1dpbmRvd0FjdGl2ZSk6CisKIDIwMTItMDMtMTQgIEFuZGVy
cyBDYXJsc3NvbiAgPGFuZGVyc2NhQGFwcGxlLmNvbT4KIAogICAgICAgICBGaXggVUkgcHJvY2Vz
cyBjcmFzaCB3aGVuIGEgcGx1Zy1pbiBwcm9jZXNzIGNyYXNoZXMgd2l0aCBhIG1vZGFsIGRpYWxv
ZyBzaG93aW5nCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvcXQvUXRQYWdl
Q2xpZW50LmNwcCBiL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9xdC9RdFBhZ2VDbGllbnQuY3Bw
CmluZGV4IGQ3MDQ5Y2IzYjYyMDg1OWFmOTM2YWU0ZjZiNmU4YzRmMDEwNTdlYWUuLmJmYzdlZjQy
YWM0YmExNWQ3ZDRhZDE3Y2VkYzhiNTY5MmMyODY5MDggMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJL
aXQyL1VJUHJvY2Vzcy9xdC9RdFBhZ2VDbGllbnQuY3BwCisrKyBiL1NvdXJjZS9XZWJLaXQyL1VJ
UHJvY2Vzcy9xdC9RdFBhZ2VDbGllbnQuY3BwCkBAIC0yOSw2ICsyOSw3IEBACiAjaW5jbHVkZSAi
cXF1aWNrd2Vidmlld19wLmgiCiAjaW5jbHVkZSAicXF1aWNrd2Vidmlld19wX3AuaCIKICNpbmNs
dWRlIDxRR3VpQXBwbGljYXRpb24+CisjaW5jbHVkZSA8UVF1aWNrQ2FudmFzPgogI2luY2x1ZGUg
PFdlYkNvcmUvQ3Vyc29yLmg+CiAjaW5jbHVkZSA8V2ViQ29yZS9EcmFnRGF0YS5oPgogI2luY2x1
ZGUgPFdlYkNvcmUvRmxvYXRSZWN0Lmg+CkBAIC0yNTUsOCArMjU2LDcgQEAgV2ViQ29yZTo6SW50
U2l6ZSBRdFBhZ2VDbGllbnQ6OnZpZXdTaXplKCkKIAogYm9vbCBRdFBhZ2VDbGllbnQ6OmlzVmll
d1dpbmRvd0FjdGl2ZSgpCiB7Ci0gICAgLy8gRklYTUU6IFRoZSBzY2VuZSBncmFwaCBkb2VzIG5v
dCBoYXZlIHRoZSBjb25jZXB0IG9mIGJlaW5nIGFjdGl2ZSBvciBub3Qgd2hlbiB0aGlzIHdhcyB3
cml0dGVuLgotICAgIHJldHVybiB0cnVlOworICAgIHJldHVybiBtX3dlYlZpZXctPmNhbnZhcygp
LT5pc0FjdGl2ZSgpOwogfQogCiBib29sIFF0UGFnZUNsaWVudDo6aXNWaWV3Rm9jdXNlZCgpCg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>132903</attachid>
            <date>2012-03-20 14:53:12 -0700</date>
            <delta_ts>2012-03-23 16:02:58 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-81143-20120320185238.patch</filename>
            <type>text/plain</type>
            <size>1751</size>
            <attacher name="Jesus Sanchez-Palencia">jesus</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTExMjM3CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggZGM4NjdmMjFlZjJhMjA1
MGM4ZTM1ZTQ4NWJlNzM0NzA4MDQ3YjMyOC4uMGY5OWJlOTJmZDQ5YmE4ZDY4MWQ4NDVkMzFmNjc1
NjBlODM5MDYxYyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDEyLTAzLTIwICBKZXN1
cyBTYW5jaGV6LVBhbGVuY2lhICA8amVzdXMucGFsZW5jaWFAb3BlbmJvc3NhLm9yZz4KKworICAg
ICAgICBbUXRdW1dLMl0gSW1wbGVtZW50IFBhZ2VDbGllbnQ6OmlzVmlld1dpbmRvd0FjdGl2ZSgp
CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04MTE0Mwor
CisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFRoaXMgcGF0
Y2ggaW1wbGVtZW50cyBQYWdlQ2xpZW50Ojppc1ZpZXdXaW5kb3dBY3RpdmUoKQorICAgICAgICBu
b3cgdGhhdCBRUXVpY2tDYW52YXM6OmlzQWN0aXZlKCkgaXMgYXZhaWxhYmxlIChmcm9tIFFXaW5k
b3cpLgorCisgICAgICAgICogVUlQcm9jZXNzL3F0L1F0UGFnZUNsaWVudC5jcHA6CisgICAgICAg
IChRdFBhZ2VDbGllbnQ6OmlzVmlld1dpbmRvd0FjdGl2ZSk6CisKIDIwMTItMDMtMTQgIEplc3Vz
IFNhbmNoZXotUGFsZW5jaWEgIDxqZXN1cy5wYWxlbmNpYUBvcGVuYm9zc2Eub3JnPgogCiAgICAg
ICAgIFtXSzJdIEFkZCBQYWdlIFZpc2liaWxpdHkgQVBJIHN1cHBvcnQKZGlmZiAtLWdpdCBhL1Nv
dXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9xdC9RdFBhZ2VDbGllbnQuY3BwIGIvU291cmNlL1dlYktp
dDIvVUlQcm9jZXNzL3F0L1F0UGFnZUNsaWVudC5jcHAKaW5kZXggZDcwNDljYjNiNjIwODU5YWY5
MzZhZTRmNmI2ZThjNGYwMTA1N2VhZS4uYWYwNzk3YjkwOWI4MzIzNmNmNzk2MTFmYmViZTZhM2Fl
NGY0NzQyYiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL3F0L1F0UGFnZUNs
aWVudC5jcHAKKysrIGIvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL3F0L1F0UGFnZUNsaWVudC5j
cHAKQEAgLTI5LDYgKzI5LDcgQEAKICNpbmNsdWRlICJxcXVpY2t3ZWJ2aWV3X3AuaCIKICNpbmNs
dWRlICJxcXVpY2t3ZWJ2aWV3X3BfcC5oIgogI2luY2x1ZGUgPFFHdWlBcHBsaWNhdGlvbj4KKyNp
bmNsdWRlIDxRUXVpY2tDYW52YXM+CiAjaW5jbHVkZSA8V2ViQ29yZS9DdXJzb3IuaD4KICNpbmNs
dWRlIDxXZWJDb3JlL0RyYWdEYXRhLmg+CiAjaW5jbHVkZSA8V2ViQ29yZS9GbG9hdFJlY3QuaD4K
QEAgLTI1NSw4ICsyNTYsOSBAQCBXZWJDb3JlOjpJbnRTaXplIFF0UGFnZUNsaWVudDo6dmlld1Np
emUoKQogCiBib29sIFF0UGFnZUNsaWVudDo6aXNWaWV3V2luZG93QWN0aXZlKCkKIHsKLSAgICAv
LyBGSVhNRTogVGhlIHNjZW5lIGdyYXBoIGRvZXMgbm90IGhhdmUgdGhlIGNvbmNlcHQgb2YgYmVp
bmcgYWN0aXZlIG9yIG5vdCB3aGVuIHRoaXMgd2FzIHdyaXR0ZW4uCi0gICAgcmV0dXJuIHRydWU7
CisgICAgaWYgKCFtX3dlYlZpZXcgfHwgIW1fd2ViVmlldy0+Y2FudmFzKCkpCisgICAgICAgIHJl
dHVybiBmYWxzZTsKKyAgICByZXR1cm4gbV93ZWJWaWV3LT5jYW52YXMoKS0+aXNBY3RpdmUoKTsK
IH0KIAogYm9vbCBRdFBhZ2VDbGllbnQ6OmlzVmlld0ZvY3VzZWQoKQo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>133579</attachid>
            <date>2012-03-23 16:03:05 -0700</date>
            <delta_ts>2012-04-11 08:54:15 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-81143-20120323200232.patch</filename>
            <type>text/plain</type>
            <size>3471</size>
            <attacher name="Jesus Sanchez-Palencia">jesus</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTExODU5CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggNGFmNDJjMGQ3NjZiOGFl
MTNiMmZkZTZkNGFhNmNjODhkMmJiMTIwNi4uMDg1MDRjYzFmNzg0ZTFhMWM4NDQwOTBiOWQxNmMw
NmExZWU4YzI3ZCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDEyLTAzLTIzICBKZXN1
cyBTYW5jaGV6LVBhbGVuY2lhICA8amVzdXMucGFsZW5jaWFAb3BlbmJvc3NhLm9yZz4KKworICAg
ICAgICBbUXRdW1dLMl0gSW1wbGVtZW50IFBhZ2VDbGllbnQ6OmlzVmlld1dpbmRvd0FjdGl2ZSgp
CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04MTE0Mwor
CisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFRoaXMgcGF0
Y2ggaW1wbGVtZW50cyBQYWdlQ2xpZW50Ojppc1ZpZXdXaW5kb3dBY3RpdmUoKQorICAgICAgICBu
b3cgdGhhdCBRUXVpY2tDYW52YXM6OmlzQWN0aXZlKCkgaXMgYXZhaWxhYmxlIChmcm9tIFFXaW5k
b3cpLgorCisgICAgICAgICogVUlQcm9jZXNzL3F0L1F0UGFnZUNsaWVudC5jcHA6CisgICAgICAg
IChRdFBhZ2VDbGllbnQ6OmlzVmlld1dpbmRvd0FjdGl2ZSk6CisKIDIwMTItMDMtMjMgIFNoZXJp
ZmYgQm90ICA8d2Via2l0LnJldmlldy5ib3RAZ21haWwuY29tPgogCiAgICAgICAgIFVucmV2aWV3
ZWQsIHJvbGxpbmcgb3V0IHIxMTE4NTUuCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9VSVBy
b2Nlc3MvcXQvUXRQYWdlQ2xpZW50LmNwcCBiL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9xdC9R
dFBhZ2VDbGllbnQuY3BwCmluZGV4IGQ3MDQ5Y2IzYjYyMDg1OWFmOTM2YWU0ZjZiNmU4YzRmMDEw
NTdlYWUuLmFmMDc5N2I5MDliODMyMzZjZjc5NjExZmJlYmU2YTNhZTRmNDc0MmIgMTAwNjQ0Ci0t
LSBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9xdC9RdFBhZ2VDbGllbnQuY3BwCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9xdC9RdFBhZ2VDbGllbnQuY3BwCkBAIC0yOSw2ICsyOSw3
IEBACiAjaW5jbHVkZSAicXF1aWNrd2Vidmlld19wLmgiCiAjaW5jbHVkZSAicXF1aWNrd2Vidmll
d19wX3AuaCIKICNpbmNsdWRlIDxRR3VpQXBwbGljYXRpb24+CisjaW5jbHVkZSA8UVF1aWNrQ2Fu
dmFzPgogI2luY2x1ZGUgPFdlYkNvcmUvQ3Vyc29yLmg+CiAjaW5jbHVkZSA8V2ViQ29yZS9EcmFn
RGF0YS5oPgogI2luY2x1ZGUgPFdlYkNvcmUvRmxvYXRSZWN0Lmg+CkBAIC0yNTUsOCArMjU2LDkg
QEAgV2ViQ29yZTo6SW50U2l6ZSBRdFBhZ2VDbGllbnQ6OnZpZXdTaXplKCkKIAogYm9vbCBRdFBh
Z2VDbGllbnQ6OmlzVmlld1dpbmRvd0FjdGl2ZSgpCiB7Ci0gICAgLy8gRklYTUU6IFRoZSBzY2Vu
ZSBncmFwaCBkb2VzIG5vdCBoYXZlIHRoZSBjb25jZXB0IG9mIGJlaW5nIGFjdGl2ZSBvciBub3Qg
d2hlbiB0aGlzIHdhcyB3cml0dGVuLgotICAgIHJldHVybiB0cnVlOworICAgIGlmICghbV93ZWJW
aWV3IHx8ICFtX3dlYlZpZXctPmNhbnZhcygpKQorICAgICAgICByZXR1cm4gZmFsc2U7CisgICAg
cmV0dXJuIG1fd2ViVmlldy0+Y2FudmFzKCktPmlzQWN0aXZlKCk7CiB9CiAKIGJvb2wgUXRQYWdl
Q2xpZW50Ojppc1ZpZXdGb2N1c2VkKCkKZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rv
b2xzL0NoYW5nZUxvZwppbmRleCBjOGI4OTc0ZTgwYmYyNjAwZWQyZDFmMmVhYjVjZDIzNTdlNDQ5
ZjkwLi44Njk4M2NlZDU5Yzc3NjBlYWZkM2E1ZmYwNWM1NTVjZjI0OTNiMjUzIDEwMDY0NAotLS0g
YS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTggQEAK
KzIwMTItMDMtMjMgIEplc3VzIFNhbmNoZXotUGFsZW5jaWEgIDxqZXN1cy5wYWxlbmNpYUBvcGVu
Ym9zc2Eub3JnPgorCisgICAgICAgIFtRdF1bV0syXSBJbXBsZW1lbnQgUGFnZUNsaWVudDo6aXNW
aWV3V2luZG93QWN0aXZlKCkKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTgxMTQzCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK
KyAgICAgICAgRml4IHRoZSBXcmFwcGVyV2luZG93IGZyb20gb3VyIFBsYXRmb3JtV2ViVmlldwor
ICAgICAgICBzbyBpdCBjb3JyZWN0bHkgY3JlYXRlcyB0aGUgcGxhdGZvcm0gcmVsYXRlZAorICAg
ICAgICBzdHVmZiBmb3IgUVdpbmRvdywgd2hpY2ggaXMgbm90IGNyZWF0ZWQgdW5sZXNzCisgICAg
ICAgIFFXaW5kb3c6OnNldFZpc2libGUoKSBvciBRV2luZG93OjpzaG93KCkgYXJlIGNhbGxlZC4K
KworICAgICAgICAqIFdlYktpdFRlc3RSdW5uZXIvcXQvUGxhdGZvcm1XZWJWaWV3UXQuY3BwOgor
ICAgICAgICAoV1RSOjpQbGF0Zm9ybVdlYlZpZXc6OlBsYXRmb3JtV2ViVmlldyk6CisKIDIwMTIt
MDMtMjMgIFBldGVyIEJldmVybG9vICA8cGV0ZXJAY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFVu
cmV2aWV3ZWQgYnVpbGRmaXggZm9yIHRoZSBDaHJvbWl1bSBBbmRyb2lkIGJ1aWxkZXIuCmRpZmYg
LS1naXQgYS9Ub29scy9XZWJLaXRUZXN0UnVubmVyL3F0L1BsYXRmb3JtV2ViVmlld1F0LmNwcCBi
L1Rvb2xzL1dlYktpdFRlc3RSdW5uZXIvcXQvUGxhdGZvcm1XZWJWaWV3UXQuY3BwCmluZGV4IDdm
NTEyMzk5OTJhYjg4M2RjZGE3ZjA0MDFmODQ1MzhlMWZjNDNiZjYuLmYwZmE2NDA4OTljZjEzOTE4
MjNkMDNkYWVhNWY5YjI5ZDcyMmU3OTYgMTAwNjQ0Ci0tLSBhL1Rvb2xzL1dlYktpdFRlc3RSdW5u
ZXIvcXQvUGxhdGZvcm1XZWJWaWV3UXQuY3BwCisrKyBiL1Rvb2xzL1dlYktpdFRlc3RSdW5uZXIv
cXQvUGxhdGZvcm1XZWJWaWV3UXQuY3BwCkBAIC03Niw2ICs3NiwxMiBAQCBQbGF0Zm9ybVdlYlZp
ZXc6OlBsYXRmb3JtV2ViVmlldyhXS0NvbnRleHRSZWYgY29udGV4dFJlZiwgV0tQYWdlR3JvdXBS
ZWYgcGFnZUdybwogewogICAgIFFRdWlja1dlYlZpZXdFeHBlcmltZW50YWwgZXhwZXJpbWVudGFs
KG1fdmlldyk7CiAgICAgZXhwZXJpbWVudGFsLnNldFJlbmRlclRvT2Zmc2NyZWVuQnVmZmVyKHRy
dWUpOworCisgICAgLy8gUVdpbmRvdyBkb2VzIG5vdCBjcmVhdGUgdGhlIHVuZGVybHlpbmcgcGxh
dGZvcm0KKyAgICAvLyBzcGVjaWZpYyBiaXRzIHVubGVzcyBRV2luZG93OjpzZXRWaXNpYmxlKCkg
b3IgUVdpbmRvdzo6c2hvdygpIGFyZSBjYWxsZWQsCisgICAgLy8gYW5kIHdpdGhvdXQgaXQgUVdp
bmRvdzo6aXNBY3RpdmUoKSB3aWxsIGFsd2F5cyByZXR1cm4gZmFsc2UuCisgICAgLy8gRm9yIHRo
ZXNlIGNhc2VzIFFXaW5kb3c6OmNyZWF0ZSgpIGlzIHByb3ZpZGVkLgorICAgIG1fd2luZG93LT5j
cmVhdGUoKTsKIH0KIAogUGxhdGZvcm1XZWJWaWV3Ojp+UGxhdGZvcm1XZWJWaWV3KCkK
</data>

          </attachment>
      

    </bug>

</bugzilla>