Closed Bug 996914 Opened 9 years ago Closed 8 years ago

Breakdown: [New Tab Page] about:newtab should be unprivileged

Categories

(Firefox :: New Tab Page, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: MarcoM, Assigned: Gijs)

References

Details

(Whiteboard: p=1 s=it-32c-31a-30b.3 [qa-])

+++ This bug was initially created as a clone of Bug #776477 +++
Flags: firefox-backlog+
Mass-move to Firefox::New Tab Page.

Filter on new-tab-page-component.
Component: Tabbed Browser → New Tab Page
Tim, what exactly is necessary here? I would have figured:

1) switch page to use HTML instead of XUL (which is liable to still be a 13-point bug, but I suspect will be equally tricky to really split up more than that?). The RTL issue can be partially solved by setting the dir attribute on the root to the relevant entity value (IIRC this is what some of our other html about: pages do) - although obviously we might still need more CSS or other adjustments to make stuff work correctly.
2) update all the tests that we have
3) toggle the relevant bits for about:newtab in the about redirector

If this sounds right to you, I can file/repurpose all the relevant bugs and we can close this one...
Flags: needinfo?(ttaubert)
Another somewhat difficult part is rewriting all the logic in NewTabUtils.jsm to work with Events like the new search input does. The whole page isn't e10s ready at all and thus currently requires chrome privs to load pinned sites etc. Another 13-point bug here...
Flags: needinfo?(ttaubert)
Marco, can you add this to this iteration's backlog?
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Flags: needinfo?(mmucci)
Whiteboard: p=0 → p=1 [qa-]
Filed bug 1021653, bug 1021654, bug 1021667.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Added to Iteration 32.3
Flags: needinfo?(mmucci)
Whiteboard: p=1 [qa-] → p=1 s=it-32c-31a-30b.3 [qa-]
You need to log in before you can comment on or make changes to this bug.