Cleanup: avoid foo ? true : false, part 1.

chrome/browser/external_protocol/external_protocol_handler.cc

 // Copyright (c) 2012 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/external_protocol/external_protocol_handler.h"
 
 #include <set>
 
 #include "base/bind.h"
 #include "base/logging.h"
@@ skipping 215 matching lines @@
   // Set in the stored prefs.
   // TODO(pkasting): http://b/1119651 This kind of thing should go in the
   // preferences on the profile, not in the local state.
   PrefService* pref = g_browser_process->local_state();
   if (pref) {  // May be NULL during testing.
     DictionaryPrefUpdate update_excluded_schemas(pref, prefs::kExcludedSchemes);
 
     if (state == UNKNOWN) {
       update_excluded_schemas->Remove(scheme, NULL);
     } else {
-      update_excluded_schemas->SetBoolean(scheme,
-                                          state == BLOCK ? true : false);
+      update_excluded_schemas->SetBoolean(scheme, (state == BLOCK));
     }
   }
 }
 
 // static
 void ExternalProtocolHandler::LaunchUrlWithDelegate(const GURL& url,
                                                     int render_process_host_id,
                                                     int tab_contents_id,
                                                     Delegate* delegate) {
   DCHECK_EQ(MessageLoop::TYPE_UI, MessageLoop::current()->type());
@@ skipping 48 matching lines @@
 // static
 void ExternalProtocolHandler::RegisterPrefs(PrefService* prefs) {
   prefs->RegisterDictionaryPref(prefs::kExcludedSchemes);
 }
 
 // static
 void ExternalProtocolHandler::PermitLaunchUrl() {
   DCHECK_EQ(MessageLoop::TYPE_UI, MessageLoop::current()->type());
   g_accept_requests = true;
 }