Remove UA override for domains (starting 2013-12-10)

RESOLVED FIXED in mozilla30

Status

()

Core
Networking
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: karlcow, Assigned: karlcow)

Tracking

unspecified
mozilla30
ARM
Gonk (Firefox OS)
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

4 years ago
Placeholder for the next batch of domains UA override change.
(Assignee)

Comment 1

4 years ago
Remove UA override for urbanspoon.com - Bug 843162
Depends on: 843162
(Assignee)

Updated

4 years ago
See Also: → bug 941911
(Assignee)

Comment 2

4 years ago
Remove UA override for maruccisports.com - Bug 843124
Depends on: 843124
(Assignee)

Comment 3

4 years ago
Remove UA override for www.yahoo.com - Bug 826338
Depends on: 826338
(Assignee)

Comment 4

4 years ago
Remove UA override for sat.gob.mx - Bug 878647
Depends on: 878647
(Assignee)

Comment 5

4 years ago
Remove UA override for deviantart.com - Bug 843136
Depends on: 843136
(Assignee)

Comment 6

4 years ago
Remove UA override for wwwhatsnew.com - Bug 843116
Depends on: 843116
(Assignee)

Comment 7

4 years ago
Remove UA override for tylted.com - Bug 843176
Depends on: 843176
(Assignee)

Comment 8

4 years ago
Remove UA override for different OLX domains Bug 878645, Bug 826720, Bug 827672, Bug 828433
Depends on: 878645, 826720, 827672, 828433
(Assignee)

Comment 9

4 years ago
Remove UA override for scribd.com Bug 827668
Depends on: 827668
(Assignee)

Comment 10

4 years ago
Created attachment 8365746 [details] [diff] [review]
ua-override-bug948642.patch

Patche for the last batch of UA override removal. :) Yahoo, olx, Scribd included.
Attachment #8365746 - Flags: review?(lmandel)
(Assignee)

Updated

4 years ago
See Also: → bug 965060
Comment on attachment 8365746 [details] [diff] [review]
ua-override-bug948642.patch

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

Looks like you scribd.com should also be removed. Please confirm by either removing the override or dropping the bug dependency.
Attachment #8365746 - Flags: review?(lmandel) → review-
(Assignee)

Comment 12

4 years ago
Created attachment 8369023 [details] [diff] [review]
ua-override-bug948642.patch

Sorry Lawrence for the previous patch.
Attachment #8369023 - Flags: review?(lmandel)
Flags: needinfo?(lmandel)
Attachment #8365746 - Attachment is obsolete: true
Flags: needinfo?(lmandel)
Comment on attachment 8369023 [details] [diff] [review]
ua-override-bug948642.patch

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

No apology necessary Karl. That's why I'm here.

Updated patch looks good. Glad to squash these overrides. Keep up the good work.
Attachment #8369023 - Flags: review?(lmandel) → review+
(Assignee)

Comment 14

4 years ago
Thanks for the review. Adding checkin-needed.
Keywords: checkin-needed
https://hg.mozilla.org/integration/b2g-inbound/rev/d19ea42d339f
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/d19ea42d339f
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.