Condition the ShowManageSSLCertificates button on operating system.

chrome/browser/ui/webui/options/browser_options_handler.cc

Index: chrome/browser/ui/webui/options/browser_options_handler.cc
diff --git a/chrome/browser/ui/webui/options/browser_options_handler.cc b/chrome/browser/ui/webui/options/browser_options_handler.cc
index 4af7dda304d8210edb1788f4cc73abbabd97e667..0f92504329aec758798d62a604a72c08af98daab 100644
--- a/chrome/browser/ui/webui/options/browser_options_handler.cc
+++ b/chrome/browser/ui/webui/options/browser_options_handler.cc
@@ -701,7 +701,7 @@ void BrowserOptionsHandler::RegisterMessages() {
       "defaultZoomFactorAction",
       base::Bind(&BrowserOptionsHandler::HandleDefaultZoomFactor,
                  base::Unretained(this)));
-#if !defined(USE_NSS) && !defined(USE_OPENSSL)

[Dan Beam, 2014/09/24 18:41:08]

why not add a USE_BORINGSSL?

[davidben, 2014/09/24 18:50:15]

The problem isn't OpenSSL -> BoringSSL (all references in code still say OpenSSL
right now; BoringSSL is our in-house OpenSSL). It's Mac going NSS ->
OpenSSL/BoringSSL. These flags are kinda awkward and really should be renamed.
They match the GYP flags which are similarly confused. I believe GN's flags were
newer so they get it right. There's:

USE_NSS = port uses NSS for /certificates/. This is currently CrOS and Linux.
USE_OPENSSL = port uses OpenSSL for /all crypto/. If undefined, general crypto
is NSS. This used to be only Android. Now it's Mac as well. Eventually it will
be everything and there won't be any NSS anywhere.
USE_OPENSSL_CERTS = port uses OpenSSL for /certificates/. This is just Android
and the unfinished Linux+OpenSSL port.

So USE_NSS is really USE_NSS_CERTS and what should be USE_NSS is
!defined(USE_OPENSSL).

One fix would be to replace USE_OPENSSL with USE_OPENSSL_CERTS. I can flip it
that way if you prefer. That would work as well. (This code should be built on
Mac.) I opted to flip it to the OS ones because this is the other half to the
showManageSSLCertificates message, and that one is keyed by OS.

https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/res...

+#if defined(OS_WIN) || defined(OS_MACOSX)
   web_ui()->RegisterMessageCallback(
       "showManageSSLCertificates",
       base::Bind(&BrowserOptionsHandler::ShowManageSSLCertificates,
@@ -1577,7 +1577,7 @@ void BrowserOptionsHandler::ShowNetworkProxySettings(
 }
 #endif
 
-#if !defined(USE_NSS) && !defined(USE_OPENSSL)
+#if defined(OS_WIN) || defined(OS_MACOSX)
 void BrowserOptionsHandler::ShowManageSSLCertificates(
     const base::ListValue* args) {
   content::RecordAction(UserMetricsAction("Options_ManageSSLCertificates"));