Closed Bug 1298512 Opened 3 years ago Closed 3 years ago
Require Host Match argument to ns Cookie Service::Check Prefs is unused
58 bytes, text/x-review-board-request
It should be safe to remove this argument. If `./mach build` succeeds, it's good enough to submit a patch! Code: netwerk/cookies/nsCookieService.cpp
Build passes, no test failures either (I ran the full suite just as a sanity check).
Comment on attachment 8785519 [details] Bug 1298512 - Remove aRequireHostMatch to nsCookieService::CheckPrefs; https://reviewboard.mozilla.org/r/74684/#review72552
Attachment #8785519 - Flags: review?(josh) → review+
Thanks for doing this! I've instructed our automation to merge the patch, so that should begin shortly.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/97f0ec5c3a9a Remove aRequireHostMatch to nsCookieService::CheckPrefs; r=jdm
You need to log in before you can comment on or make changes to this bug.