[Material] Show "Dangerous" for Safe Browsing Security State

chrome/browser/ui/toolbar/chrome_toolbar_model_delegate.cc

 // Copyright 2016 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/toolbar/chrome_toolbar_model_delegate.h"
 
 #include "base/logging.h"
 #include "chrome/browser/autocomplete/chrome_autocomplete_scheme_classifier.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/search/search.h"
@@ skipping 73 matching lines @@
     const {
   scoped_refptr<net::X509Certificate> cert;
   content::NavigationEntry* entry = GetNavigationEntry();
   if (entry) {
     content::CertStore::GetInstance()->RetrieveCert(entry->GetSSL().cert_id,
                                                     &cert);
   }
   return cert;
 }
 
+bool ChromeToolbarModelDelegate::GetFailsMalwareCheck() const {
+  content::WebContents* web_contents = GetActiveWebContents();
+  // If there is no active WebContents (which can happen during toolbar
+  // initialization), assume it did not failed.

[Peter Kasting, 2016/09/13 05:55:59]

Nit: If there's no WebContents, there's no current page, so nothing _can_ have
failed.  So maybe:

  // During toolbar initialization there may be no active WebContents.
  content::WebContents* web_contents = GetActiveWebContents();
  return web_contents &&
         ChromeSecurityStateModelClient::FromWebContents(web_contents)
             ->GetSecurityInfo().fails_malware_check;

[spqchan, 2016/09/13 21:23:41]

Done.

+  if (!web_contents)
+    return false;
+
+  auto* client = ChromeSecurityStateModelClient::FromWebContents(web_contents);
+  return client->GetSecurityInfo().fails_malware_check;
+}
+
 content::NavigationController*
 ChromeToolbarModelDelegate::GetNavigationController() const {
   // This |current_tab| can be null during the initialization of the toolbar
   // during window creation (i.e. before any tabs have been added to the
   // window).
   content::WebContents* current_tab = GetActiveWebContents();
   return current_tab ? &current_tab->GetController() : nullptr;
 }
 
 content::NavigationEntry* ChromeToolbarModelDelegate::GetNavigationEntry()
     const {
   content::NavigationController* controller = GetNavigationController();
   return controller ? controller->GetVisibleEntry() : nullptr;
 }
 
 Profile* ChromeToolbarModelDelegate::GetProfile() const {
   content::NavigationController* controller = GetNavigationController();
   return controller
              ? Profile::FromBrowserContext(controller->GetBrowserContext())
              : nullptr;
 }