When bug 545955 landed it had completely broken the mac build. We found this to be due to the inclusion of textbox.css into extraCustomizeItems.xul, which NeilAway pointed out may be a bad thing as that's designed to be included in xbl files anyway (unless he was talking about the content version).
In any case, removing it fixed the mac issues, but broke the icons and text on the global search bar on Windows and Linux (i.e. broke = not displayed). As this seemed the better of two evils, we've left the textbox.css removed from the file for now and this bug is for fixing up the search icon and text.
Created attachment 440736 [details]
screenshot of Linux behaviour
Yeah, I added that to get the search icon to show up. The autocomplete widget lacks such niceties, and I wanted to keep search.xml as simple as possible, so I added the button back in extraCustomizeItems.xml instead of in the XBL content for search.xml. My assumption was that accordingly the styling had to come from the document since it would not be part of the anonymous sub-content, but maybe the XBL CSS will affect the children? We also can move stuff back into search.xml...
I pushed a bustage-fix for this, testing the patch on all 3 platforms first:
Can you attach the patch also to this bug, so that other people can review it, and include the necessary changes also in their themes?
Created attachment 442008 [details] [diff] [review]
Use gtk-find for gnomestripe
Same as bug 525971.
Comment on attachment 442008 [details] [diff] [review]
Use gtk-find for gnomestripe
Thx for the patch. r=mkmelin
(Would have been better to put the patch in a follow-up bug though)
(In reply to comment #6)
Thank you for reviewing and a comment.
I'll do so next time I found a bug which is a follow-up one.
Could you add "checkin-needed" to keywords?
I have no permission to check in.
(In reply to comment #7)
> (In reply to comment #6)
> Thank you for reviewing and a comment.
> I'll do so next time I found a bug which is a follow-up one.
> Could you add "checkin-needed" to keywords?
> I have no permission to check in.
No, we're going to need to move this to a follow-up bug. As this bug is already marked fixed for beta 2 and we've already shipped that. Otherwise we're not going to be easily able to track this getting fixed.
As you didn't know about that, I'll file a follow-up bug and transfer the patch there in a few mins.
I'll file new bug for comment #5 shortly.
(In reply to comment #9)
File as bug 563647.
I think this method is gone in comm-central (See Bug Bug 547224).
I got this error.
-- Exception object --
+ message (string) 'this._updateVisibleText is not a function'
+ fileName (string) 'chrome://messenger/content/search.xml'
+ lineNumber (number) 140
+ stack (string) '()@chrome://messenger/content/search.xml:140
+ name (string) 'TypeError'
-- Stack Trace --
(In reply to comment #11)
> I think this method is gone in comm-central (See Bug Bug 547224).
Yes, this is gone from mozilla-central and so affects comm-central but not comm-1.9.2. Please file a new bug for it if it does not already exist.