Closed
Bug 168353
Opened 22 years ago
Closed 22 years ago
Send Link doesn't work
Categories
(MailNews Core :: Composition, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: scottputterman, Assigned: vparthas)
References
Details
(Keywords: regression, Whiteboard: [adt1])
Attachments
(1 file)
839 bytes,
patch
|
sspitzer
:
review+
sspitzer
:
superreview+
|
Details | Diff | Splinter Review |
using the 9/12 build.
1. Go to a page.
2. Choose Send Link
nothing happens.
Get js error:
Error: missing ) after argument list
Source Code:
sendLink(Components.lookupMethod(window._content,
'location').call(window._content).href,
Components.lookupMethod(window._content.document,
'title').call(window._content.document);
Reporter | ||
Comment 1•22 years ago
|
||
This is a bad regression. Let's try to get this fixed pretty quickly.
Keywords: nsbeta1+,
regression
Whiteboard: [adt1]
I can reproduce this. Looking into it ASAP.
Status: NEW → ASSIGNED
Comment 4•22 years ago
|
||
was editor broken at all?
*** Bug 168280 has been marked as a duplicate of this bug. ***
Comment 6•22 years ago
|
||
Comment on attachment 98971 [details] [diff] [review]
Patch V1
r/sr=sspitzer
check in. thanks varada.
Attachment #98971 -
Flags: superreview+
Attachment #98971 -
Flags: review+
Comment 7•22 years ago
|
||
No editor wasn't broken. r=caillon too for this patch. Sigh. Sorry about this.
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Comment 10•22 years ago
|
||
Using trunk build 20021118 on winxp, macosx and linux this is fixed. verified.
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•