Bug 1546606 Comment 10 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Some more further work on top of Philipp's patch, also rebased with Prettier formatting.

Philipp, when you have a chance I could use some feedback on next steps here.  I've got the test running, but it has one failure (noted in comments), and if you comment out the failing part, it fails in two other places.  I didn't get far trying to troubleshoot those failures.

I've added "TODO DAV" comments in a few places to mark areas that seem WIP, so you might want to take a look at those.  (Some are just reminders to delete something if it ends up not being used.)

This is a summary of the changes:
- Use a single export for CalDavRequest
- Use a single export for CalDavUtils
- Capitalize names for the new jsm modules
- Fix the header iterator in test_caldav_requests.js
- Other small test fixes
- Do a basic checkRedirected function implementation, etc.
- Various edits and updates to satisfy eslint, etc.
Some more further work on top of Philipp's patch, also rebased with Prettier formatting.

Philipp, when you have a chance I could use some feedback on next steps here.  I've got the test running, but it has one failure (noted in comments), and if you comment out the failing part, it fails in two other places.  I didn't get far trying to troubleshoot those failures.

I've added "TODO DAV" comments in a few places to mark areas that seem WIP, so you might want to take a look at those.  (Some are just reminders to delete something if it ends up not being used.)

This is a summary of some of the changes:
- Use a single export for CalDavRequest
- Use a single export for CalDavUtils
- Capitalize names for the new jsm modules
- Fix the header iterator in test_caldav_requests.js
- Other small test fixes
- Do a basic checkRedirect function implementation, etc.
- Various edits and updates to satisfy eslint, etc.

Back to Bug 1546606 Comment 10