Closed
Bug 1059662
Opened 10 years ago
Closed 10 years ago
embed-apps should be restricted to in-proc apps or we need to fix oop-embed-apps
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla36
People
(Reporter: kanru, Assigned: kanru)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
|
(deleted),
patch
|
fabrice
:
review+
|
Details | Diff | Splinter Review |
In bug 1044333 we realized that apps managed by an OOP app would use the incorrect AppId, the parent app's AppId, so it would cause privilege escalation.
In the short term we should disallow apps with embed-apps permission to be run OOP.
|
Assignee | |
Updated•10 years ago
|
Assignee: nobody → kchen
|
|
Assignee | |
Comment 1•10 years ago
|
||
https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=b445bb36bc51
This might break some emulator tests..
Attachment #8513262 -
Flags: review?(fabrice)
|
||
Comment 2•10 years ago
|
||
Comment on attachment 8513262 [details] [diff] [review]
Disallow OOP app to embed in-proc apps
Review of attachment 8513262 [details] [diff] [review]:
-----------------------------------------------------------------
Please ask review once treeherder is happy!
Attachment #8513262 -
Flags: review?(fabrice)
|
|
Assignee | |
Comment 3•10 years ago
|
||
https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=a74ef8f2086e
Tree is greenish.. failing tests are disabled since they use embed-apps in oop env.
Attachment #8513262 -
Attachment is obsolete: true
Attachment #8517381 -
Flags: review?(fabrice)
|
|
||
Updated•10 years ago
|
Attachment #8517381 -
Flags: review?(fabrice) → review+
|
|
Assignee | |
Comment 4•10 years ago
|
||
|
||
Comment 5•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
|
||
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
Yeah you are right, Few app like Grammarly which we need for running https://apkblink.com/ is not working in the browsers. Although the app works in other browsers. As contact to their support team they answered me to reinstall you browser. I did the same but nothing happened facing same issue.
You need to log in
before you can comment on or make changes to this bug.
Description
•