An embedded widget is valid if the |src| is in the |widgetPages|. The rule should always work after the app is installed.

Hello fabrice, I'm not sure how the marketplace generates the mini-manifest. If there's |widgetPages| in the main-manifest, will the |widgetPages| be cloned to the mini-manifest?

(In reply to Junior Hsu [:juniorhsu] from comment #1) > Hello fabrice, > I'm not sure how the marketplace generates the mini-manifest. > If there's |widgetPages| in the main-manifest, will the |widgetPages| be > cloned to the mini-manifest? No, there is no reason to add the widget pages to the mini manifest. How is this related to comment #0? Can you describes what the issue is?

The |widgetPages| is not in webapps.json file. Actually, the _saveWidgetsFullPage[1] is not called as installing a package app. My origin guess is owing to mini-manifest but it's wrong. I'm not sure where to save the widget information. Maybe start tracking from updateAppHandlers[2]? [1] http://dxr.mozilla.org/mozilla-central/source/dom/apps/Webapps.jsm#339 [2] http://dxr.mozilla.org/mozilla-central/source/dom/apps/Webapps.jsm#1817

Yep, we can add that to updateAppHandlers().

Hello Junior, Attached is a main manifest to reproduce the bug. 'widgetPages' entry in webapps.json is an empty array just after installing a package app.

Hello Junior, Attached is a mini manifest to reproduce the bug. ("xxx.xxx.xxx.xxx" is the IP of test server.)

Save widgetPages in UpdateAppHandlers and remove redundant saving

tbpl: https://tbpl.mozilla.org/?tree=Try&rev=44db856f9ff9

Treeherder with permission: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=44db856f9ff9 Treeherder without permission: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=781e193ac2ef

Sorry for missing metadata (facepalm)

Add metadata; carry r+.