Closed Bug 1179086 Opened 7 years ago Closed 7 years ago

ContentEventHandler should use \n for native linebreaker rather than \r

Categories

(Core :: DOM: UI Events & Focus Handling, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla42
Tracking Status
firefox42 --- fixed

People

(Reporter: masayuki, Assigned: masayuki)

References

()

Details

(Keywords: inputmethod)

Attachments

(1 file)

On Mac OS X, looks like that typical text files use \n rather than \r. So, we should use that. Then, we can skip to replace \n to \r.
Attached patch PatchSplinter Review
See comment 0 for the detail.
Attachment #8628194 - Flags: review?(smichaud)
Attachment #8628194 - Flags: review?(bugs)
Attachment #8628194 - Flags: review?(joshmoz)
Attachment #8628194 - Flags: review?(bugs)
Attachment #8628194 - Flags: review+
Comment on attachment 8628194 [details] [diff] [review]
Patch

Good catch :-)
Attachment #8628194 - Flags: review?(smichaud) → review+
Attachment #8628194 - Flags: review?(joshmoz) → review+
https://hg.mozilla.org/mozilla-central/rev/b72b79f155ac
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
Component: Event Handling → User events and focus handling
You need to log in before you can comment on or make changes to this bug.