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

Unified Diff: chrome/browser/resources/ssl/blocking.html

Issue 23965003: New SSL blocking screen (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Refactored to share more code Created 7 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/resources/ssl/blocking.html
diff --git a/chrome/browser/resources/ssl/blocking.html b/chrome/browser/resources/ssl/blocking.html
new file mode 100644
index 0000000000000000000000000000000000000000..31ddb63f2b59eadc9b9c12b23b76c71771085c0c
--- /dev/null
+++ b/chrome/browser/resources/ssl/blocking.html
@@ -0,0 +1,49 @@
+<!DOCTYPE html>
+<html i18n-values="dir:textDirection">
+<head>
+ <meta name="viewport" content="width=device-width, initial-scale=1.0,
+ maximum-scale=1.0, user-scalable=no">
+ <title i18n-content="title"></title>
+ <link rel="stylesheet" href="../../../renderer/resources/neterror.css">
+ <link rel="stylesheet" href="blocking.css">
+ <script src="../../../../ui/webui/resources/js/util.js"></script>
+ <script src="../../../renderer/resources/neterror.js"></script>
+ <script src="shared.js"></script>
Patrick Dubroy 2013/09/18 15:01:35 Can you give this a slightly more descriptive name
felt 2013/10/18 05:53:12 Done.
+ <script src="blocking.js"></script>
+</head>
+<body id="t">
+ <div id="main-frame-error">
+ <div id="box">
+ <div id="content-top">
+ <h1>
+ <div>
+ <img class="icon" id="icon-lock">
+ </div>
+ <span i18n-content="headline"></span>
+ </h1>
+ <div class="explanation-par">
Patrick Dubroy 2013/09/18 15:01:35 Looks like this should be a <p>.
felt 2013/10/18 05:53:12 Done.
+ <span i18n-values=".innerHTML:message"></span>
+ </div>
+ <button id="reload-button" i18n-content="reloadMsg"></button>
+ <button id="more-less-button" i18n-content="more"></button>
+ </div>
+ <div id="help-box-outer" class="hidden">
+ <div id="help-box-inner">
+ <div class="ssl-help-box-inner">
+ <h2><span i18n-content="moreTitle"></span></h2>
Patrick Dubroy 2013/09/18 15:01:35 I think you can get rid of the <span> here, and ju
felt 2013/10/18 05:53:12 Done.
+ <span i18n-values=".innerHTML:moreMessage"></p>
Patrick Dubroy 2013/09/18 15:01:35 Mismatched opening and closing tags. I think you c
felt 2013/10/18 05:53:12 Done.
+ </div>
+ <div class="ssl-help-box-inner">
+ <h2><span i18n-content="techTitle"></span></h2>
+ <p i18n-content="failure"></p>
+ <span i18n-content="errorType"></span><br>
Patrick Dubroy 2013/09/18 15:01:35 Why are these <span>, and not <p>?
felt 2013/10/18 05:53:12 The <p> adds too much whitespace between them. I w
Patrick Dubroy 2013/10/21 07:48:50 I see.
+ <span i18n-content="subject"></span><br>
+ <span i18n-content="issuer"></span><br>
+ <span i18n-content="fingerprint"></span>
+ </div>
+ </div>
+ </div>
+ </div>
+ </div>
+</body>
+</html>

Powered by Google App Engine
This is Rietveld 408576698