Closed Bug 948642 Opened 12 years ago Closed 12 years ago

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

Categories

(Core :: Networking, defect)

Product:
Core
Component:
Networking
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla30

People

(Reporter: karlcow, Assigned: karlcow)

References

Details

Attachments

(1 file, 1 obsolete file)

ua-override-bug948642.patch
6.51 KB, patch
lmandel
: review+
Details | Diff | Splinter Review
Placeholder for the next batch of domains UA override change.
Remove UA override for urbanspoon.com - Bug 843162
Depends on: 843162
See Also: → 941911
Remove UA override for maruccisports.com - Bug 843124
Depends on: 843124
Remove UA override for www.yahoo.com - Bug 826338
Depends on: 826338
Remove UA override for sat.gob.mx - Bug 878647
Depends on: 878647
Remove UA override for deviantart.com - Bug 843136
Depends on: 843136
Remove UA override for wwwhatsnew.com - Bug 843116
Depends on: 843116
Remove UA override for tylted.com - Bug 843176
Depends on: 843176
Remove UA override for different OLX domains Bug 878645, Bug 826720, Bug 827672, Bug 828433
Depends on: 878645, 826720, 827672, 828433
Remove UA override for scribd.com Bug 827668
Depends on: 827668
Attached patch ua-override-bug948642.patch (obsolete) — — Splinter Review
Patche for the last batch of UA override removal. :) Yahoo, olx, Scribd included.
Attachment #8365746 - Flags: review?(lmandel)
See Also: → 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-
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+
Thanks for the review. Adding checkin-needed.
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/d19ea42d339f
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: