Reduce number of unnamed-type-template-args violations.

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

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/webui/options/certificate_manager_handler.h"
 
 #include "base/file_util.h"  // for FileAccessProvider
 #include "base/memory/scoped_vector.h"
 #include "base/safe_strerror_posix.h"
 #include "base/string_number_conversions.h"
@@ skipping 448 matching lines @@
 
 void CertificateManagerHandler::GetCATrust(const ListValue* args) {
   net::X509Certificate* cert = CallbackArgsToCert(args);
   if (!cert) {
     web_ui_->CallJavascriptFunction("CertificateEditCaTrustOverlay.dismiss");
     return;
   }
 
   int trust = certificate_manager_model_->cert_db().GetCertTrust(
       cert, net::CA_CERT);
-  using base::FundamentalValue;
-  FundamentalValue ssl_value(bool(trust & net::CertDatabase::TRUSTED_SSL));
-  FundamentalValue email_value(bool(trust & net::CertDatabase::TRUSTED_EMAIL));
-  FundamentalValue obj_sign_value(
-      bool(trust & net::CertDatabase::TRUSTED_OBJ_SIGN));
-  web_ui_->CallJavascriptFunction(
-      "CertificateEditCaTrustOverlay.populateTrust",
+  base::FundamentalValue ssl_value(!!(trust & net::CertDatabase::TRUSTED_SSL));

[Nico, 2011/08/19 20:34:23]

(I remember seeing !! for the first time in some open source project many years
ago, and it took me a month or so to figure out why it's not a no-op. I prefer
the explicit cast that was here before. But I don't feel too strongly about it.)

+  base::FundamentalValue email_value(
+      !!(trust & net::CertDatabase::TRUSTED_EMAIL));
+  base::FundamentalValue obj_sign_value(
+      !!(trust & net::CertDatabase::TRUSTED_OBJ_SIGN));
+  web_ui_->CallJavascriptFunction("CertificateEditCaTrustOverlay.populateTrust",
       ssl_value, email_value, obj_sign_value);
 }
 
 void CertificateManagerHandler::EditCATrust(const ListValue* args) {
   net::X509Certificate* cert = CallbackArgsToCert(args);
   bool fail = !cert;
   bool trust_ssl = false;
   bool trust_email = false;
   bool trust_obj_sign = false;
   fail |= !CallbackArgsToBool(args, 1, &trust_ssl);
@@ skipping 510 matching lines @@
   // TODO(xiyuan): Use async way when underlying supports it.
   base::FundamentalValue ready(cryptohome->Pkcs11IsTpmTokenReady());
   web_ui_->CallJavascriptFunction("CertificateManager.onCheckTpmTokenReady",
                                   ready);
 }
 #endif
 
 gfx::NativeWindow CertificateManagerHandler::GetParentWindow() const {
   return web_ui_->tab_contents()->view()->GetTopLevelNativeWindow();
 }