Closed
Bug 1057230
Opened 10 years ago
Closed 10 years ago
Follow up bug 1020157, bug 1040561: Inproc-app-remote-browser doesn't have correct mContainingAppId
Categories
(Core :: DOM: Content Processes, defect)
Core
DOM: Content Processes
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: kanru, Assigned: kershaw, NeedInfo)
References
Details
Attachments
(1 file)
13.25 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Reporter | ||
Comment 1•10 years ago
|
||
Assignee: nobody → kchen
Reporter | ||
Comment 2•10 years ago
|
||
Reporter | ||
Comment 3•10 years ago
|
||
Reporter | ||
Comment 4•10 years ago
|
||
Attachment #8477400 -
Flags: review?(khuey)
Attachment #8477400 -
Flags: review?(khuey) → review+
Reporter | ||
Updated•10 years ago
|
Reporter | ||
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
Assignee: kchen → kechang
Assignee | ||
Comment 5•10 years ago
|
||
Since ContentParent doesn't keep app id now, it seems we don't have to take care this bug anymore.
@kanru,
Could you confirm this?
Flags: needinfo?(kchen)
Reporter | ||
Comment 6•10 years ago
|
||
(In reply to Kershaw Chang [:kershaw] from comment #5)
> Since ContentParent doesn't keep app id now, it seems we don't have to take
> care this bug anymore.
>
> @kanru,
> Could you confirm this?
I think so. I still want to revisit some of the refactoring I made in this patch so keep the ni to myself.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•