Closed Bug 1716299 Opened 3 years ago Closed 3 years ago

Use of unknown property Ci.nsIDocumentEncoder.OutRaw in test_nsIEditor_outputToString.html

Categories

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

defect

Tracking

()

RESOLVED FIXED
91 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox89 --- unaffected
firefox90 --- unaffected
firefox91 --- fixed

People

(Reporter: standard8, Assigned: masayuki)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Bug 1713867 added test_nsIEditor_outputToString.html, we have a manually (at the moment) run eslint rule that has picked up the following:

editor/libeditor/tests/test_nsIEditor_outputToString.html
   32:51  error  Use of unknown property Ci.nsIDocumentEncoder.OutRaw  mozilla/ci-checks (eslint)
   38:51  error  Use of unknown property Ci.nsIDocumentEncoder.OutRaw  mozilla/ci-checks (eslint)
   44:51  error  Use of unknown property Ci.nsIDocumentEncoder.OutRaw  mozilla/ci-checks (eslint)
   53:51  error  Use of unknown property Ci.nsIDocumentEncoder.OutRaw  mozilla/ci-checks (eslint)
   59:51  error  Use of unknown property Ci.nsIDocumentEncoder.OutRaw  mozilla/ci-checks (eslint)
   65:51  error  Use of unknown property Ci.nsIDocumentEncoder.OutRaw  mozilla/ci-checks (eslint)
   85:56  error  Use of unknown property Ci.nsIDocumentEncoder.OutRaw  mozilla/ci-checks (eslint)
   91:56  error  Use of unknown property Ci.nsIDocumentEncoder.OutRaw  mozilla/ci-checks (eslint)
   97:56  error  Use of unknown property Ci.nsIDocumentEncoder.OutRaw  mozilla/ci-checks (eslint)
  105:56  error  Use of unknown property Ci.nsIDocumentEncoder.OutRaw  mozilla/ci-checks (eslint)
  111:56  error  Use of unknown property Ci.nsIDocumentEncoder.OutRaw  mozilla/ci-checks (eslint)
  117:56  error  Use of unknown property Ci.nsIDocumentEncoder.OutRaw  mozilla/ci-checks (eslint)
Flags: needinfo?(masayuki)

Thanks, I wonder, why it was not detected by the treeherder's jobs.

Assignee: nobody → masayuki
Status: NEW → ASSIGNED
Flags: needinfo?(masayuki)
Severity: -- → N/A
Priority: -- → P3

nsIDocumentEncoder.OutputRaw was misspelled as nsIDocumentEncoder.OutRaw.

Then, I investigate the expected result more. Then, I see that when only this
flag is set, any markups should be ignored except <br>. Therefore, I modified
the 2 test results which check <p>, <div> and <br> elements.

(In reply to Masayuki Nakano [:masayuki] (he/him)(JST, +0900) from comment #1)

Thanks, I wonder, why it was not detected by the treeherder's jobs.

The rule is a manually run ESLint one, as it depends on build output, and we haven't got around to setting up a specific job for it.

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 91 Branch
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: