Closed Bug 1518552 Opened 6 years ago Closed 6 years ago

Fork saxparser to common/ after its removal in bug 1447707

Categories

(MailNews Core :: General, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 66.0

People

(Reporter: jorgk-bmo, Unassigned)

References

Details

No description provided.
Summary: Fork saxparser to common after it's removal in bug 1447707 → Fork saxparser to common/ after its removal in bug 1447707

I'll push the removed code to C-C now.

Keywords: leave-open
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/837f28a0a4f2 Fork saxparser to common/ after its removal in bug 1447707. rs=bustage-fix

I'll see whether I can get the tests going.

Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/53e0e1023adc Follow-up: Remove unneeded test_sanitizer.js. rs=bustage-fix DONTBUILD

Geoff, could you please take a look at the linting problems this introduced. And if you could get the test going, that would be a bonus.

Flags: needinfo?(geoff)

Test already works:
https://taskcluster-artifacts.net/LvgPOcyFRcyIUcoiUXSuyA/0/public/logs/live_backing.log
TEST-PASS | comm/common/saxparser/test/unit/test_parser.js

Also the one we don't need since it's still in M-C worked:
TEST-PASS | comm/common/saxparser/test/unit/test_sanitizer.js

So all that's left is fixing the linting errors:
c:\mozilla-source\comm-central\comm\common\saxparser\test\unit.eslintrc.js
6:4 error Missing trailing comma. comma-dangle (eslint)

c:\mozilla-source\comm-central\comm\common\saxparser\test\unit\test_parser.js
24:20 error Unexpected named method 'startDocument'. func-names (eslint)
27:18 error Unexpected named method 'endDocument'. func-names (eslint)
etc.

I can of course put // eslint-disable-line func-names onto those.

Flags: needinfo?(geoff)
Keywords: leave-open
Target Milestone: --- → Thunderbird 66.0

Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/715f97e4c9ae
Follow-up: Fix linting errors. rs=bustage-fix DONTBUILD

Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.