113293 - Smoketest documentation for test B.26 is out of date

Status: VERIFIED FIXED

(Documentation Graveyard :: Help Viewer, defect)

Description

[Jay Farrell]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; OpenVMS COMPAQ_AlphaStation_DS10_617_MH; en-US;
rv:0.9.6) Gecko/20011112
BuildID:    2001112014

Step 6 says:

  Select Tasks | Privacy and Security | Form Manager | Demonstration. 

  Select a sample from the available list

            

  Problem is, there IS no Demonstration under Form Manager.

  There is only "View Stored Form Data" and "View Sites", neither of 

  which has a sample to choose from.



Reproducible: Always
Steps to Reproduce:
1.select tasks | Privacy and Security | Forms manager 
2.see if Demonstration appears
3.

Asa Dotzler, [e-mail], has requested this bug be opened and
assigned to him.

Comment 1

[Asa Dotzler [:asa]]

It's moved 

Step 6 should say:

"Select Debug | Form Manager Samples menu. 
 Select a sample from the available list"

Comment 2

[Asa Dotzler [:asa]]

oops, I misread. right. missing demonstration. adding morse to see if he knows
where this has gone and how we should update the smoketest.

Comment 3

[Stephen P. Morse]

Asa comment #1 is correct, so I don't understand what he meant in his comment 
#2.

But of course this will work only in debug builds.  In optimized builds it's 
much harder to get to the demo files.  You need to enter the following url into 
the location field:

   chrome://communicator/locale/wallet/index.html

Comment 4

[Colin Blake]

Attachment: Patch to fix B.26

This changes the text in B.26 as per Asa's comments in comment #1.

I have also added the chrome URL from comment #3.

Asa, if you give me an r= I'll check it in.

Comment 5

[Asa Dotzler [:asa]]

looks good to me. r=asa. Thanks!

Comment 6

[Stephen P. Morse]

Looks good to me in spirit, but I don't quite understand how the diff was 
generated.  Specifically, the following line is in the diff as an unchanged 
item

   <td><font size=-1>1. Select </font><tt>Debug | Form Manager Samples</tt>.

implying that it was there all along.  But isn't this one of the things that we 
are changing to?

Comment 7

[Colin Blake]

"Debug | Form Manager Samples" was already present at the start of B.26 step 6. 
That's the one I didn't change. Later on in the diff you'll see the "Debug | 
Form Manager Samples" that I added (without a <td>).

The "GET / HTTP/1.0" change at the beginning was just Communicator 4.7 making a 
gratuitous cosmetic change for me.

Comment 8

[Stephen P. Morse]

So in other words somebody did at one time make the wording changes, but did it 
only in one place and missed the other one.

Comment 9

[Colin Blake]

Would appear so, yes.

Comment 10

[Colin Blake]

Fix checked in. Rev 1.83. Marking fixed.

Comment 11

[Brant Gurganus]

v