There have been a few random complaints about this on the forum, but none of the people ever followed through. However, hendy noticed this when he last pulled out his G3 iBook for testing and made the sample page in the URL field. You can see from his screenshot that 2.0* active windows have the inactive-style (pinstriped) bookmark bar instead of the former gradient bar. The last two checkins that touched this code are bug 401340 and bug 417948, but both of them cross-landed, so that makes them unlikely. hendy was investigating, and as of last night, his working theory was maybe the differences in - (BOOL)hasUnifiedToolbarAppearance in BrowserWindow between branch and trunk could be the problem (those were introduced by bug 425519 on the trunk): http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/camino/src/browser/BrowserWindow.mm&rev=1.38&mark=199#199 http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/camino/src/browser/BrowserWindow.mm&rev=18.104.22.168&mark=197#197
31/3/2008 has the gradient, 1/4/2008 doesn't. The only relevent checkin between those dates is indeed Bug 425519.
Created attachment 367528 [details] [diff] [review] Fix v1.0 Looks like the and statement wasn't being correctly converted to a BOOL, as |hasUnifiedToolbarAppearance| was always returning NO. I tried a cast, but that didn't work either. This ternary statement fixes it up, though.
Comment on attachment 367528 [details] [diff] [review] Fix v1.0 Sigh; I should have noticed that was wrong at the time I made the change. r/sr=smorgan.
Landed on cvs trunk.