Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(47)

Unified Diff: chrome/browser/ui/webui/certificate_viewer_ui_browsertest.cc

Issue 7861024: Adds testing infrastructure to web_ui_browsertest to support testing HtmlDialogs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/certificate_viewer_ui_browsertest.cc
diff --git a/chrome/browser/ui/webui/certificate_viewer_ui_browsertest.cc b/chrome/browser/ui/webui/certificate_viewer_ui_browsertest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..45aec64cee2a355dfdc893869f85a0281c88b461
--- /dev/null
+++ b/chrome/browser/ui/webui/certificate_viewer_ui_browsertest.cc
@@ -0,0 +1,46 @@
+// Copyright (c) 2011 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 "base/file_path.h"
+#include "base/time.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_window.h"
+// #include "chrome/browser/ui/webui/chrome_web_ui.h"
flackr 2011/09/09 15:11:04 This will be uncommented before submission. I am w
+#include "chrome/browser/ui/webui/web_ui_browsertest.h"
+#include "chrome/browser/certificate_viewer.h"
+#include "chrome/common/url_constants.h"
+#include "chrome/test/base/ui_test_utils.h"
+#include "net/base/test_certificate_data.h"
+#include "net/base/x509_certificate.h"
+#include <iostream>
Paweł Hajdan Jr. 2011/09/09 22:13:55 Do not #include <iostream>. This results in some s
flackr 2011/09/15 18:41:52 Oops, was using some cerr statements to help debug
+
+class CertificateViewerUITest : public WebUIBrowserTest {
+ public:
+ virtual ~CertificateViewerUITest() {}
+
+ virtual void SetUpOnMainThread() {
+ // Need to enable more WebUI when runtime flag is in use.
+ // ChromeWebUI::SetMoreWebUI(true);
flackr 2011/09/09 15:11:04 This will be uncommented before submission. I am w
+
+ scoped_refptr<net::X509Certificate> google_cert(
+ net::X509Certificate::CreateFromBytes(
+ reinterpret_cast<const char*>(google_der), sizeof(google_der)));
+
+ DCHECK(browser());
+ DCHECK(browser()->window());
+
+ ObserveNextHtmlDialog();
+ ShowCertificateViewer(browser()->window()->GetNativeHandle(), google_cert);
+ WaitForHtmlDialogLoad();
+ AddLibrary(FilePath(FILE_PATH_LITERAL(
+ "certificate_viewer_dialog_test.js")));
+ WebUIBrowserTest::SetUpOnMainThread();
+
+ }
+};
+
+WEB_UI_UNITTEST_F1(CertificateViewerUITest, testDialogURL, new StringValue(
+ chrome::kChromeUICertificateViewerURL));
+WEB_UI_UNITTEST_F(CertificateViewerUITest, testCN);
+WEB_UI_ASYNC_UNITTEST_F(CertificateViewerUITest, testDetails);
« no previous file with comments | « no previous file | chrome/browser/ui/webui/html_dialog_ui.cc » ('j') | chrome/browser/ui/webui/web_ui_browsertest.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698