Closed Bug 87910 Opened 23 years ago Closed 23 years ago

Cannot 'Edit Page' from browser if composer window is already open

Categories

(Core :: DOM: Editor, defect, P2)

Product:
Core
Component:
DOM: Editor
Type:
defect

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla0.9.3

People

(Reporter: shrir, Assigned: sfraser_bugs)

References

Details

(Whiteboard: [QAHP] [nsBranch+,PDT+])

Attachments

(1 file)

nsEditorShell patch
735 bytes, patch
Details | Diff | Splinter Review 
hmm..not sure if this is XP apps..but I will start with composer

windows branch 0626

steps:
1 launch browser
2 do 'Tasks|Composer'
3 Keeping the composer window in the background, go to google.com in the browser 
  window
4 Now click on 'File|Edit Page'

Trust me..u won't see the page open in composer (unless u close the blank 
composer window and redo step 3+4).
reassign :simon
Assignee: beppe → sfraser
Some file URL stuff broke. Taking.
Status: NEW → ASSIGNED
Hardware: PC → All
dougt broke this in his landing.

dougt added an over-eager error return which we intended to catch lower down.
The patch removes this.
Summary: cannot 'Edit Page' from browser if composer window is already open → Cannot 'Edit Page' from browser if composer window is already open
r=dougt.

do you want me to check this in?  This should also go on the branch.
Keywords: nsBranch
adding my approval to dougt's eagernes of the nsBranch keyword
Priority: -- → P2
Target Milestone: --- → mozilla0.9.3
changing OS to ALL...happens on Mac also with 6/27 branch bits.
OS: Windows NT → All
*** Bug 88197 has been marked as a duplicate of this bug. ***
adding QAHP.....easy to run into this problem.
Whiteboard: [QAHP]
Fixed.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
can we fix on branch?
Keywords: vtrunk
this did not get fixed on the branch....using 7/5 branch
build, it is not fixed.
Reopening for the branch.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
marking nsBranch+
Whiteboard: [QAHP] → [QAHP] nsBranch+
mark PDT+ to check into the branch. If possible, pls get this in for Friday am
builds.
Whiteboard: [QAHP] nsBranch+ → [QAHP] [nsBranch+,PDT+]
Fixed on the branch.
Status: REOPENED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → FIXED
verified on 7/9 trunk...will verify on branch also.
verified on branch and trunk using 7/9 build

marking verified and removing vtrunk keyword.
Status: RESOLVED → VERIFIED
Keywords: vtrunk
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: