Brian Grinstead [:bgrins] Assignee
	Description • 7 years ago

Once the new frontend is loaded in an HTML document, we should have the browser console use it.

Note that the browser console, via hudservice, relies on a xul window to persist the width/height of the window.  So we will probably need to add a file like browser-console.xul that has a full width/height iframe pointing to webconsole.html, and then have hudservice load the xul file.

	Brian Grinstead [:bgrins] Assignee
	Comment 3 • 6 years ago

I'm going to start looking into this since it seems like it's also going to require some changes to better support top-level HTML windows. For example here are some of the things I've noticed so far:

* persisting the width/height of the window after the user resizes it and then closes/reopens
* injecting top level menu items
* supporting context menu
* possibly commandDispatcher support (or removing the need for it in places like https://searchfox.org/mozilla-central/source/toolkit/content/macWindowMenu.js#24)

A workaround option would be to keep webconsole.xul but have it load a 100% width/height iframe pointing to webconsole.html. That seems like a pretty reasonable first step if we fix the other blockers of this bug but haven't worked out solutions to the above list.

	Nicolas Chevobbe [:nchevobbe]
	Comment 9 • 6 years ago
mozreview-review

Comment on attachment 8947648 [details]
Bug 1362023 - Enable new frontend in the Browser Console unconditionally;

https://reviewboard.mozilla.org/r/217366/#review241838

yay :)

	Pulsebot
	Comment 10 • 6 years ago

Pushed by bgrinstead@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9866b049f5d8
Enable new frontend in the Browser Console unconditionally;r=nchevobbe

	Raul Gurzau (:RaulG)
	Comment 11 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/9866b049f5d8

	Chris Mills [:cmills]
	Comment 12 • 6 years ago

See https://bugzilla.mozilla.org/show_bug.cgi?id=1347127#c15