Closed Bug 1621734 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 22196 - Reveal only the URL at the frame open time in performance.measureMemory

Categories

(Testing :: web-platform-tests, task, P4)

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox76 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla76
Tracking Flags:
Tracking Status
firefox76 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Ulan Degenbaev <ulan@chromium.org> wrote:

Reveal only the URL at the frame open time in performance.measureMemory

This changes the reporting in performance.measureMemory to reveal only
the URL at the frame open time for cross-origin (same-site) iframes and
thus prevents URL if the iframe navigates after loading.

This is done by saving the first cross-origin URL loaded in LocalFrame
in LocalFrame::DidAttachDocument and using that URL for reporting.

The patch also adds a new test for redirecting iframe and fixes
the existing tests to wait until child iframes are loaded.

Bug:1049093
Change-Id: I0162efb82061f036450586dea0b191cacdce23f2

Reviewed-on: https://chromium-review.googlesource.com/2098722
WPT-Export-Revision: 26bc554e7b6a8afcaa079b8c80dab37ee3aa16e0

PR 22196 applied with additional changes from upstream: 63d868ddd979a22b1389ab3b46633ffed776de53, ebcb931b73c52f49bb17f1443ba54b21e90e0967
Test result changes from PR not available.
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b125b93d3bb2 [wpt PR 22196] - Reveal only the initial frame URL in performance.measureMemory, a=testonly https://hg.mozilla.org/integration/autoland/rev/1d669ca911c7 [wpt PR 22196] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/b125b93d3bb2
https://hg.mozilla.org/mozilla-central/rev/1d669ca911c7

Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox76: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla76
You need to log in before you can comment on or make changes to this bug.