Removed implicit cast from size_t to int, prevent an unlikely but theoretically possible integer ov…

chrome/browser/ssl/ssl_error_info.h

Index: chrome/browser/ssl/ssl_error_info.h
diff --git a/chrome/browser/ssl/ssl_error_info.h b/chrome/browser/ssl/ssl_error_info.h
index 2713374cf232bdfe3b1a4c74e118344523514e54..493f02350b52aa99c4df4ed59c533b86d115181d 100644
--- a/chrome/browser/ssl/ssl_error_info.h
+++ b/chrome/browser/ssl/ssl_error_info.h
@@ -53,10 +53,10 @@ class SSLErrorInfo {
   // |cert_status|.  Returns the number of errors found.
   // Callers only interested in the error count can pass NULL for |errors|.
   // TODO(wtc): Document |cert_id| and |url| arguments.
-  static int GetErrorsForCertStatus(int cert_id,
-                                    net::CertStatus cert_status,
-                                    const GURL& url,
-                                    std::vector<SSLErrorInfo>* errors);
+  static size_t GetErrorsForCertStatus(int cert_id,

[meacer, 2015/03/23 21:54:30]

I was going to suggest renaming this to GetNumErrorsForCertStatus, but it looks
like the return value isn't used at all:
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...

Perhaps you could just return a bool and DCHECK() it, or just use void?

+                                       net::CertStatus cert_status,
+                                       const GURL& url,
+                                       std::vector<SSLErrorInfo>* errors);
 
   // A description of the error.
   const base::string16& details() const { return details_; }