Added web::GetCommittedItemWithUniqueID function.

ios/web/net/crw_ssl_status_updater.mm

 // 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.
 
 #import "ios/web/net/crw_ssl_status_updater.h"
 
 #import "base/mac/scoped_nsobject.h"
 #import "base/strings/sys_string_conversions.h"
+#include "ios/web/navigation/navigation_manager_util.h"
 #import "ios/web/public/navigation_item.h"
 #import "ios/web/public/navigation_manager.h"
 #include "ios/web/public/ssl_status.h"
 #import "ios/web/web_state/wk_web_view_security_util.h"
 #include "net/cert/x509_certificate.h"
 #include "url/gurl.h"
 
 #if !defined(__has_feature) || !__has_feature(objc_arc)
 #error "This file requires ARC support."
 #endif
@@ skipping 99 matching lines @@
   }
 }
 
 #pragma mark - Private
 
 - (void)updateSSLStatusForItemWithID:(int)navigationItemID
                                trust:(ScopedCFTypeRef<SecTrustRef>)trust
                                 host:(NSString*)host
                    withSecurityStyle:(SecurityStyle)style
                           certStatus:(CertStatus)certStatus {
-  // The searched item almost always be the last one, so walk backward rather
-  // than forward.
-  for (int i = _navigationManager->GetItemCount() - 1; 0 <= i; i--) {
-    web::NavigationItem* item = _navigationManager->GetItemAtIndex(i);
-    if (item->GetUniqueID() != navigationItemID)
-      continue;
+  web::NavigationItem* item =
+      web::GetCommittedItemWithUniqueID(_navigationManager, navigationItemID);
+  if (!item)
+    return;
 
-    // NavigationItem's UniqueID is preserved even after redirects, so
-    // checking that cert and URL match is necessary.
-    scoped_refptr<net::X509Certificate> cert(web::CreateCertFromTrust(trust));
-    std::string GURLHost = base::SysNSStringToUTF8(host);
-    web::SSLStatus& SSLStatus = item->GetSSL();
-    if (item->GetURL().SchemeIsCryptographic() && !!SSLStatus.certificate &&
-        SSLStatus.certificate->Equals(cert.get()) &&
-        item->GetURL().host() == GURLHost) {
-      web::SSLStatus previousSSLStatus = item->GetSSL();
-      SSLStatus.cert_status = certStatus;
-      SSLStatus.security_style = style;
-      if (!previousSSLStatus.Equals(SSLStatus)) {
-        [self didChangeSSLStatusForNavigationItem:item];
-      }
+  // NavigationItem's UniqueID is preserved even after redirects, so
+  // checking that cert and URL match is necessary.
+  scoped_refptr<net::X509Certificate> cert(web::CreateCertFromTrust(trust));
+  std::string GURLHost = base::SysNSStringToUTF8(host);
+  web::SSLStatus& SSLStatus = item->GetSSL();
+  if (item->GetURL().SchemeIsCryptographic() && !!SSLStatus.certificate &&
+      SSLStatus.certificate->Equals(cert.get()) &&
+      item->GetURL().host() == GURLHost) {
+    web::SSLStatus previousSSLStatus = item->GetSSL();
+    SSLStatus.cert_status = certStatus;
+    SSLStatus.security_style = style;
+    if (!previousSSLStatus.Equals(SSLStatus)) {
+      [self didChangeSSLStatusForNavigationItem:item];
     }
-    return;
   }
 }
 
 - (void)scheduleSSLStatusUpdateUsingTrust:(ScopedCFTypeRef<SecTrustRef>)trust
                                      host:(NSString*)host {
   // Use Navigation Item's unique ID to locate requested item after
   // obtaining cert status asynchronously.
   int itemID = _navigationManager->GetLastCommittedItem()->GetUniqueID();
 
   DCHECK(_dataSource);
@@ skipping 12 matching lines @@
 
 - (void)didChangeSSLStatusForNavigationItem:(web::NavigationItem*)navItem {
   if ([_delegate respondsToSelector:
           @selector(SSLStatusUpdater:didChangeSSLStatusForNavigationItem:)]) {
     [_delegate SSLStatusUpdater:self
         didChangeSSLStatusForNavigationItem:navItem];
   }
 }
 
 @end