Closed Bug 1171828 Opened 4 years ago Closed 4 years ago

Remove UA override for domains (starting 2015-06-05)

Categories

(Core :: Networking, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla41
Tracking Status
firefox41 --- fixed

People

(Reporter: karlcow, Assigned: karlcow)

References

()

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1103680 +++

This is the placeholder for the next batch of UA override removal.
https://wiki.mozilla.org/Compatibility/Mobile/WipeOutUAOverides

Previous Batch was Bug 1103680
Depends on: 826514
No longer depends on: 826335, 1021132, 1103680
See Also: → 1103680
Depends on: 826711
Remove UA override for 

    11   // bug 826514, estadao.com.br
    12   "estadao.com.br": "\\(Mobile#(Android; Mobile",
    13   // bug 826711, bb.com.br
    14   "bb.com.br": "\\(Mobile#(Android; Mobile",
Depends on: 827573
Remove UA override for

    21   // bug 827573, webmotors.com.br
    22   "webmotors.com.br": "\\(Mobile#(Android; Mobile",
Remove UA override for


    27   // bug 827678, marca.com
    28   "marca.com": "\\(Mobile#(Android; Mobile",
Depends on: 827678
Depends on: 828371
Remove UA override for 



    29   // bug 828371, ingbank.pl
    30   "ingbank.pl": "\\(Mobile#(Android; Mobile",



    35   // bug 828422, publico.es
    36   "publico.es": "\\(Mobile#(Android; Mobile",



Need to fix this (wrong comment for wrong number):

    39   // bug 843129, 11870.com
this one is already fixed

    40   "iphonejuegosgratis.com": "\\(Mobile#(Android; Mobile",
and this one became invalid.




    51   // bug 878232, hazipatika.com
    52   "hazipatika.com": "\\(Mobile#(Android; Mobile",


    63   // bug 878253, vatera.hu
    64   "vatera.hu": "\\(Mobile#(Android; Mobile",


    69   // bug 878264, haber.ba
    70   "haber.ba": "\\(Mobile#(Android; Mobile",
    71   // bug 878271, kurir-info.rs
    72   "kurir-info.rs": "\\(Mobile#(Android; Mobile",


    75   // bug 878277, naslovi.net
    76   "naslovi.net": "\\(Mobile#(Android; Mobile",
    77   // bug 878649, univision.com
    78   "univision.com": "\\(Mobile#(Android; Mobile",



    81   // bug 878655, vesti-online.com
    82   "vesti-online.com": "\\(Mobile#(Android; Mobile"
Attachment #8615854 - Flags: review?(lmandel)
This is the patch for the last overdue batch of UA override removal.
Assignee: nobody → kdubost
Whoa, awesome. That's a lot of lines removed. :)
Comment on attachment 8615854 [details] [diff] [review]
ua-override-bug1171828.patch

(In reply to Mike Taylor [:miketaylr] from comment #7)
> Whoa, awesome. That's a lot of lines removed. :)

+1 to that!
Attachment #8615854 - Flags: review?(lmandel) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/6d1ae04935f1
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
See Also: → 1180543
You need to log in before you can comment on or make changes to this bug.