Bug 663919 Search engine icon missing in urlbar search r=IanN a=IanN
authorNeil Rashbrook <neil@parkwaycc.co.uk>
Sat, 09 Jul 2011 16:34:01 +0100
changeset 8127 4f630202911791bbc9c7aef18ba5ec468278224d
parent 8126 651f261e49c437dd5c8e0f46fd03c0559cab3635
child 8128 3f57dc8c8349525c69d52fa63e1d7b636c0c1f48
push id56
push userphilip.chee@gmail.com
push dateSat, 09 Jul 2011 16:45:08 +0000
treeherdercomm-aurora@4f6302029117 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersIanN, IanN
bugs663919
Bug 663919 Search engine icon missing in urlbar search r=IanN a=IanN
suite/browser/urlbarBindings.xml
--- a/suite/browser/urlbarBindings.xml
+++ b/suite/browser/urlbarBindings.xml
@@ -322,17 +322,18 @@
       <method name="addEngine">
         <parameter name="aName"/>
         <parameter name="aIcon"/>
         <parameter name="aSearchBarUrl"/>
         <body><![CDATA[
           var box = document.createElement("box");
           box.setAttribute("class", "autocomplete-search-engine");
           box.setAttribute("name", aName);
-          box.setAttribute("icon", aIcon);
+          if (aIcon)
+            box.setAttribute("icon", aIcon.spec);
           box.setAttribute("searchBarUrl", aSearchBarUrl);
           box.setAttribute("engineIndex", this.childNodes.length);
           this.mSearchBox.appendChild(box);
         ]]></body>
       </method>
 
       <method name="clearEngines">
         <body><![CDATA[