Closed Bug 1318432 Opened 3 years ago Closed 3 years ago

Make it possible to switch on rust-url at runtime

Categories

(Core :: Networking, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: manishearth, Assigned: valentin)

References

Details

(Whiteboard: [necko-active])

Attachments

(1 file)

Currently, if MOZ_RUST_URLPARSE is set, each nsStandardURL will contain a mirror RustURL. When parsing/setting, the method is run on both. If there are mismatches when getting, the mismatch is reported via MOZ_LOG.

The inline MOZ_RUST_URLPARSE_FALLBACK define controls whether or not to fall back to the nsStandardURL value when there is a mismatch. It is enabled by default, but can be disabled to get an idea of what tests will fail with RustURL.

We should have a runtime pref that controls the fallback.
MozReview-Commit-ID: 3BfqFWIpS7F
Assignee: nobody → valentin.gosu
Status: NEW → ASSIGNED
Comment on attachment 8812762 [details] [diff] [review]
Make it possible to switch on rust-url at runtime

Review of attachment 8812762 [details] [diff] [review]:
-----------------------------------------------------------------

This patch adds a runtime pref to enable the rust parser. It defaults to false as we have discovered several regressions (both crashes and performance) caused by bug 1151899.
Attachment #8812762 - Flags: review?(daniel)
Whiteboard: [necko-active]
Attachment #8812762 - Flags: review?(daniel) → review+
https://hg.mozilla.org/mozilla-central/rev/688a4bbf8018
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
it appears that this change caused many performance improvements:
https://treeherder.mozilla.org/perf.html#/alerts?id=4261
More accurately, it fixed a perf regression caused by https://bugzilla.mozilla.org/show_bug.cgi?id=1151899 :)
You need to log in before you can comment on or make changes to this bug.