Fixes #1945 Restore library panels

[keianhzo]

Fixes #1945 Restore library panels. I really don't like to have a flag to know if the user has initiated the interaction so if there is a better alternative I'd be happy to change this. Without the flag the panels would be closed during the restore when about:blank and home are loaded as we always close them when there is a navigation event.

[bluemarvin]

Getting a first run crash when updating FxR with this patch applied.

[keianhzo]

This should be ready for review.

[bluemarvin]

I agree this isn't pretty but I can't think of anything cleaner. @MortimerGoro last chance to comment before landing it.

[MortimerGoro]

@bluemarvin @keianhzo would it be possible to use a URL based approach instead of adding the extra flags and enum?

For example map about://history or about://bookmarks kind of URLs to the library panels

[bluemarvin]

@bluemarvin @keianhzo would it be possible to use a URL based approach instead of adding the extra flags and enum?

For example map about://history or about://bookmarks kind of URLs to the library panels

That could be workable. I'm not sure how we would make it work with the GV history system though. I like the concept, it isn't clear how we would implement it.

[keianhzo]

@bluemarvin @MortimerGoro That a good idea, I've added support for it. It doesn't seem to mess up the history and it's a much cleaner solution for restoring.

[bluemarvin]

Much cleaner. Seems to work well too.