Closed
Bug 709523
Opened 13 years ago
Closed 13 years ago
Remove dead editor code
Categories
(Core :: DOM: Editor, defect)
Core
DOM: Editor
Tracking
()
RESOLVED
FIXED
mozilla11
People
(Reporter: Ms2ger, Assigned: Ms2ger)
References
Details
Attachments
(8 files)
1.79 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
2.22 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
2.48 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
2.84 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
4.28 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
2.46 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
2.30 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
776 bytes,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
Because there's so much of it I'm not going to file a bug per patch.
Attachment #580681 -
Flags: review?(ehsan)
Assignee | ||
Comment 1•13 years ago
|
||
Attachment #580682 -
Flags: review?(ehsan)
Assignee | ||
Comment 2•13 years ago
|
||
Attachment #580683 -
Flags: review?(ehsan)
Assignee | ||
Comment 3•13 years ago
|
||
Attachment #580684 -
Flags: review?(ehsan)
Assignee | ||
Comment 4•13 years ago
|
||
Attachment #580685 -
Flags: review?(ehsan)
Assignee | ||
Comment 5•13 years ago
|
||
Attachment #580686 -
Flags: review?(ehsan)
Assignee | ||
Comment 6•13 years ago
|
||
Attachment #580687 -
Flags: review?(ehsan)
Assignee | ||
Comment 7•13 years ago
|
||
Attachment #580740 -
Flags: review?(ehsan)
Updated•13 years ago
|
Attachment #580681 -
Flags: review?(ehsan) → review+
Updated•13 years ago
|
Attachment #580682 -
Flags: review?(ehsan) → review+
Updated•13 years ago
|
Attachment #580683 -
Flags: review?(ehsan) → review+
Updated•13 years ago
|
Attachment #580684 -
Flags: review?(ehsan) → review+
Updated•13 years ago
|
Attachment #580685 -
Flags: review?(ehsan) → review+
Comment 8•13 years ago
|
||
Comment on attachment 580686 [details] [diff] [review] Part f: nsHTMLEditUtils::IsAddress So, we were including a hear for bool! Nice :P
Attachment #580686 -
Flags: review?(ehsan) → review+
Updated•13 years ago
|
Attachment #580687 -
Flags: review?(ehsan) → review+
Updated•13 years ago
|
Attachment #580740 -
Flags: review?(ehsan) → review+
Comment 9•13 years ago
|
||
Nice work, thanks Ms2ger!
Assignee | ||
Comment 10•13 years ago
|
||
Thanks for the reviews! https://hg.mozilla.org/mozilla-central/rev/cd7bd3b20de0 https://hg.mozilla.org/mozilla-central/rev/b9e76ec6ac89 https://hg.mozilla.org/mozilla-central/rev/9c4cd94a11d7 https://hg.mozilla.org/mozilla-central/rev/6b33453c5a4f https://hg.mozilla.org/mozilla-central/rev/faf5d722417b https://hg.mozilla.org/mozilla-central/rev/77b565997e66 https://hg.mozilla.org/mozilla-central/rev/3d6f670f417f https://hg.mozilla.org/mozilla-central/rev/ab91118be560
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
You need to log in
before you can comment on or make changes to this bug.
Description
•