Closed
Bug 984530
Opened 7 years ago
Closed 7 years ago
Bing search tags are not working properly
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox29+ fixed, firefox30+ fixed, firefox31+ fixed)
RESOLVED
FIXED
Firefox 31
People
(Reporter: mconnor, Assigned: mconnor)
References
Details
Attachments
(1 file)
3.32 KB,
patch
|
mfinkle
:
review+
Sylvestre
:
approval-mozilla-aurora+
Sylvestre
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
In looking into the Yahoo report (bug 983723) it looks like Bing is just plain broken on 27/28 (haven't looked into older bugs). The plugin is set up to only send codes if the engine is default, but even if that's correct it's not working. It's also using the desktop codes, rather than the codes for Firefox for Android. I believe, and I'm waiting for Joanne to confirm, that we should be sending the codes at _least_ for all beta/official searches, and potentially for all channels. The patch should be straightforward once we've decided exactly what it needs to do.
Updated•7 years ago
|
Group: mozilla-employee-confidential
Updated•7 years ago
|
Summary: Yahoo search tags are not working properly → Bing search tags are not working properly
Assignee | ||
Comment 1•7 years ago
|
||
These codes are not branch-specific, and not position-specific. Making use of tablet vs. mobile because those tags are different.
Updated•7 years ago
|
Attachment #8401626 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 2•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/4846a0ac4a50
Comment 3•7 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/4846a0ac4a50
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 31
Assignee | ||
Comment 4•7 years ago
|
||
Comment on attachment 8401626 [details] [diff] [review] bingSearchCodes [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 903084 User impact if declined: missing rev credit for Bing Testing completed (on m-c, etc.): baking for a while Risk to taking this patch (and alternatives if risky): it's not working now, it won't change much String or IDL/UUID changes made by this patch: none
Attachment #8401626 -
Flags: approval-mozilla-beta?
Attachment #8401626 -
Flags: approval-mozilla-aurora?
Comment 5•7 years ago
|
||
Comment on attachment 8401626 [details] [diff] [review] bingSearchCodes Approving because it is a small change.
Attachment #8401626 -
Flags: approval-mozilla-beta?
Attachment #8401626 -
Flags: approval-mozilla-beta+
Attachment #8401626 -
Flags: approval-mozilla-aurora?
Attachment #8401626 -
Flags: approval-mozilla-aurora+
Updated•7 years ago
|
status-firefox29:
--- → affected
status-firefox30:
--- → affected
status-firefox31:
--- → fixed
tracking-firefox29:
--- → +
tracking-firefox30:
--- → +
tracking-firefox31:
--- → +
Comment 6•7 years ago
|
||
The bing search is no longer present in the default search engines. Logged bug 997179 for this issue. Tested on Nightly 31.0a1 (2014-04-16) using Google Nexus 7 (Android 4.4.2).
Comment 7•7 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/aa841fec9c84 https://hg.mozilla.org/releases/mozilla-beta/rev/148684fea6d4
Updated•4 months ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•