Closed Bug 1288030 Opened 8 years ago Closed 7 years ago

Stop BackgroundPageThumbs relying on the hidden window loading chrome: URIs

Categories

(Firefox :: New Tab Page, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 1338215
Tracking Status
firefox50 --- affected

People

(Reporter: Gijs, Unassigned)

References

Details

Right now the background page thumb code:

1) gets the hidden window;
2) inserts an iframe into it that loads the chrome URI version of about:mozilla
3) inserts a XUL <browser type=content> into that iframe
4) loads pages into there in order to take screenshots and populate the thumbnail db.


When I spoke to Drew on IRC, it seems that when they tried to do this inside a browser window instead, the thumbnails were always completely black, because it wasn't possible to have the browser be display: none or similar and still get it to render.

At the moment this code blocks us from making the hidden window less special (bug 1145470) because it needs to link to this chrome:// URI. Apparently the thumbnailer needs the contents of the iframe to be system-principal'd ( https://dxr.mozilla.org/mozilla-central/rev/4c05938a64a7fde3ac2d7f4493aee1c5f2ad8a0a/toolkit/components/thumbnails/BackgroundPageThumbs.jsm#153-155 ) so I'm not entirely sure how to square that circle. It'd be great if other folks have ideas about how to do that...
Welp, looks like I fixed this in a dupe.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.