Closed Bug 1510699 Opened 6 years ago Closed 5 years ago

[wpt-sync] Sync PR 14282 - Revert "Align resource timing buffer full processing to spec PR 168 (reland)"

Categories

(Core :: DOM: Core & HTML, enhancement, P4)

Product:
Core
Component:
DOM: Core & HTML
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla66
Tracking Flags:
Tracking Status
firefox66 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream]) 

Sync web-platform-tests PR 14282 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/14282
Details from upstream follow.

Ned Nguyen <nednguyen@google.com> wrote:
>  Revert "Align resource timing buffer full processing to spec PR 168 (reland)"
>  
>  This reverts commit 6dd060768d1c1183e0d29e422a2213f106d98299.
>  
>  Reason for revert: suspect causing  WebRtcDisableEncryptionFlagBrowserTest.VerifyEncryption to be flaky (FindIt's report)
>  
>  BUG:chromium:909704
>  
>  Original change's description:
>  > Align resource timing buffer full processing to spec PR 168 (reland)
>  > 
>  > This change implements the processing model from PR 168[1], when
>  > it comes to setResourceTimingBufferSize(), clearResourceTimings()
>  > and the firing of the resourcetimingbufferfull event.
>  > 
>  > This is a reland of
>  > https://chromium-review.googlesource.com/c/chromium/src/+/1345269
>  > (but with nicer tests).
>  > 
>  > [1] https://github.com/w3c/resource-timing/pull/168
>  > 
>  > Change-Id: I10431bfda7f79b484c7ee5c608bb8a360d905339
>  > Bug: 908181, 908414
>  > Reviewed-on: https://chromium-review.googlesource.com/c/1350950
>  > Reviewed-by: Nicolás Peña Moreno \<npm@chromium.org>
>  > Commit-Queue: Yoav Weiss \<yoavweiss@chromium.org>
>  > Cr-Commit-Position: refs/heads/master@{#611174}
>  
>  TBR=npm@chromium.org,yoavweiss@chromium.org
>  
>  Change-Id: Ia4bf26c6be1e7c4ae35b904bac25fe031f854604
>  No-Presubmit: true
>  No-Tree-Checks: true
>  No-Try: true
>  Bug: 908181, 908414
>  Reviewed-on: https://chromium-review.googlesource.com/c/1354123
>  Reviewed-by: Ned Nguyen \<nednguyen@google.com>
>  Commit-Queue: Ned Nguyen \<nednguyen@google.com>
>  Cr-Commit-Position: refs/heads/master@{#611722}
>
PR 14282 applied with additional changes from upstream: 06720fac129ee8916f1a9309ebc0bfa134604bfe, 40e50ecff750feff8e62dc98a72e9bca3a8c94cf, cccb52680a601f4de432124092c808e91fad8a15
Component: web-platform-tests → DOM
Product: Testing → Core
Ran 3 tests and 14 subtests
OK     : 3
PASS   : 5
FAIL   : 9
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4b81c4c2722a
[wpt PR 14282] - Revert "Align resource timing buffer full processing to spec PR 168 (reland)", a=testonly
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7cf604bd53de
[wpt PR 14282] - Revert "Align resource timing buffer full processing to spec PR 168 (reland)", a=testonly
https://hg.mozilla.org/mozilla-central/rev/7cf604bd53de
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox66: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.