Move Webapps.js and Webapps.jsm out of dom/base

RESOLVED FIXED in mozilla15

Status

()

Core
DOM: Apps
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: fabrice, Assigned: fabrice)

Tracking

Trunk
mozilla15
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 628902 [details] [diff] [review]
patch

Cleaning after myself since they never should have been there.

No changes needed to the various installer/package-manifest.in
Attachment #628902 - Flags: review?(mounir)
OS: Linux → All
Hardware: x86_64 → All
Version: Other Branch → Trunk
Comment on attachment 628902 [details] [diff] [review]
patch

Review of attachment 628902 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with the following change.

::: dom/apps/src/Makefile.in
@@ +15,5 @@
> +  Webapps.js \
> +  Webapps.manifest \
> +  $(NULL)
> +
> +EXTRA_PP_JS_MODULES += Webapps.jsm \

Please move WebApps.jsm to a new line. It should look like this:
EXTRA_PP_JS_MODULES += \
  WebApps.jsm \
  $(NULL)
Attachment #628902 - Flags: review?(mounir) → review+
(Assignee)

Comment 2

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/f3a8184df76c
https://hg.mozilla.org/mozilla-central/rev/f3a8184df76c
Assignee: nobody → fabrice
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla15

Updated

5 years ago
Component: DOM: Mozilla Extensions → DOM: Apps
You need to log in before you can comment on or make changes to this bug.