Closed Bug 1661299 Opened 4 years ago Closed 4 years ago

Don't try to read web app manifest from iframes

Categories

(GeckoView :: General, defect, P3)

Unspecified
All

Tracking

(firefox82 fixed)

RESOLVED FIXED
82 Branch
Tracking Status
firefox82 --- fixed

People

(Reporter: agi, Assigned: agi)

Details

Attachments

(1 file)

Just noticed we don't check if the document is top level when we try to get the web app manifest https://searchfox.org/mozilla-central/rev/27932d4e6ebd2f4b8519865dad864c72176e4e3b/mobile/android/actors/ContentDelegateChild.jsm#148

08-26 10:18:10.522 16610 16661 E Web Content: [JavaScript Error: "Error: Window must be a top-level browsing context." {file: "resource://gre/modules/ManifestObtainer.jsm" line: 151}]
Severity: -- → S3
Priority: -- → P3
Assignee: nobody → agi
Status: NEW → ASSIGNED
Pushed by asferro@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3cdf93208afd
Read web-app manifest on top-level frames only. r=esawin
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: