1451081 - Make viewing source work again in SeaMonkey after bug 1418403

Status: NEW

(SeaMonkey :: General, enhancement, P3)

Description

[Frank-Rainer Grahl (:frg)]

Bug 1418403 broke viewing source code in SeaMonkey and TB. TB restored some functionality in bug 1439021 but we need to further adapt this.

+++ This bug was initially created as a clone of Bug #1418403 +++

This was mentioned in https://bugzilla.mozilla.org/show_bug.cgi?id=1416372#c3, so spinning it off into a new bug.

From https://bugzilla.mozilla.org/show_bug.cgi?id=1416372#c2:

When we collected data on the different modes 2 years ago (bug 1203624 comment 11), we saw:

View Source in Browser:         56,700 uses (99.30%)
View Source in Window:             357 uses ( 0.62%)
View Source External (Success):     44 uses ( 0.08%)
View Source External (Failed):      48 uses (re-counted as browser / window above)

Maybe these numbers are low enough that we should remove the pref and the additional code to support both modes.

Comment 1

[Frank-Rainer Grahl (:frg)]

Attachment: 1451081-viewSource-wip.diff

WIP patch from stefanh with a quick copy/paste from me (browser.js to navigator.js). We think we should keep the e10s parts to ease later porting.

We would prefer to open a new browser window insetad of a tab but this might not be so easy as I thought.

Stefan do you want to take the bug?

Comment 2

[Stefan [:stefanh]]

frg, the patch you provided is on top of a bunch of other patches. Hmm, skip the overlay now when it doesn't open in a new window?

Comment 3

[Frank-Rainer Grahl (:frg)]

> frg, the patch you provided is on top of a bunch of other patches.

Sent you my patch queue. SeaMoneky dumps without it.

> Hmm, skip the overlay now when it doesn't open in a new window?

The overlay probably needs to be removed. Doesn't make sense any longer with opening the source directly in the browser and they need to go anyway. But hard to test with mail broken. For opening the source in a Window we would need to create a new browser and then get the tab. handURIToExistingBrowser or another function can probably not be used because we need the tab value. So also not sure if it is worth it to figure it out. Maybe in a followup bug.

Comment 4

[Frank-Rainer Grahl (:frg)]

Attachment: 1451081-WIP-viewSource-257.patch

Rebased 2.57 wip patch

Comment 5

[Frank-Rainer Grahl (:frg)]

Attachment: 1451081-WIP-viewSource.patch

Rebased c-c wip patch