New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 787246 link

Starred by 2 users
Status: Assigned
Owner:
█ mgiuca@chromium.org
Long OOO (go/where-is-mgiuca)
Cc:
█ mgiuca@chromium.org
joelhockey@chromium.org
sa...@chromium.org
esprehn@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task

Blocked on:
issue 704441


Sign in to add a comment

Onion soup content/renderer/installedapp/

Project Member Reported by slangley@chromium.org, Nov 21 2017 
The code inside of content/renderer/installedapp/ needs to inside of blink as part of the onion soup project (modules/installedapp/?).

Comment 1 by mgiuca@chromium.org, Nov 21 2017

Cc: sa...@chromium.org
Components: Blink>AppManifest
Design doc for this work:

https://docs.google.com/document/d/1v-CCUll7iyvgxn4JMWJ_8sG1QNds_KVBgUnxJEDJ0dc/edit

Current architecture:
https://docs.google.com/drawings/d/1D9zDpCXe3GWVYO2XS4w0mIDY21CHbGhjQVOOJtcWWQw/edit

At the time, this was implemented with as much code in Blink as possible, but RelatedAppsFetcher could not be put into Blink because it depended on the Manifest (which was in Chrome).

This seems to still be blocked on onion-souping ManifestManager (Issue 704441), which I see is already listed as a blocker.

Comment 2 by slangley@chromium.org, Jan 9 2018

Cc: mgiuca@chromium.org
Owner: joelhockey@chromium.org

Comment 3 by mgiuca@chromium.org, Jan 9 2018 

Note that we may be removing this code when the origin trial ends at the end of January. This is still up in the air. It may not be worth refactoring it now.

Comment 4 by joelhockey@chromium.org, Jan 9 2018

Cc: joelhockey@chromium.org
Owner: mgiuca@chromium.org
I will leave this work for now and put mgiuca as owner.
►

Sign in to add a comment