Fix webui cert viewer showing wrong cert chain on NSS and no chain on OpenSSL.

chrome/browser/ui/certificate_dialogs.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_CERTIFICATE_DIALOGS_H_
 #define CHROME_BROWSER_UI_CERTIFICATE_DIALOGS_H_
 
 #include "net/cert/x509_certificate.h"
 #include "ui/shell_dialogs/select_file_dialog.h"
 
 namespace content {
 class WebContents;
 }
 
 void ShowCertSelectFileDialog(ui::SelectFileDialog* select_file_dialog,
                               ui::SelectFileDialog::Type type,
                               const base::FilePath& suggested_path,
                               gfx::NativeWindow parent,
                               void* params);
 
-void ShowCertExportDialog(content::WebContents* web_contents,
-                          gfx::NativeWindow parent,
-                          net::X509Certificate::OSCertHandle cert);
+// Show a dialog to save the first certificate or the whole chain encompassed by

[msw, 2014/07/11 22:42:45]

nit: I think omitting "the first certificate or" would reduce confusion...
Otherwise, a caller might think they can somehow pass iterators encompassing 10
certs and somehow only have the first one saved. (or maybe I'm just confused)

[mattm, 2014/07/11 22:49:50]

The dialog lets the user choose between saving either the first cert or the
whole chain (both in a few different formats).

[msw, 2014/07/11 23:02:02]

Acknowledged, sorry for my confusion.

+// the iterators.
+void ShowCertExportDialog(
+    content::WebContents* web_contents,
+    gfx::NativeWindow parent,
+    net::X509Certificate::OSCertHandles::iterator certs_begin,
+    net::X509Certificate::OSCertHandles::iterator certs_end);
 
 #endif  // CHROME_BROWSER_UI_CERTIFICATE_DIALOGS_H_