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 7935 402abcbf2da0723db2b74de19de09d055a8ca3ab
parent 7934 c8aad432778ab28e3c89108e58abce1cd4f6de28
child 7936 98972adac7ef33b21d99fc55e25a065e9d10dad4
push id39
push userphilip.chee@gmail.com
push dateSat, 09 Jul 2011 16:47:28 +0000
treeherdercomm-beta@402abcbf2da0 [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[