Closed Bug 1371346 Opened 3 years ago Closed 2 years ago

Remove Presentation API system add-on


(Firefox :: General, enhancement, P3)




Firefox 61
Tracking Status
firefox61 --- fixed


(Reporter: ehsan, Assigned: Usul)


(Blocks 1 open bug)


(Whiteboard: [fxperf:p2])


(1 file)

browser/base/content/test/performance/browser_startup.js indicates that the following scripts are loaded before opening the first window:

Whiteboard: [fxperf]
Priority: -- → P3
Mike, I see you reviewed this file when it was added. Do you know if this is dead code at this point?
It's only built-in if not 'RELEASE_OR_BETA':
Flags: needinfo?(mconley)
Good question.

Hey overholt, is the Presentation API still something we plan on working on in the nearer term? Or can we remove this code for now?
Flags: needinfo?(mconley) → needinfo?(overholt)
No plans in the near term AFAIK.
Flags: needinfo?(overholt)
Okay, thanks!

Let's get this out of the tree for now. We can revive the dead code should we decide to pursue the API in the future.
Summary: presentation add-on shouldn't load anything before opening and painting the first browser window → Remove Presentation API system add-on
Whiteboard: [fxperf] → [fxperf:p2]
Blocks: 1449052
Attached patch 1371346.patchSplinter Review
Attachment #8967785 - Flags: review?(mconley)
Comment on attachment 8967785 [details] [diff] [review]

Review of attachment 8967785 [details] [diff] [review]:


Can you confirm that the tests that are under dom/presentation/tests still pass / don't rely on this stuff?
Attachment #8967785 - Flags: review?(mconley) → review+
Can someone push to try for me?
Pushed by
Remove unused/dead Presentation API code. r=mconley
Try looks good, but my patch only stopped including the presentation directory and didn't actually remove it.

However, there's really no difference from the tests point of view, I think. I think we're good to land this.

Pushed to inbound as:
Backed right out again. Apparently there's some locale stuff that we're missing that didn't burn try, amazingly.
Pushed by
Remove unused/dead Presentation API code. r=mconley
Assignee: nobody → ludovic
Flags: needinfo?(mconley)
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
You need to log in before you can comment on or make changes to this bug.