352005 - Sunbird doesn't include view source support

Status: VERIFIED FIXED

(Calendar :: General, defect)

Description

[Matthew (lilmatt) Willis]

Sunbird doesn't include view source support.

Unfortunately, this means that when a user clicks a chrome:// link in the Error Console, no source editor pops up, but instead a warning about a missing toolkit xul file appears.... maybe.  The dialog is hidden and buried beneath a bunch of other windows.

On Mac, clicking on that chrome:// link adds two empty entries under the Window menu.  You have to close the Error Console to find the error message.

"Alert
The file
jar:file:///Users/mwillis/moz/printing/mozilla/obj-i386-apple-darwin8.7.1/dist/CalendarDebug.app/Contents/MacOS/chrome/toolkit.jar!/content/global/viewSource.xul
cannot be found. Please check the location and try again.


We need to remove "MOZ_VIEW_SOURCE= " from configure.in to fix this. The two lines that should be added to /m/cal/installer/windows/packages-static are already there.

Comment 1

[Matthew (lilmatt) Willis]

Attachment: removes disabling of MOZ_VIEW_SOURCE= from configure.in

We should pick up MOZ_VIEW_SOURCE=1 from the defaults.

Requesting r1 from bsmedberg since we're touching configure.in.

We'll want to land this on 1.8.1 as well to keep things in sync, after baking it on trunk.

Comment 2

[Michiel van Leeuwen (email: mvl+moz@)]

We can live without this for 0.3

Comment 3

[Michiel van Leeuwen (email: mvl+moz@)]

Comment on attachment 237578 [details] [diff] [review]
removes disabling of MOZ_VIEW_SOURCE=   from configure.in

r=mvl

Comment 4

[Benjamin Smedberg]

Comment on attachment 237578 [details] [diff] [review]
removes disabling of MOZ_VIEW_SOURCE=   from configure.in

The closer we can get to the xulrunner platform defaults the happier I'll be.

Comment 5

[Matthew (lilmatt) Willis]

Patch checked in on trunk.

Will let bake before requesting a181.

Comment 6

[Martin Schröder [:mschroeder]]

On trunk with latest Sunbird nightly I get an Alert dialog "view-source is not a registered protocol" and the following error in the console:

Error: uncaught exception: [Exception... "Component returned failure code: 0x804b0012 [nsIWebNavigation.loadURI]"  nsresult: "0x804b0012 (<unknown>)"  location: "JS frame :: chrome://global/content/viewSource.js :: viewSource :: line 143"  data: no]

Comment 7

[Matthew (lilmatt) Willis]

Attachment: adds "viewsource" to the default necko protocols

I forgot to add the protocol to our list.

Comment 8

[Matthew (lilmatt) Willis]

Attachment: add'l prefs for view-source

Additional prefs for view-source.
Omitting the find-bar one gives this exception:

JavaScript error: , line 0: uncaught exception: [Exception... "Component returned failure code: 0x8000ffff (NS_ERROR_UNEXPECTED) [nsIPrefBranch.getIntPref]"  nsresult: "0x8000ffff (NS_ERROR_UNEXPECTED)"  location: "JS frame :: chrome://global/content/findBar.js :: anonymous :: line 97"  data: no]

Comment 9

[Michiel van Leeuwen (email: mvl+moz@)]

Comment on attachment 242136 [details] [diff] [review]
adds "viewsource" to the default necko protocols

r2=mvl

Comment 10

[Michiel van Leeuwen (email: mvl+moz@)]

Comment on attachment 242137 [details] [diff] [review]
add'l prefs for view-source

r=mvl

Comment 11

[Matthew (lilmatt) Willis]

Checked in on trunk. Baking before asking for a181

Comment 12

[Matthew (lilmatt) Willis]

Baked. 
Landed on MOZILLA_1_8_BRANCH.

-> FIXED

Comment 13

[Martin Schröder [:mschroeder]]

VERIFIED with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.4pre) Gecko/20070406 Calendar/0.5pre.