124753 - [FIX]view-source: can't wrap text/plain

Status: VERIFIED FIXED

(SeaMonkey :: UI Design, enhancement, P3)

Description

[timeless]

hixie believes in long lines.  but i can't find a web browser that wraps them 
:-(

<blockquote src="bz#mozilla">
[wrap long lines] only works in the "html" mode of viewsource... so... for 
text/plain docs we never use the viewsource parser.  We just use the normal 
parser. well... all but text/plain _do_ go to view-source. let me summarize the 
current situation and you tell me what you think text/css and 
application/javascript go through the "pretty" view source view when you view 
the source among other things that allows line wrapping. text/plain does _not_ 
go through there.  It goes through the "just show it as plain text" route
so has a different background color and no wrapping allowed. text/css and 
application/javascript _may_ eventually get highlighted, but text/plain never 
will.
I can trivially make text/plain follow the same exact path as the others... it 
may make view source on plaintext docs a _tiny_ bit slower, but that's about 
it.
text/plain was odd because CNavDTD (the standard parser) was giving it "primary 
detect" in viewsource mode
</blockquote>

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: patch

Fix for the bug

Comment 2

[harishd]

Comment on attachment 68816 [details] [diff] [review]
patch

r=harishd

Comment 3

[Johnny Stenback (:jst)]

Comment on attachment 68816 [details] [diff] [review]
patch

sr=jst

Comment 4

[Boris Zbarsky [:bzbarsky]]

checked in.

Comment 5

[Patty Mac]

Verified the patch.