Launchpad has imported 21 comments from the remote bug at https://bugzilla.mozilla.org/show_bug.cgi?id=1570700.
If you reply to an imported comment from within Launchpad, your comment will be sent to the remote bug automatically. Read more about Launchpad's inter-bugtracker facilities at https://help.launchpad.net/InterBugTracking. ------------------------------------------------------------------------ On 2019-08-01T17:21:03+00:00 Ssborbis wrote: User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:68.0) Gecko/20100101 Firefox/68.0 Steps to reproduce: browser.search.get() run on ubuntu 18.04 64bit, FF 68.0.1 (debug console, web extension background page, etc ) Actual results: An unexpected error occured Expected results: Promise resolves with array of one-click search engines Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/0 ------------------------------------------------------------------------ On 2019-08-01T20:03:12+00:00 Ehumphries wrote: ~~I need to check on FFx build vs Canonical build, Canonical may be removing it.~~ Edited: reporter did confirm it's a distro issue. Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/1 ------------------------------------------------------------------------ On 2019-08-01T20:14:23+00:00 Ssborbis wrote: Confirmed it only occurs in the Canonical build. Tarball 68.0.1 works as expected Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/2 ------------------------------------------------------------------------ On 2019-08-01T20:25:58+00:00 Ehumphries wrote: Thank you for checking that! This is always frustrating with distro builds, they may disable features for any number of reasons, which can cause things like this. One more step, if you have time. Can you install this sample extension on the Canonical build and see if it works or not? https://github.com/mdn/webextensions-examples/tree/master/menu-search https://developer.mozilla.org/en-US/docs/Mozilla/Add- ons/WebExtensions/Temporary_Installation_in_Firefox Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/3 ------------------------------------------------------------------------ On 2019-08-01T21:12:28+00:00 Ssborbis wrote: Using menu-search as temp add-on Canonical - fail ( Error: An unexpected error occurred .......... undefined ) Official - works as expected Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/4 ------------------------------------------------------------------------ On 2019-08-01T21:20:23+00:00 Mozilla-kaply wrote: This is working with other distros. Maybe something Snap related? Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/5 ------------------------------------------------------------------------ On 2019-08-01T22:22:48+00:00 Ssborbis wrote: (In reply to Mike Kaply [:mkaply] from comment #5) > This is working with other distros. Maybe something Snap related? I had an addon user report this bug on ubuntu 18.04 64bit, confirmed on my vbox install. I'll test snap vs apt Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/6 ------------------------------------------------------------------------ On 2019-08-01T23:27:09+00:00 R-rob-c wrote: Cannot reproduce on Ubuntu 18.04, Firefox 68.0.1 (Canonical, installed via `apt`, Dutch locale). If you are able to reproduce, could you check whether the global JavaScript console (Ctrl-Shift-J) contains any errors? The error could be caused by https://searchfox.org/mozilla-central/rev/b38e3beb658b80e1ed03e0fdf64d225bd4a40327/browser/components/extensions/parent/ext-search.js#56 ... when a search engine is defined with a non-existing icon URL. Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/7 ------------------------------------------------------------------------ On 2019-08-01T23:48:11+00:00 R-rob-c wrote: I went to https://packages.ubuntu.com/bionic/web/firefox and downloaded the source package from http://archive.ubuntu.com/ubuntu/pool/main/f/firefox/firefox_68.0.1+build1-0ubuntu0.18.04.1.debian.tar.xz Inside that package, there is `debian/searchplugins/amazondotcom.xml` (and many other `amazon*.xml` files). They reference `resource://search-plugins/images/amazon.ico`. This file was removed in part 3 of bug 1496075, so any request to fetch that image URL will fail, and hence break `browser.search.get`. Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/8 ------------------------------------------------------------------------ On 2019-08-02T00:04:02+00:00 Ssborbis wrote: (In reply to Rob Wu [:robwu] from comment #8) > I went to https://packages.ubuntu.com/bionic/web/firefox > and downloaded the source package from > http://archive.ubuntu.com/ubuntu/pool/main/f/firefox/firefox_68.0.1+build1-0ubuntu0.18.04.1.debian.tar.xz > > Inside that package, there is `debian/searchplugins/amazondotcom.xml` (and > many other `amazon*.xml` files). They reference > `resource://search-plugins/images/amazon.ico`. > This file was removed in part 3 of bug 1496075, so any request to fetch that > image URL will fail, and hence break `browser.search.get`. Yep, simply deleting the Amazon engine without an icon fixed the issue. I'll let the user know. Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/9 ------------------------------------------------------------------------ On 2019-08-02T00:41:34+00:00 R-rob-c wrote: Created attachment 9082493 Bug 1570700 - Drop support for resource/chrome icons in search engines All built-in engines have migrated from OpenSearch to WebExtensions. WebExtensions do not support resource: or chrome:-URLs in their `favicon_url` field, so the "resource:" and "chrome:" URLs can only be used by external opensearch XML files. These should not rely on internal resources from omni.ja, as the bug shows. So just drop support for "chrome:" and "resource:"-URLs, as we don't need them any more. Current OpenSearch engines that relied on chrome/resource:-URLs can either replace the URL with a data:-URL, or migrate to WebExtensions. Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/10 ------------------------------------------------------------------------ On 2019-08-02T00:48:03+00:00 R-rob-c wrote: (linking to meta bug 1517486, because the proposed patch removes a feature that is not supported by the WebExtension format) Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/11 ------------------------------------------------------------------------ On 2019-08-02T01:07:21+00:00 Ehumphries wrote: Thanks everyone for their help getting to next steps on this bug! Y'all rock. 🙌 Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/12 ------------------------------------------------------------------------ On 2019-08-02T16:21:44+00:00 Mozilla-kaply wrote: I'll fix the URLs in the partner distro repository. Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/13 ------------------------------------------------------------------------ On 2019-08-07T18:27:06+00:00 Pulsebot wrote: Pushed by r...@robwu.nl: https://hg.mozilla.org/integration/autoland/rev/778471f56de7 Drop support for resource/chrome icons in search engines r=mkaply,daleharvey,mixedpuppy Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/14 ------------------------------------------------------------------------ On 2019-08-08T09:39:08+00:00 Ncsoregi wrote: https://hg.mozilla.org/mozilla-central/rev/778471f56de7 Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/15 ------------------------------------------------------------------------ On 2019-08-13T20:40:14+00:00 Ryanvm wrote: Is this something we should consider for uplift or can it ride Fx70 to release? Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/16 ------------------------------------------------------------------------ On 2019-08-13T20:45:07+00:00 Mozilla-kaply wrote: > Is this something we should consider for uplift or can it ride Fx70 to release? Getting it on 69 would be nice, but the Snap problem specifically will be fixed by a change to their distro. Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/17 ------------------------------------------------------------------------ On 2019-08-13T20:51:12+00:00 R-rob-c wrote: Let's ride the train. The number of users that are affected by this is probably small. A work-aroumd for them is to delete the (opensearch) file/engine. Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/18 ------------------------------------------------------------------------ On 2019-08-26T12:34:20+00:00 Mcurtean wrote: Created attachment 9088118 console browser search.png Hello, I've tried to reproduce this on Ubuntu 18.04.2 LTS 64-bit on Firefox Release 68.0.1 by downloading and temporarily adding in about:debugging the "menu-search" extension manifest from comment 3. After which debugging mode was enabled, a Debug performed and also I used the the browser.search.get() function in the console. No error was received; for more details please check the "console browser search" attachment. If manual QA is still needed on this issue, could you please add the "qe+" verify flag and also provided more steps or details in order to test? Thank you Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/22 ------------------------------------------------------------------------ On 2019-08-26T12:45:35+00:00 R-rob-c wrote: There is automated test coverage. Reply at: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/1840597/comments/23 ** Changed in: firefox Status: Unknown => Fix Released ** Changed in: firefox Importance: Unknown => Medium -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1840597 Title: Web Extension API, `browser.search.get` throws error `An unexpected error occurred` To manage notifications about this bug go to: https://bugs.launchpad.net/firefox/+bug/1840597/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
