Bug 148533 - REGRESSION (r188820): fast/dom/HTMLObjectElement/object-as-frame.html is flaky
Summary: REGRESSION (r188820): fast/dom/HTMLObjectElement/object-as-frame.html is flaky
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2015-08-27 11:17 PDT by Alexey Proskuryakov
Modified: 2015-08-31 01:34 PDT (History)
10 users (show)

See Also:


Attachments
patch (3.73 KB, patch)
2015-08-28 06:50 PDT, Antti Koivisto
buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ews106 for mac-mavericks-wk2 (396.96 KB, application/zip)
2015-08-28 07:02 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews102 for mac-mavericks (292.12 KB, application/zip)
2015-08-28 07:18 PDT, Build Bot
no flags Details
patch (3.91 KB, patch)
2015-08-28 07:34 PDT, Antti Koivisto
buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ews102 for mac-mavericks (559.76 KB, application/zip)
2015-08-28 08:06 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews104 for mac-mavericks-wk2 (600.87 KB, application/zip)
2015-08-28 08:11 PDT, Build Bot
no flags Details
patch (11.54 KB, patch)
2015-08-28 08:33 PDT, Antti Koivisto
cdumez: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 2015-08-27 11:17:13 PDT
fast/dom/HTMLObjectElement/object-as-frame.html become flaky after the patch for decoding data URLs in WebContent process.

This is hard to reproduce locally, but it happens frequently enough on bots 

I could sometimes reproduce on El Capitan by running tests like this:

run-webkit-tests fast/dom/HTMLObjectElement -f --repeat 100 --no-retry

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&showExpectations=true&revision=188926&tests=fast%2Fdom%2FHTMLObjectElement%2Fobject-as-frame.html

@@ -4,5 +4,5 @@
 Loading 'data:text/html,Updated contents of iframe' in iframe...
 Loading 'data:text/html,Updated contents of object' in object...
 PASS window.frames[0].location.toString() is "data:text/html,Updated contents of iframe"
-PASS window.frames[1].location.toString() is "data:text/html,Updated contents of object"
+FAIL window.frames[1].location.toString() should be data:text/html,Updated contents of object. Was data:text/html,New contents of frame.
Comment 1 Radar WebKit Bug Importer 2015-08-27 11:23:58 PDT
<rdar://problem/22459237>
Comment 2 Alexey Proskuryakov 2015-08-27 11:36:21 PDT
Marked as flaky in r189041.
Comment 3 Antti Koivisto 2015-08-28 04:53:58 PDT
This is an existing bug where <object> may cause document load event to be fire too early. Faster data URL decoding just changed timings and made it more likely here.
Comment 4 Antti Koivisto 2015-08-28 06:50:55 PDT
Created attachment 260147 [details]
patch
Comment 5 Build Bot 2015-08-28 07:02:44 PDT
Comment on attachment 260147 [details]
patch

Attachment 260147 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/116618

Number of test failures exceeded the failure limit.
Comment 6 Build Bot 2015-08-28 07:02:56 PDT
Created attachment 260148 [details]
Archive of layout-test-results from ews106 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5
Comment 7 Build Bot 2015-08-28 07:18:41 PDT
Comment on attachment 260147 [details]
patch

Attachment 260147 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/116648

Number of test failures exceeded the failure limit.
Comment 8 Build Bot 2015-08-28 07:18:47 PDT
Created attachment 260149 [details]
Archive of layout-test-results from ews102 for mac-mavericks

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-mavericks  Platform: Mac OS X 10.9.5
Comment 9 Antti Koivisto 2015-08-28 07:34:56 PDT
Created attachment 260150 [details]
patch
Comment 10 Build Bot 2015-08-28 08:06:26 PDT
Comment on attachment 260150 [details]
patch

Attachment 260150 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/116738

New failing tests:
webarchive/loading/object.html
http/tests/loading/basic-auth-resend-wrong-credentials.html
http/tests/loading/basic-credentials-sent-automatically.html
Comment 11 Build Bot 2015-08-28 08:06:42 PDT
Created attachment 260151 [details]
Archive of layout-test-results from ews102 for mac-mavericks

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-mavericks  Platform: Mac OS X 10.9.5
Comment 12 Build Bot 2015-08-28 08:11:24 PDT
Comment on attachment 260150 [details]
patch

Attachment 260150 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/116745

New failing tests:
webarchive/loading/object.html
http/tests/loading/basic-auth-resend-wrong-credentials.html
http/tests/loading/basic-credentials-sent-automatically.html
Comment 13 Build Bot 2015-08-28 08:11:34 PDT
Created attachment 260152 [details]
Archive of layout-test-results from ews104 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5
Comment 14 Antti Koivisto 2015-08-28 08:33:50 PDT
Created attachment 260153 [details]
patch
Comment 15 Chris Dumez 2015-08-28 09:15:44 PDT
Comment on attachment 260153 [details]
patch

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

r=me

> Source/WebCore/ChangeLog:8
> +        <object> element may cause document load even fire too early. By making data url

"event", "data URI".
Comment 16 Antti Koivisto 2015-08-28 09:27:03 PDT
"URL" is the correct term. "URI" should not be used, see https://url.spec.whatwg.org
Comment 17 Antti Koivisto 2015-08-31 01:34:41 PDT
https://trac.webkit.org/r189164