Fix Android unit tests on cedar
Categories
(Firefox :: Search, enhancement, P1)
Tracking
()
People
(Reporter: mikedeboer, Assigned: mikedeboer)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
bug 1492475 introduced a large amount of changes, but getting an Android build done locally took too much time to understand why Services.search
wasn't resolved properly on Android. Does it need a clobber? Probably not, but time needs to be spent here to resolve this. However, this can continue without blocking other work.
We obviously won't merge cedar into m-c when the Search Service doesn't work on Android.
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Comment 2•6 years ago
|
||
Updated•6 years ago
|
Assignee | ||
Comment 3•6 years ago
|
||
Assignee | ||
Comment 4•6 years ago
|
||
Florian, I re-requested review, because I changed direction substantially... the approach I've taken in the new patch is to fix up the Android Search Service shim. I explained things further in the commit message.
Assignee | ||
Comment 5•6 years ago
|
||
n-i, Florian, because I'm not sure whether the Phabricator flagging actually works.
Comment 6•6 years ago
|
||
(In reply to Mike de Boer [:mikedeboer] from comment #5)
n-i, Florian, because I'm not sure whether the Phabricator flagging actually works.
I have no idea what "the Phabricator flagging" means in this context, so I guess that means it didn't do what you wanted.
Updated•6 years ago
|
Assignee | ||
Comment 7•6 years ago
|
||
Assignee | ||
Updated•6 years ago
|
Description
•