13068 - Page Source delivers bad markup when copied to clipboard (generated content)

Status: VERIFIED INVALID

(Core :: DOM: Selection, defect, P4)

Description

[Mats Palmgren (inactive)]

To repeat:
1. Load http://www.w3.org (or any page of your choice)
2. Click menu-item View -> Page Source
3. Select the text. Here's how I did it:
   Scroll to the bottom, click to the right of "</HTML>" and drag left and keep
   dragging left (even outside window) until the content is selected.
4. Click menu-item Edit -> Copy
5. Paste into the text editor of your choice (emacs 20.4 in my case)

Observations:
All start and end tags have a space after the initial "<".
All named entities have a space between the "&" and the name, e.g. "& reg;".
Line breaks are converted to space. Tabs are converted to 8 spaces.
There is an extra space after all end tags and most of the named entities.
The HTML end tag is not properly closed (I get "< /HTML" but no ">").
There is a space between attribute names and the "=".
The CONTENT attribute name in the META tags are missing, for example:
< META http-equiv ="Content-Type" ="text/html; charset=iso-8859-1">

Version and OS:
Nightly build 1999-09-01-09-M10 on Windows NT. (Also tried on Aug 31. build on
Sun/solaris 2.6/sparc and it core dumped, I will attach a stack trace.)

Comment 1

[Mats Palmgren (inactive)]

Attachment: Solaris stack trace

Comment 2

[Eli Goldberg]

mats, this is an excellent and through bug report; thank you!

Reassigning to Akkana; reproduced on 1999 9/2/99 Mac OS build. Akkana, I'll play
around with this a bit in the next few minutes.

Comment 3

[Eli Goldberg]

<duh, "thorough" bug report. friday afternoon it is. ;->

Comment 4

[Eli Goldberg]

BTW, I copied some HTML to a foo.txt file, loaded it within Mozilla, copied it,
and pasted it to a text editor (BBEdit), and it didn't result in the below-
described problem.

This only occurs from View Source.

Comment 5

[Sammy Ford]

adding myself a cc since i first implemented this

Comment 6

[Sammy Ford]

i am looking at this now, i think it is a combination of switching dtd's and
using the editor to display the source,

Comment 7

[Akkana Peck]

I'll work with sford on figuring out what's going on.

Comment 8

[Eli Goldberg]

Please note 13278, which may either be a subissue of this bug, or a duplicate.

Comment 9

[Akkana Peck]

Changing Component to Output.

Comment 10

[Joseph Elwell]

Currently the clipboard isn't working at all (Build: 1999110911) , so I can't
test this theory. But, this looks a lot like a bug that I reported earlier that
is now fixed. Check bug 16175

Comment 11

[Adam]

MozMacM14 (01.28.2000) shows similar problem - text is copied w/o the brackets "<>" and line breaks are represented by spaces.  Copying 
from Page Source window unusable.

Comment 12

[Akkana Peck]

Selection basically doesn't work in the view source window, because view source
was implemented using generated content, which we don't know how to support in
selection.  This is a problem Mike Judge is working on.  

Comment 13

[mjudge]

hmm yes we need to make the iterators for copy/serialize to use the new 
generatedcontent iterators. I will talk to akkana about this. m16

Comment 14

[rubydoo123]

this is dependent on 12460

Comment 15

[rubydoo123]

adding help wanted to the keywords

Comment 16

[rubydoo123]

oops -- wrong copy string there: setting to nsbeta3+

Comment 17

[rubydoo123]

adding brackets to status whiteboard

Comment 18

[Charles Manske]

We no longer use the same View Source strategy as we used to (like the 
Browser), so we no longer have any generated content and this problem goes 
away.

Comment 19

[Blake Ross]

*SPAM*: Changing the QA contact of all open/resolved Selection bugs from 
[e-mail] to [e-mail].  After the many great years of service 
Eli has given to Mozilla, it's time for him to move on; he has accepted a 
position at Eazel.  We'll be sad to see him go, and I'll do my best to fill his 
spot...

Comment 20

[Blake Ross]

Yep, View Source is no longer generated content and clipboard copying works 
fine on mac, win32 and linux (today's builds) -- verified fixed