Bug 37121 - REGRESSION(r57081): We think it triggered two tests to fail on Tiger because of race conditions (Requested by abarth on #webkit).
Summary: REGRESSION(r57081): We think it triggered two tests to fail on Tiger because ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 36864
  Show dependency treegraph
 
Reported: 2010-04-05 15:51 PDT by WebKit Review Bot
Modified: 2010-04-05 16:30 PDT (History)
5 users (show)

See Also:


Attachments
ROLLOUT of r57081 (10.33 KB, patch)
2010-04-05 15:51 PDT, WebKit Review Bot
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description WebKit Review Bot 2010-04-05 15:51:21 PDT
http://trac.webkit.org/changeset/57081 broke the build:
We think it triggered two tests to fail on Tiger because of race conditions (Requested by abarth on #webkit).

This is an automatic bug report generated by the sheriff-bot. If this bug
report was created because of a flaky test, please file a bug for the flaky
test (if we don't already have one on file) and dup this bug against that bug
so that we can track how often these flaky tests case pain.

"Only you can prevent forest fires." -- Smokey the Bear
Comment 1 WebKit Review Bot 2010-04-05 15:51:40 PDT
Created attachment 52584 [details]
ROLLOUT of r57081
Comment 2 Adam Barth 2010-04-05 16:07:39 PDT
Comment on attachment 52584 [details]
ROLLOUT of r57081

Clearing flags on attachment: 52584

Committed r57100: <http://trac.webkit.org/changeset/57100>
Comment 3 Adam Barth 2010-04-05 16:07:45 PDT
All reviewed patches have been landed.  Closing bug.
Comment 4 Eric Seidel (no email) 2010-04-05 16:10:10 PDT
This also affected Qt.  Ossy reproduced the failure locally and confirmed that the rollout caused the failure to go away.

I've not yet been able to reproduce the failure on Leopard.  I don't have access to a Tiger machine.
Comment 5 Eric Seidel (no email) 2010-04-05 16:11:35 PDT
Our understanding of this failure is that this regression made it possible for load events to fire on the outermost document, before the inner <object> elements had a chance to schedule their loads.

<object> load scheduling is crazy.  it happens in the renderers via  updateWidget which is scheduled from attach().
Comment 6 Eric Seidel (no email) 2010-04-05 16:30:07 PDT
Looks like this rollout fixed both the Qt bot and Tiger.