Support installing web extensions with applications.gecko.id

rb/lib/selenium/webdriver/firefox/extension.rb

@@ -87,6 +87,14 @@ def read_id_from_manifest_json(directory)
           return unless File.exist?(manifest_path)
 
           manifest = JSON.parse(File.read(manifest_path))
+          applications_gecko_id(manifest) || name_and_version(manifest)
+        end
+
+        def applications_gecko_id(manifest)
+          manifest.dig('applications', 'gecko', 'id')&.strip
+        end
+
+        def name_and_version(manifest)
           [manifest['name'].delete(' '), manifest['version']].join('@')
         end
       end # Extension

rb/spec/integration/selenium/webdriver/firefox/profile_spec.rb

@@ -147,13 +147,20 @@ def read_generated_prefs(from = nil)
           expect(Dir.exist?(extension_directory)).to eq(true)
         end
 
-        it 'can install web extension' do
+        it 'can install web extension without id' do
           mooltipass = File.expand_path('../../../../../../third_party/firebug/mooltipass-1.1.87.xpi', __dir__)
           profile.add_extension(mooltipass)
           extension_directory = File.expand_path('extensions/[email protected]', profile.layout_on_disk)
           expect(Dir.exist?(extension_directory)).to eq(true)
         end
 
+        it 'can install web extension with id' do
+          ext = File.expand_path('../../../../../../third_party/firebug/favourite_colour-1.1-an+fx.xpi', __dir__)
+          profile.add_extension(ext)
+          extension_directory = File.expand_path('extensions/[email protected]', profile.layout_on_disk)
+          expect(Dir.exist?(extension_directory)).to eq(true)
+        end
+
         describe 'with browser' do
           before do
             profile['browser.startup.homepage'] = url_for('simpleTest.html')