Closed
Bug 106839
Opened 23 years ago
Closed 23 years ago
amTargettedCorrectly should be isTargetedCorrectly
Categories
(Core :: DOM: Editor, defect)
Core
DOM: Editor
Tracking
()
VERIFIED
FIXED
People
(Reporter: timeless, Assigned: timeless)
References
Details
Attachments
(2 files)
1.51 KB,
patch
|
bbaetz
:
review+
brendan
:
superreview+
|
Details | Diff | Splinter Review |
1.51 KB,
patch
|
Details | Diff | Splinter Review |
spun off from bug 106832 per brendan, this is for a code change. http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/editor/libeditor/html/nsHTMLDataTransfer.cpp&rev=1.29&mark=1050-1051,1054#1040 http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/editor/libeditor/text/nsPlaintextDataTransfer.cpp&rev=1.15&mark=413-414,417#410
whoops i meant bug 106386 not bug 106832. sorry.
Status: NEW → ASSIGNED
Summary: amTargettedCorrectly should be amTargetedCorrectly → amTargettedCorrectly should be amTargetedCorrectly
Comment 3•23 years ago
|
||
Comment on attachment 55182 [details] [diff] [review] s/amTargettedCorrectly/amTargetedCorrectly/g r=bbaetz, if those indentation changes are correct - theres no context in the diff for me to see.
Attachment #55182 -
Flags: review+
Comment 4•23 years ago
|
||
Comment on attachment 55182 [details] [diff] [review] s/amTargettedCorrectly/amTargetedCorrectly/g "am" seems weird -- most ppl use "is". Why is the unstated pronoun "I" rather than "it" here? sr=brendan@mozilla.org
Attachment #55182 -
Flags: superreview+
Comment 5•23 years ago
|
||
Bogus stack, still. I declare this a dup of 106860, which I'm gonna morph into "FastLoad Service leaking?" /be *** This bug has been marked as a duplicate of 106860 ***
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
Re-opening. I think you meant to mark bug 106859 a DUP of bug 106860, instead of this.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
fix checked in r=glazou (change was to match brendan's query. namely we couldn't think of a reason not to)
Status: ASSIGNED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → FIXED
Summary: amTargettedCorrectly should be amTargetedCorrectly → amTargettedCorrectly should be isTargetedCorrectly
You need to log in
before you can comment on or make changes to this bug.
Description
•