815801 - Remove all of the site specific user agent overrides as part of backing out bug 588909 [port bug 815743]

Status: RESOLVED FIXED

(SeaMonkey :: Preferences, defect)

Description

[Jens Hatlak (:InvisibleSmiley)]

We live in turbulent times. It's really funny to see how Mozilla turns its hundreds of millions of release users into beta testers. But I digress...

Bug 815743, which just landed pretty much everywhere, reverts bug 588909 ("Replace Gecko/<date> with Gecko/<version> in UA string"). Besides changes to configure.in that I cannot really comment on ("Part 1") it removed again all newly introduced prefs for site-specific UA overrides ("Part 2") since obviously there's nothing to override anymore once the default UA is back to normal.

As I said no idea about configure.in (Callek, ewong?). Otherwise on our side we need to revert the changes made in bug 801318 and bug 809517.

Comment 1

[Jens Hatlak (:InvisibleSmiley)]

Attachment: remove UA overrides

Comment 2

[Ian Neal]

(In reply to Jens Hatlak (:InvisibleSmiley) from comment #0)
> As I said no idea about configure.in (Callek, ewong?). Otherwise on our side
> we need to revert the changes made in bug 801318 and bug 809517.

As far as I can see comm-central would need the MOZ_UA_BUILDID change, but not 100% sure.

Comment 3

[Ian Neal]

Comment on attachment 685835 [details] [diff] [review]
remove UA overrides

[Triage Comment]
Needed on c-a and c-b so approving
Will leave build team to decide on c-r

Comment 4

[Jens Hatlak (:InvisibleSmiley)]

Base bug 815743 has been backed out "because of leaks" for now. Thankfully I had already decided to delay landing anything on our side until it has baked a bit on the m-c side of things... Anyway I guess we need to closely monitor the situation and make sure to stay in sync with any actions taken on the core side, esp. wrt. the release branch.

Comment 5

[Philip Chee]

> -pref("general.useragent.complexOverride.moodle", true); // bug 797703
The leak was caused by removing this line. Leave this in but flip it to false.

Comment 6

[Jens Hatlak (:InvisibleSmiley)]

(In reply to Philip Chee from comment #5)
> > -pref("general.useragent.complexOverride.moodle", true); // bug 797703
> The leak was caused by removing this line. Leave this in but flip it to
> false.

OK, makes sense. Anyone feel free to land the patch with that addressed (I can't for ~10 hours).

Comment 7

[Jens Hatlak (:InvisibleSmiley)]

Attachment: remove UA overrides v2 [Checkin: Comment 8]

Patch with comment 5 addressed

Comment 8

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 686283 [details] [diff] [review]
remove UA overrides v2 [Checkin: Comment 8]

http://hg.mozilla.org/comm-central/rev/2d4a1f0d87e9

Comment 9

[Philip Chee]

Comment on attachment 686283 [details] [diff] [review]
remove UA overrides v2 [Checkin: Comment 8]

We need this on SM 2.14.1 as well. The core backout patch landed on all branches including ESR17

Comment 10

[Philip Chee]

Pushed to all branches:
http://hg.mozilla.org/releases/comm-aurora/rev/23b5727e2052
http://hg.mozilla.org/releases/comm-beta/rev/514d7c9d6822
http://hg.mozilla.org/releases/comm-release/rev/eeb30b209095