Followup to bug 499262, those two HAVE_CPP should have been removed too in http://hg.mozilla.org/mozilla-central/rev/e5f0916b32d3. The HAVE_CPP macros are not set by anything in configure/buildsystem, and googling for them yields nothing but that mozilla code.
Created attachment 548554 [details] [diff] [review] Remove leftovers Not even compile tested.. maybe needs a try run ?
Comment on attachment 548554 [details] [diff] [review] Remove leftovers Yes, it should have a try run (build only) before landing.
Try run finished, i can't make any sense from the red linux build opt nor the b-m test failure, i've asked for a rebuild but it doesnt seem to do anything...
After asking for a bunch of single rebuilds, all is green. setting checkin-needed.