Closed
Bug 1177723
Opened 9 years ago
Closed 9 years ago
Remove remaining database system search integration
Categories
(Firefox for Android Graveyard :: Data Providers, defect)
Tracking
(firefox39+ fixed, firefox40+ fixed, firefox41+ fixed, firefox42+ fixed, fennec39+)
People
(Reporter: rnewman, Assigned: rnewman, NeedInfo)
References
Details
Attachments
(1 file)
7.84 KB,
patch
|
mfinkle
:
review+
lizzard
:
approval-mozilla-aurora+
lizzard
:
approval-mozilla-beta+
lizzard
:
approval-mozilla-release+
|
Details | Diff | Splinter Review |
Following on from Bug 1111221, we should additionally remove search_suggest_query stuff from BrowserProvider and its manifest entry.
Assignee | ||
Comment 1•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=12c804e64d2f
Assignee | ||
Comment 2•9 years ago
|
||
Figure I might as well upload this.
Attachment #8628074 -
Flags: review?(mark.finkle)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → rnewman
Status: NEW → ASSIGNED
Updated•9 years ago
|
Attachment #8628074 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 3•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/a11965c0a317
Assignee | ||
Comment 4•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=2928db3d7239
Assignee | ||
Comment 5•9 years ago
|
||
Looks like this might need to chase Bug 1111221; initial test results from partner were wrong. Try build: https://treeherder.mozilla.org/#/jobs?repo=try&revision=1b8565ac4c81
Assignee | ||
Comment 6•9 years ago
|
||
Also if this works, I owe mfinkle a hat-eating.
tracking-fennec: --- → 39+
status-firefox39:
--- → affected
Assignee | ||
Comment 7•9 years ago
|
||
Comment on attachment 8628074 [details] [diff] [review] Remove remaining database system search integration. v1 Risk is relatively low in my opinion and mfinkle's. Already asked kbrosnan if he can verify the try push. Nothing in that should be affected by release flags, so if everything works the findings should apply to release. We can point the partner at that build and reverify, if we have time before 39 release, or we can ship and follow up.
Flags: needinfo?(mconnor)
Flags: needinfo?(kbrosnan)
Attachment #8628074 -
Flags: approval-mozilla-release?
Attachment #8628074 -
Flags: approval-mozilla-aurora?
Comment 8•9 years ago
|
||
OK, tracking for everything. This is a followup from bug 1111220 and is important for a partner distribution. We may need to build fennec 39 again if this fix holds.
status-firefox40:
--- → affected
status-firefox41:
--- → affected
status-firefox42:
--- → affected
tracking-firefox39:
--- → +
tracking-firefox40:
--- → +
tracking-firefox41:
--- → +
tracking-firefox42:
--- → +
Assignee | ||
Comment 9•9 years ago
|
||
Explanation of what's going on, as far as I understand it: With Fennec installed, a device fails the Google Services Test Suite. It fails in testWidgetPresence: Launcher from package com.android.systemui does not contain search widget provided by com.google.android.googlequicksearchbox (Google App) This system functionality was removed in Android 4.mumble. My impression, then is that this is an Android bug: our manifests support the system search widget on earlier Androids, and so we're causing a load that the harness no longer expects. Each of these two fixes remove another chunk of that search functionality. If this doesn't do it, I'll need to request access to GSTS to debug further.
Assignee | ||
Comment 10•9 years ago
|
||
Main areas to verify: that we don't painfully regress any system search stuff in GB (probably not because we removed it?), and that basic smoketests (saving history, recording searches from Search Activity) are all green everywhere.
Comment 11•9 years ago
|
||
Am I waiting for verification on the fix before we rebuild? Or should I be rebuilding fennec 39 now? I can start such a build either before 4pm or so pacific time, or maybe very late tonight. I can also set it up in ship-it and someone else can put in the correct changeset if it's ready tonight and needs to happen.
Flags: needinfo?(rnewman)
Assignee | ||
Comment 12•9 years ago
|
||
We can rebuild now. I would encourage us to wait for Kevin's verification before pushing builds out, though!
Flags: needinfo?(rnewman)
Comment 13•9 years ago
|
||
This looks good manually. Tested on a Android 5.0, 4.4 and 2.3 device. Filed a few bugs but they are not regressions from this.
Flags: needinfo?(kbrosnan)
Assignee | ||
Comment 14•9 years ago
|
||
Thanks, Kevin. Liz: patch is attached to this bug, and applied cleanly to mozilla-release when I pushed to Try. I'll check in later tonight to see if I can help with anything.
Comment 15•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/a11965c0a317
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
Comment 16•9 years ago
|
||
Comment on attachment 8628074 [details] [diff] [review] Remove remaining database system search integration. v1 Approved for uplift to aurora and release.
Comment 17•9 years ago
|
||
Comment on attachment 8628074 [details] [diff] [review] Remove remaining database system search integration. v1 Actually this needs to uplift to 39 (on m-r) 40 (on m-b) and 41 (m-a) since we did the merge already.
Attachment #8628074 -
Flags: approval-mozilla-beta+
Updated•9 years ago
|
Attachment #8628074 -
Flags: approval-mozilla-release?
Attachment #8628074 -
Flags: approval-mozilla-release+
Attachment #8628074 -
Flags: approval-mozilla-aurora?
Attachment #8628074 -
Flags: approval-mozilla-aurora+
Updated•3 years 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
•