Closed
Bug 1711628
Opened 3 years ago
Closed 3 years ago
Remove some obsolete UA string code: MOZ_UA_OS_AGNOSTIC, #pragma warning, do_GetService("http")
Categories
(Core :: Networking: HTTP, task, P3)
Core
Networking: HTTP
Tracking
()
RESOLVED
FIXED
90 Branch
People
(Reporter: cpeterson, Assigned: cpeterson)
References
Details
Attachments
(3 files)
I have a larger patch set refactoring some UA string code (bug 1556223), but I'd like to spin out these small unrelated patches and land them first.
Assignee | ||
Comment 1•3 years ago
|
||
The MOZ_UA_OS_AGNOSTIC #ifdefs were added for Firefox OS because it hoped to omit OS ("Firefox OS" or "Android") in its UA string.
Assignee | ||
Comment 2•3 years ago
|
||
Also remove some obsolete MSVC #pragmas since we no longer support MSVC.
Depends on D115331
Assignee | ||
Comment 3•3 years ago
|
||
Depends on D115332
Pushed by cpeterson@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3663feeda8f8 UA String: Remove unused MOZ_UA_OS_AGNOSTIC #ifdefs. r=hsivonen,necko-reviewers,dragana https://hg.mozilla.org/integration/autoland/rev/694438b64baf UA String: If GetVersionEx() fails, report Windows NT 10.0 instead of omitting Windows version. r=hsivonen,necko-reviewers,dragana https://hg.mozilla.org/integration/autoland/rev/4217ed273ebb navigator.platform: Avoid unnecessary do_GetService("http") on Windows and macOS. r=hsivonen
Comment 5•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/3663feeda8f8
https://hg.mozilla.org/mozilla-central/rev/694438b64baf
https://hg.mozilla.org/mozilla-central/rev/4217ed273ebb
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•