Remove unused resource://gre/modules/PresentationDeviceInfoManager.jsm

RESOLVED FIXED in Firefox 56

Status

()

enhancement
P3
normal
RESOLVED FIXED
2 years ago
5 months ago

People

(Reporter: florian, Assigned: dbugs)

Tracking

(Blocks 1 bug, {good-first-bug})

unspecified
mozilla56
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox56 fixed)

Details

Attachments

(1 attachment)

This was added in bug 1080474, and is only referenced at http://searchfox.org/mozilla-central/rev/7419b368156a6efa24777b21b0e5706be89a9c2f/b2g/chrome/content/shell.js#26

There may be a lot more things to remove from dom/presentation/, as it's not really clear to me if this code was ever used outside of b2g (dom.presentation.enabled is true for android, but it seems that was added as part of the same patch).
Maybe S.C. has something to comment on this usage.
Flags: needinfo?(schien)
PresentationDeviceInfoManager.jsm is something we can removed from the code base.
dom/presentation includes two API: Presentation API and Presentation Device Info API.
Presentation API is currently used in Fennec but Device Info API is B2G chrome-only API.

Here is the list of file that can be removed:
dom/webidl/PresentationDeviceInfoManager.webidl
dom/presentation/PresentationDeviceInfoManager.js
dom/presentation/PresentationDeviceInfoManager.jsm
dom/presentation/PresentationDeviceInfoManager.manifest
dom/presentation/tests/mochitest/test_presentation_device_info_permission.html
dom/presentation/tests/mochitest/test_presentation_device_info.html

Files that references to these files will also need corresponding modification.

@flo, what's the priority and impact of this bug?
Flags: needinfo?(schien) → needinfo?(florian)
(In reply to Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz) from comment #2)

> @flo, what's the priority and impact of this bug?

The impact is reducing the size of the Firefox builds we ship. The priority would be the same as other bugs blocking bug 1306391.
Flags: needinfo?(florian)
Keywords: good-first-bug
Priority: -- → P3
Assignee: nobody → dbugs
Attachment #8870457 - Flags: review?(bugs) → review?(schien)
Comment on attachment 8870457 [details]
Bug 1351089 - Remove unused PresentationDeviceInfoManager.jsm

https://reviewboard.mozilla.org/r/141896/#review145770

Thanks for working on this! However, this patch is still far away from r+.
Here is the list of things you should notice:
1. Only files I mentioned in comment #2 should be removed. Other files should be kept and fixed.
2. You'll need to make sure the rest of Presetation API tests are passed, and these test cases are writen in both e10s and non-e10s mochitest. When you do local test please make sure you run both `./mach mochitest dom/presentation` and `./mach mochitest --disable-e10s dom/presentation`.
Attachment #8870457 - Flags: review?(schien)
No longer blocks: 1369194
Comment on attachment 8870457 [details]
Bug 1351089 - Remove unused PresentationDeviceInfoManager.jsm

https://reviewboard.mozilla.org/r/141896/#review159762

Doing a quick DXR search and find some more files to modified.
- tools/lint/eslint/modules.json
- mobile/android/installer/package-manifest.in
- dom/tests/mochitest/general/test_interfaces.js
You'll need to add @smaug in the reviewer list since modifying test_interfaces.js requires a DOM peer review.
Attachment #8870457 - Flags: review?(schien)
Comment on attachment 8870457 [details]
Bug 1351089 - Remove unused PresentationDeviceInfoManager.jsm

https://reviewboard.mozilla.org/r/141896/#review160322

r+ for the .webidl and test_interfaces.js
Attachment #8870457 - Flags: review?(bugs) → review+
Comment on attachment 8870457 [details]
Bug 1351089 - Remove unused PresentationDeviceInfoManager.jsm

https://reviewboard.mozilla.org/r/141896/#review160442

Thanks!
Attachment #8870457 - Flags: review?(schien) → review+
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1f16321b2fff
Remove unused PresentationDeviceInfoManager.jsm r=schien,smaug
https://hg.mozilla.org/mozilla-central/rev/1f16321b2fff
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
This bug removed some of the compiler warnings. Thank you!

== Change summary for alert #7763 (as of July 08 2017 08:21 UTC) ==

Improvements:

  2%  compiler warnings summary linux64 pgo      1,351.50 -> 1,330.83

For up to date results, see: https://treeherder.mozilla.org/perf.html#/alerts?id=7763
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.