Rename SafeBrowsingErrorUI to SafeBrowsingLoudErrorUI

components/security_interstitials/core/safe_browsing_loud_error_ui.h

Index: components/security_interstitials/core/safe_browsing_loud_error_ui.h
diff --git a/components/security_interstitials/core/safe_browsing_loud_error_ui.h b/components/security_interstitials/core/safe_browsing_loud_error_ui.h
new file mode 100644
index 0000000000000000000000000000000000000000..4150e6a68b7db02f9cef432a517e50cff652d093
--- /dev/null
+++ b/components/security_interstitials/core/safe_browsing_loud_error_ui.h
@@ -0,0 +1,50 @@
+// 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.
+
+#ifndef COMPONENTS_SECURITY_INTERSTITIALS_CORE_SAFE_BROWSING_LOUD_ERROR_UI_H_
+#define COMPONENTS_SECURITY_INTERSTITIALS_CORE_SAFE_BROWSING_LOUD_ERROR_UI_H_
+
+#include "base/macros.h"
+#include "base/time/time.h"
+#include "base/values.h"
+#include "components/security_interstitials/core/base_safe_browsing_error_ui.h"
+#include "components/security_interstitials/core/controller_client.h"
+#include "url/gurl.h"
+
+namespace security_interstitials {
+
+// This class displays UI for Safe Browsing errors that block page loads. This
+// class is purely about visual display; it does not do any error-handling logic
+// to determine what type of error should be displayed when.

[Nathan Parker, 2017/05/12 17:02:47]

Add a comment as to what "Loud" means (big red interstitial that's fully
interactive, and is used on desktop/android but only parts of WebView)

[edwardjung, 2017/05/12 17:45:53]

Done.

+class SafeBrowsingLoudErrorUI
+    : public security_interstitials::BaseSafeBrowsingErrorUI {
+ public:
+  SafeBrowsingLoudErrorUI(
+      const GURL& request_url,
+      const GURL& main_frame_url,
+      BaseSafeBrowsingErrorUI::SBInterstitialReason reason,
+      const BaseSafeBrowsingErrorUI::SBErrorDisplayOptions& display_options,
+      const std::string& app_locale,
+      const base::Time& time_triggered,
+      ControllerClient* controller);
+
+  ~SafeBrowsingLoudErrorUI() override;
+
+  void PopulateStringsForHTML(base::DictionaryValue* load_time_data) override;

[Nathan Parker, 2017/05/12 17:02:47]

// Implement BaseSafeBrowsingErrorUI

[edwardjung, 2017/05/12 17:45:53]

Done.

+  void HandleCommand(SecurityInterstitialCommands command) override;
+
+ private:
+  // Fills the passed dictionary with the values to be passed to the template
+  // when creating the HTML.
+  void PopulateExtendedReportingOption(base::DictionaryValue* load_time_data);
+  void PopulateMalwareLoadTimeData(base::DictionaryValue* load_time_data);
+  void PopulateHarmfulLoadTimeData(base::DictionaryValue* load_time_data);
+  void PopulatePhishingLoadTimeData(base::DictionaryValue* load_time_data);
+
+  DISALLOW_COPY_AND_ASSIGN(SafeBrowsingLoudErrorUI);
+};
+
+}  // security_interstitials
+
+#endif  // COMPONENTS_SECURITY_INTERSTITIALS_CORE_SAFE_BROWSING_LOUD_ERROR_UI_H_