<?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>55469</bug_id>
          
          <creation_ts>2011-03-01 07:46:45 -0800</creation_ts>
          <short_desc>REGRESSION (r79863): Lots of dom/html/level2/html/HTMLFrameElement*.html tests crashing in FrameView::paintOverhangAreas in WebKit2</short_desc>
          <delta_ts>2011-03-03 09:57:38 -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>Tools / Tests</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>http://build.webkit.org/results/Windows%207%20Release%20(WebKit2%20Tests)/r79994%20(3567)/CrashLog_0b08_2011-03-01_06-13-18-438.txt</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar, LayoutTestFailure, PlatformOnly, Regression</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Adam Roben (:aroben)">aroben</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bdakin</cc>
    
    <cc>sam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>359890</commentid>
    <comment_count>0</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-01 07:46:45 -0800</bug_when>
    <thetext>Lots of dom/html/level2/html/HTMLFrameElement*.html tests are crashing in FrameView::paintOverhangAreas in WebKit2 on Windows. See the URL for a crash log. You can find more crash logs here:

http://build.webkit.org/results/Windows%207%20Release%20(WebKit2%20Tests)/r79994%20(3567)/

(Note that these crashes are being reported as hangs due to bug 44121.)

I&apos;m not sure when the crashes started. Buildbot says around r79865, but that doesn&apos;t really make sense to me.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>359892</commentid>
    <comment_count>1</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-01 07:47:14 -0800</bug_when>
    <thetext>&lt;rdar://problem/9068539&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>359932</commentid>
    <comment_count>2</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-01 09:16:35 -0800</bug_when>
    <thetext>Looks like Mac is crashing, too: http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(WebKit2%20Tests)/r80001%20(9140)/results.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>360315</commentid>
    <comment_count>3</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-01 15:38:59 -0800</bug_when>
    <thetext>Alexey says this started happening over the weekend.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>360954</commentid>
    <comment_count>4</comment_count>
    <who name="Sam Weinig">sam</who>
    <bug_when>2011-03-02 12:01:00 -0800</bug_when>
    <thetext>It seems like r79863, is the likely cause of this, specifically the change in InjectedBundlePage.cpp (http://trac.webkit.org/changeset/79863/trunk/Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp).  It seems that we are calling out to WebKit in an inconsistent state, so that when we go to paint a FrameView, it is no longer in the tree. We could add a null check for the page, but I think that would just be papering over the issue.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>360962</commentid>
    <comment_count>5</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-02 12:04:33 -0800</bug_when>
    <thetext>I wonder why WebKit1 doesn&apos;t have this problem? I placed the WKBundlePageForceRepaint call in the same place we call -[WebView displayIfNeeded] in DRT on Mac. (DRT on Windows does the equivalent of this in its dump() function; we could try moving the call to the equivalent place in WTR.)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>360966</commentid>
    <comment_count>6</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-02 12:07:18 -0800</bug_when>
    <thetext>My bad for not noticing I caused this!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>360969</commentid>
    <comment_count>7</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-02 12:10:29 -0800</bug_when>
    <thetext>(In reply to comment #5)
&gt; I wonder why WebKit1 doesn&apos;t have this problem? I placed the WKBundlePageForceRepaint call in the same place we call -[WebView displayIfNeeded] in DRT on Mac.

I guess it&apos;s possible -displayIfNeeded was bailing because no display is needed. But *something* is causing the tests always to paint in DRT.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>361005</commentid>
    <comment_count>8</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-02 13:00:32 -0800</bug_when>
    <thetext>Here&apos;s a backtrace:

 	WebKit.dll!WTF::RefPtr&lt;WebCore::Frame&gt;::get()  Line 60 + 0x11 bytes	C++
 	WebKit.dll!WebCore::Page::mainFrame()  Line 135 + 0x19 bytes	C++
 	WebKit.dll!WebCore::FrameView::paintOverhangAreas(WebCore::GraphicsContext * context=0x0249bb68, const WebCore::IntRect &amp; horizontalOverhangArea={...}, const WebCore::IntRect &amp; verticalOverhangArea={...}, const WebCore::IntRect &amp; dirtyRect={...})  Line 2361 + 0x12 bytes	C++
 	WebKit.dll!WebCore::ScrollView::paint(WebCore::GraphicsContext * context=0x0249bb68, const WebCore::IntRect &amp; rect={...})  Line 943 + 0x22 bytes	C++
 	WebKit.dll!WebCore::RenderWidget::paint(WebCore::PaintInfo &amp; paintInfo={...}, int tx=0, int ty=0)  Line 299 + 0x30 bytes	C++
 	WebKit.dll!WebCore::RenderFrameSet::paint(WebCore::PaintInfo &amp; paintInfo={...}, int tx=0, int ty=0)  Line 142 + 0x1e bytes	C++
 	WebKit.dll!WebCore::RenderFrameSet::paint(WebCore::PaintInfo &amp; paintInfo={...}, int tx=0, int ty=0)  Line 142 + 0x1e bytes	C++
 	WebKit.dll!WebCore::RenderBlock::paintChildren(WebCore::PaintInfo &amp; paintInfo={...}, int tx=0, int ty=0)  Line 2367 + 0x28 bytes	C++
 	WebKit.dll!WebCore::RenderBlock::paintContents(WebCore::PaintInfo &amp; paintInfo={...}, int tx=0, int ty=0)  Line 2327	C++
 	WebKit.dll!WebCore::RenderBlock::paintObject(WebCore::PaintInfo &amp; paintInfo={...}, int tx=0, int ty=0)  Line 2439	C++
 	WebKit.dll!WebCore::RenderBlock::paint(WebCore::PaintInfo &amp; paintInfo={...}, int tx=0, int ty=0)  Line 2214 + 0x1e bytes	C++
 	WebKit.dll!WebCore::RenderLayer::paintLayer(WebCore::RenderLayer * rootLayer=0x0247dabc, WebCore::GraphicsContext * p=0x0249bb68, const WebCore::IntRect &amp; paintDirtyRect={...}, unsigned int paintBehavior=0, WebCore::RenderObject * paintingRoot=0x00000000, WTF::HashMap&lt;WebCore::OverlapTestRequestClient *,WebCore::IntRect,WTF::PtrHash&lt;WebCore::OverlapTestRequestClient *&gt;,WTF::HashTraits&lt;WebCore::OverlapTestRequestClient *&gt;,WTF::HashTraits&lt;WebCore::IntRect&gt; &gt; * overlapTestRequests=0x0012e658, unsigned int paintFlags=0)  Line 2509 + 0x3b bytes	C++
 	WebKit.dll!WebCore::RenderLayer::paintList(WTF::Vector&lt;WebCore::RenderLayer *,0&gt; * list=[1](0x025309dc {m_renderer=0x024e604c m_parent=0x0247dabc m_previous=0x00000000 ...}), WebCore::RenderLayer * rootLayer=0x0247dabc, WebCore::GraphicsContext * p=0x0249bb68, const WebCore::IntRect &amp; paintDirtyRect={...}, unsigned int paintBehavior=0, WebCore::RenderObject * paintingRoot=0x00000000, WTF::HashMap&lt;WebCore::OverlapTestRequestClient *,WebCore::IntRect,WTF::PtrHash&lt;WebCore::OverlapTestRequestClient *&gt;,WTF::HashTraits&lt;WebCore::OverlapTestRequestClient *&gt;,WTF::HashTraits&lt;WebCore::IntRect&gt; &gt; * overlapTestRequests=0x0012e658, unsigned int paintFlags=0)  Line 2563	C++
 	WebKit.dll!WebCore::RenderLayer::paintLayer(WebCore::RenderLayer * rootLayer=0x0247dabc, WebCore::GraphicsContext * p=0x0249bb68, const WebCore::IntRect &amp; paintDirtyRect={...}, unsigned int paintBehavior=0, WebCore::RenderObject * paintingRoot=0x00000000, WTF::HashMap&lt;WebCore::OverlapTestRequestClient *,WebCore::IntRect,WTF::PtrHash&lt;WebCore::OverlapTestRequestClient *&gt;,WTF::HashTraits&lt;WebCore::OverlapTestRequestClient *&gt;,WTF::HashTraits&lt;WebCore::IntRect&gt; &gt; * overlapTestRequests=0x0012e658, unsigned int paintFlags=0)  Line 2532	C++
 	WebKit.dll!WebCore::RenderLayer::paint(WebCore::GraphicsContext * p=0x0249bb68, const WebCore::IntRect &amp; damageRect={...}, unsigned int paintBehavior=0, WebCore::RenderObject * paintingRoot=0x00000000)  Line 2315	C++
 	WebKit.dll!WebCore::FrameView::paintContents(WebCore::GraphicsContext * p=0x0249bb68, const WebCore::IntRect &amp; rect={...})  Line 2312	C++
 	WebKit.dll!WebCore::ScrollView::paint(WebCore::GraphicsContext * context=0x0249bb68, const WebCore::IntRect &amp; rect={...})  Line 934 + 0x1a bytes	C++
 	WebKit.dll!WebKit::WebPage::drawRect(WebCore::GraphicsContext &amp; graphicsContext={...}, const WebCore::IntRect &amp; rect={...})  Line 588 + 0x33 bytes	C++
 	WebKit.dll!WebKit::DrawingAreaImpl::display(WebKit::UpdateInfo &amp; updateInfo={...})  Line 480	C++
 	WebKit.dll!WebKit::DrawingAreaImpl::display()  Line 403	C++
 	WebKit.dll!WebKit::DrawingAreaImpl::forceRepaint()  Line 159	C++
 	WebKit.dll!WebKit::WebPage::forceRepaintWithoutCallback()  Line 1286 + 0x1d bytes	C++
 	WebKit.dll!WKBundlePageForceRepaint(const OpaqueWKBundlePage * page=0x024806e8)  Line 219	C++
 	InjectedBundle.dll!WTR::InjectedBundlePage::didFinishLoadForFrame(const OpaqueWKBundleFrame * frame=0x02483f38)  Line 537 + 0xe bytes	C++
 	InjectedBundle.dll!WTR::InjectedBundlePage::didFinishLoadForFrame(const OpaqueWKBundlePage * page=0x024806e8, const OpaqueWKBundleFrame * frame=0x02483f38, const void * * __formal=0x0012e99c, const void * clientInfo=0x024c8b90)  Line 288	C++
 	WebKit.dll!WebKit::InjectedBundlePageLoaderClient::didFinishLoadForFrame(WebKit::WebPage * page=0x024806e8, WebKit::WebFrame * frame=0x02483f38, WTF::RefPtr&lt;WebKit::APIObject&gt; &amp; userData=0x00000000)  Line 95 + 0x2f bytes	C++
 	WebKit.dll!WebKit::WebFrameLoaderClient::dispatchDidFinishLoad()  Line 518	C++
 	WebKit.dll!WebCore::FrameLoader::checkLoadCompleteForThisFrame()  Line 2427 + 0x18 bytes	C++
 	WebKit.dll!WebCore::FrameLoader::recursiveCheckLoadComplete()  Line 2539	C++
 	WebKit.dll!WebCore::FrameLoader::checkLoadComplete()  Line 2552	C++
 	WebKit.dll!WebCore::FrameLoader::mainReceivedCompleteError(WebCore::DocumentLoader * loader=0x025542f0, const WebCore::ResourceError &amp; __formal={...})  Line 3302	C++
 	WebKit.dll!WebCore::DocumentLoader::mainReceivedError(const WebCore::ResourceError &amp; error={...}, bool isComplete=true)  Line 206	C++
 	WebKit.dll!WebCore::FrameLoader::receivedMainResourceError(const WebCore::ResourceError &amp; error={...}, bool isComplete=true)  Line 2837	C++
 	WebKit.dll!WebCore::MainResourceLoader::didCancel(const WebCore::ResourceError &amp; error={...})  Line 111	C++
 	WebKit.dll!WebCore::ResourceLoader::cancel(const WebCore::ResourceError &amp; error={...})  Line 381 + 0x1f bytes	C++
 	WebKit.dll!WebCore::ResourceLoader::cancel()  Line 371 + 0x18 bytes	C++
 	WebKit.dll!WebCore::DocumentLoader::stopLoading()  Line 248 + 0x12 bytes	C++
 	WebKit.dll!WebCore::FrameLoader::stopAllLoaders(WebCore::ClearProvisionalItemPolicy clearProvisionalItemPolicy=ShouldClearProvisionalItem)  Line 1715	C++
 	WebKit.dll!WebCore::FrameLoader::frameDetached()  Line 2587	C++
&gt;	WebKit.dll!WebCore::HTMLFrameOwnerElement::willRemove()  Line 59	C++
 	WebKit.dll!WebCore::HTMLFrameElementBase::willRemove()  Line 284	C++
 	WebKit.dll!WebCore::ContainerNode::willRemove()  Line 363 + 0x24 bytes	C++
 	WebKit.dll!WebCore::ContainerNode::willRemove()  Line 363 + 0x24 bytes	C++
 	WebKit.dll!WebCore::willRemoveChildren(WebCore::ContainerNode * container=0x0251a008)  Line 391 + 0x12 bytes	C++
 	WebKit.dll!WebCore::ContainerNode::removeChildren()  Line 510 + 0xe bytes	C++
 	WebKit.dll!WebCore::Document::implicitOpen()  Line 1963	C++
 	WebKit.dll!WebCore::Document::open(WebCore::Document * ownerDocument=0x0251a008)  Line 1926	C++
 	WebKit.dll!WebCore::JSHTMLDocument::open(JSC::ExecState * exec=0x03a00170)  Line 129	C++
 	WebKit.dll!WebCore::jsHTMLDocumentPrototypeFunctionOpen(JSC::ExecState * exec=0x03a00170)  Line 402 + 0x10 bytes	C++
 	040a006a()	
 	JavaScriptCore.dll!cti_vm_lazyLinkCall()  Line 2022 + 0x1c bytes	C++
 	JavaScriptCore.dll!JSC::Interpreter::executeCall(JSC::ExecState * callFrame=0x0252c3f8, JSC::JSObject * function=0x04031d98, JSC::CallType callType=CallTypeJS, const JSC::CallData &amp; callData={...}, JSC::JSValue thisValue={...}, const JSC::ArgList &amp; args={...})  Line 844 + 0x2a bytes	C++
 	JavaScriptCore.dll!JSC::call(JSC::ExecState * exec=0x0252c3f8, JSC::JSValue functionObject={...}, JSC::CallType callType=CallTypeJS, const JSC::CallData &amp; callData={...}, JSC::JSValue thisValue={...}, const JSC::ArgList &amp; args={...})  Line 38 + 0x3c bytes	C++
 	WebKit.dll!WebCore::JSMainThreadExecState::call(JSC::ExecState * exec=0x0252c3f8, JSC::JSValue functionObject={...}, JSC::CallType callType=CallTypeJS, const JSC::CallData &amp; callData={...}, JSC::JSValue thisValue={...}, const JSC::ArgList &amp; args={...})  Line 48 + 0x29 bytes	C++
 	WebKit.dll!WebCore::JSEventListener::handleEvent(WebCore::ScriptExecutionContext * scriptExecutionContext=, WebCore::Event * event=)  Line 123 + 0x6a bytes	C++
 	WebKit.dll!WebCore::EventTarget::fireEventListeners(WebCore::Event * event=0x0251f5b8, WebCore::EventTargetData * d=0x024ce5e8, WTF::Vector&lt;WebCore::RegisteredEventListener,1&gt; &amp; entry=[1549312]({listener={...} useCapture=??? },{listener={...} useCapture=??? },{listener={...} useCapture=??? },{listener={...} useCapture=??? },{listener={...} useCapture=??? },{listener={...} useCapture=??? },{listener={...} useCapture=??? },{listener={...} useCapture=??? },{listener={...} useCapture=??? },{listener={...} useCapture=??? },{listener={...} useCapture=??? },{listener={...} useCapture=,...))  Line 354 + 0x35 bytes	C++
 	WebKit.dll!WebCore::EventTarget::fireEventListeners(WebCore::Event * event=0x0251f5b8)  Line 325	C++
 	WebKit.dll!WebCore::DOMWindow::dispatchEvent(WTF::PassRefPtr&lt;WebCore::Event&gt; prpEvent={...}, WTF::PassRefPtr&lt;WebCore::EventTarget&gt; prpTarget={...})  Line 1585 + 0x11 bytes	C++
 	WebKit.dll!WebCore::DOMWindow::dispatchTimedEvent(WTF::PassRefPtr&lt;WebCore::Event&gt; event={...}, WebCore::Document * target=0x0251a008, double * startTime=0x02492578, double * endTime=0x02492580)  Line 1598	C++
 	WebKit.dll!WebCore::DOMWindow::dispatchLoadEvent()  Line 1558	C++
 	WebKit.dll!WebCore::Document::dispatchWindowLoadEvent()  Line 3505	C++
 	WebKit.dll!WebCore::Document::implicitClose()  Line 2091	C++
 	WebKit.dll!WebCore::FrameLoader::checkCallImplicitClose()  Line 892	C++
 	WebKit.dll!WebCore::FrameLoader::checkCompleted()  Line 841	C++
 	WebKit.dll!WebCore::FrameLoader::completed()  Line 1204	C++
 	WebKit.dll!WebCore::FrameLoader::checkCompleted()  Line 844	C++
 	WebKit.dll!WebCore::FrameLoader::finishedParsing()  Line 775	C++
 	WebKit.dll!WebCore::Document::finishedParsing()  Line 4272	C++
 	WebKit.dll!WebCore::ImageDocumentParser::finish()  Line 171 + 0x1d bytes	C++
 	WebKit.dll!WebCore::Document::finishParsing()  Line 2257 + 0x20 bytes	C++
 	WebKit.dll!WebCore::DocumentWriter::endIfNotLoadingMainResource()  Line 223	C++
 	WebKit.dll!WebCore::DocumentWriter::end()  Line 208	C++
 	WebKit.dll!WebCore::DocumentLoader::finishedLoading()  Line 286	C++
 	WebKit.dll!WebCore::FrameLoader::finishedLoading()  Line 2189	C++
 	WebKit.dll!WebCore::MainResourceLoader::didFinishLoading(double finishTime=0.00000000000000000)  Line 466	C++
 	WebKit.dll!WebCore::ResourceLoader::didFinishLoading(WebCore::ResourceHandle * __formal=0x025288c8, double finishTime=0.00000000000000000)  Line 436 + 0x18 bytes	C++
 	WebKit.dll!WebCore::didFinishLoading(_CFURLConnection * conn=0x02556650, const void * clientInfo=0x025288c8)  Line 241 + 0x26 bytes	C++</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>361007</commentid>
    <comment_count>9</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-02 13:01:17 -0800</bug_when>
    <thetext>We&apos;re crashing on this line:

    if (page-&gt;mainFrame() == m_frame) {


page is 0.

m_frame is not the same frame that is being detached.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>361046</commentid>
    <comment_count>10</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-02 13:46:27 -0800</bug_when>
    <thetext>It looks like FrameView::paintContents, which is also called at this time, does not require the Page to be non-null (at least not directly).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>361084</commentid>
    <comment_count>11</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-02 14:40:06 -0800</bug_when>
    <thetext>I can reproduce the crash in WebKit1 on Windows by adding code to force a display at the equivalent point in DRT. (As stated above, DRT on Windows normally waits until dump() is called to force a display.)

I&apos;d bet we can get the crash to go away in WebKit2 by waiting to force the repaint until dump() is called. We&apos;d still want to fix the underlying issue here, though.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>361092</commentid>
    <comment_count>12</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-02 14:50:20 -0800</bug_when>
    <thetext>(In reply to comment #11)
&gt; I&apos;d bet we can get the crash to go away in WebKit2 by waiting to force the repaint until dump() is called.

That does indeed fix the crash.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>361101</commentid>
    <comment_count>13</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-02 15:03:25 -0800</bug_when>
    <thetext>I discovered a few more things while looking at what happens in WebKit1 on Mac:

In some tests, e.g., css1/basic/class_as_selector.html, the -displayIfNeeded call in -webView:didFinishLoadForFrame: does indeed cause a display. In others, e.g., dom/html/level2/html/HTMLFrameElement01.html, the -displayIfNeeded call *does not* cause a display, and in fact we never display at all in that test in WebKit1.

So it looks like DRT on Mac and DRT on Windows have an important difference: Windows always displays, while Mac only sometimes displays.

The change I made to WTR made it always display, like DRT on Windows, but put the call in the place where Mac DRT has it. I think the best thing to do is to move it to dump() to match DRT on Windows.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>361110</commentid>
    <comment_count>14</comment_count>
      <attachid>84475</attachid>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-02 15:10:31 -0800</bug_when>
    <thetext>Created attachment 84475
Wait for force a paint in WebKitTestRunner until we&apos;re actually ready to dump the output</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>361112</commentid>
    <comment_count>15</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-02 15:11:03 -0800</bug_when>
    <thetext>We&apos;re going to track the underlying issue separately. Beth volunteered to file that bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>361117</commentid>
    <comment_count>16</comment_count>
    <who name="Beth Dakin">bdakin</who>
    <bug_when>2011-03-02 15:17:26 -0800</bug_when>
    <thetext>(In reply to comment #15)
&gt; We&apos;re going to track the underlying issue separately. Beth volunteered to file that bug.

I filed: https://bugs.webkit.org/show_bug.cgi?id=55624</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>361118</commentid>
    <comment_count>17</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-03-02 15:19:51 -0800</bug_when>
    <thetext>Committed r80175: &lt;http://trac.webkit.org/changeset/80175&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>84475</attachid>
            <date>2011-03-02 15:10:31 -0800</date>
            <delta_ts>2011-03-02 15:12:16 -0800</delta_ts>
            <desc>Wait for force a paint in WebKitTestRunner until we&apos;re actually ready to dump the output</desc>
            <filename>bug-55469-20110302181036.patch</filename>
            <type>text/plain</type>
            <size>2324</size>
            <attacher name="Adam Roben (:aroben)">aroben</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogODAxNTgKZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBi
L1Rvb2xzL0NoYW5nZUxvZwppbmRleCBkN2ZjYzNjYjZhNGVmMmEwNzhkYTk2ZWNiNThmZDJiYWNh
N2FkZjJkLi4wOWVkMDcxZDFmMGM2ZGE1ZDVjOTdmZWU1ODMxY2IxYjJiOWE2NjRiIDEwMDY0NAot
LS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMjEg
QEAKIDIwMTEtMDMtMDIgIEFkYW0gUm9iZW4gIDxhcm9iZW5AYXBwbGUuY29tPgogCisgICAgICAg
IFdhaXQgZm9yIGZvcmNlIGEgcGFpbnQgaW4gV2ViS2l0VGVzdFJ1bm5lciB1bnRpbCB3ZSdyZSBh
Y3R1YWxseSByZWFkeSB0byBkdW1wIHRoZSBvdXRwdXQKKworICAgICAgICBUaGlzIG1hdGNoZXMg
RHVtcFJlbmRlclRyZWUgb24gV2luZG93cy4KKworICAgICAgICBGaXhlcyA8aHR0cDovL3dlYmtp
dC5vcmcvYi81NTQ2OT4gPHJkYXI6Ly9wcm9ibGVtLzkwNjg1Mzk+IFJFR1JFU1NJT04gKHI3OTg2
Myk6IExvdHMgb2YKKyAgICAgICAgZG9tL2h0bWwvbGV2ZWwyL2h0bWwvSFRNTEZyYW1lRWxlbWVu
dCouaHRtbCB0ZXN0cyBjcmFzaGluZyBpbiBGcmFtZVZpZXc6OnBhaW50T3ZlcmhhbmdBcmVhcwor
ICAgICAgICBpbiBXZWJLaXQyCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgKiBXZWJLaXRUZXN0UnVubmVyL0luamVjdGVkQnVuZGxlL0luamVjdGVkQnVu
ZGxlUGFnZS5jcHA6CisgICAgICAgIChXVFI6OkluamVjdGVkQnVuZGxlUGFnZTo6ZHVtcCk6IE1v
dmVkIGNvZGUgdG8gZm9yY2UgYSBwYWludCBoZXJlLi4uCisgICAgICAgIChXVFI6OkluamVjdGVk
QnVuZGxlUGFnZTo6ZGlkRmluaXNoTG9hZEZvckZyYW1lKTogLi4uZnJvbSBoZXJlLgorCisyMDEx
LTAzLTAyICBBZGFtIFJvYmVuICA8YXJvYmVuQGFwcGxlLmNvbT4KKwogICAgICAgICBEb24ndCB0
cnkgdG8gY2FwdHVyZSBjcmFzaCBsb2dzIG9uIHBsYXRmb3JtcyB0aGF0IGhhdmVuJ3QgaW1wbGVt
ZW50ZWQgdGhhdCBmZWF0dXJlCiAKICAgICAgICAgRml4ZXMgPGh0dHA6Ly93ZWJraXQub3JnL2Iv
NTU1MDQ+IFVzZSBvZiB1bmluaXRpYWxpemVkIHZhbHVlIGluIG51bWVyaWMgbHQgKDwpIGF0CmRp
ZmYgLS1naXQgYS9Ub29scy9XZWJLaXRUZXN0UnVubmVyL0luamVjdGVkQnVuZGxlL0luamVjdGVk
QnVuZGxlUGFnZS5jcHAgYi9Ub29scy9XZWJLaXRUZXN0UnVubmVyL0luamVjdGVkQnVuZGxlL0lu
amVjdGVkQnVuZGxlUGFnZS5jcHAKaW5kZXggNjFlNzc2MTYyNGQ4MjgwYjEyYjFiNWMyYzk5NDUw
NWQ1NmJiNDQ4ZS4uMzVlMWE1ZjZkZWIyN2Y1YmZkYmVhYzU0NjAzNjI5MzcyNTU3YThmOCAxMDA2
NDQKLS0tIGEvVG9vbHMvV2ViS2l0VGVzdFJ1bm5lci9JbmplY3RlZEJ1bmRsZS9JbmplY3RlZEJ1
bmRsZVBhZ2UuY3BwCisrKyBiL1Rvb2xzL1dlYktpdFRlc3RSdW5uZXIvSW5qZWN0ZWRCdW5kbGUv
SW5qZWN0ZWRCdW5kbGVQYWdlLmNwcApAQCAtNDk1LDYgKzQ5NSwxMCBAQCB2b2lkIEluamVjdGVk
QnVuZGxlUGFnZTo6ZHVtcCgpCiAKICAgICBJbmplY3RlZEJ1bmRsZTo6c2hhcmVkKCkubGF5b3V0
VGVzdENvbnRyb2xsZXIoKS0+aW52YWxpZGF0ZVdhaXRUb0R1bXBXYXRjaGRvZ1RpbWVyKCk7CiAK
KyAgICAvLyBGb3JjZSBhIHBhaW50IGJlZm9yZSBkdW1waW5nLiBUaGlzIG1hdGNoZXMgRHVtcFJl
bmRlclRyZWUgb24gV2luZG93cy4gU2VlCisgICAgLy8gPGh0dHA6Ly93ZWJraXQub3JnL2IvNTU0
Njk+IGZvciBkZXRhaWxzLgorICAgIFdLQnVuZGxlUGFnZUZvcmNlUmVwYWludChtX3BhZ2UpOwor
CiAgICAgV0tCdW5kbGVGcmFtZVJlZiBmcmFtZSA9IFdLQnVuZGxlUGFnZUdldE1haW5GcmFtZSht
X3BhZ2UpOwogICAgIHN0cmluZyB1cmwgPSB0b1NURChhZG9wdFdLKFdLVVJMQ29weVN0cmluZyhh
ZG9wdFdLKFdLQnVuZGxlRnJhbWVDb3B5VVJMKGZyYW1lKSkuZ2V0KCkpKSk7CiAgICAgaWYgKHN0
cnN0cih1cmwuY19zdHIoKSwgImR1bXBBc1RleHQvIikpCkBAIC01MzQsOCArNTM4LDYgQEAgdm9p
ZCBJbmplY3RlZEJ1bmRsZVBhZ2U6OmRpZEZpbmlzaExvYWRGb3JGcmFtZShXS0J1bmRsZUZyYW1l
UmVmIGZyYW1lKQogICAgICAgICByZXR1cm47CiAgICAgSW5qZWN0ZWRCdW5kbGU6OnNoYXJlZCgp
LnNldFRvcExvYWRpbmdGcmFtZSgwKTsKIAotICAgIFdLQnVuZGxlUGFnZUZvcmNlUmVwYWludCht
X3BhZ2UpOwotCiAgICAgaWYgKEluamVjdGVkQnVuZGxlOjpzaGFyZWQoKS5sYXlvdXRUZXN0Q29u
dHJvbGxlcigpLT53YWl0VG9EdW1wKCkpCiAgICAgICAgIHJldHVybjsKIAo=
</data>
<flag name="review"
          id="76454"
          type_id="1"
          status="+"
          setter="bdakin"
    />
          </attachment>
      

    </bug>

</bugzilla>