Privacy policy links for extended safe browsing reporting

chrome/browser/ui/views/download/download_feedback_dialog_view.cc

 // Copyright 2014 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 "chrome/browser/ui/views/download/download_feedback_dialog_view.h"
 
 #include "base/metrics/histogram.h"
 #include "base/prefs/pref_service.h"
 #include "base/supports_user_data.h"
+#include "chrome/browser/platform_util.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/views/constrained_window_views.h"
+#include "content/public/browser/page_navigator.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util.h"
+#include "ui/views/controls/link.h"
 #include "ui/views/controls/message_box_view.h"
 #include "ui/views/widget/widget.h"
 
+using content::OpenURLParams;
+
 namespace {
 
 const void* kDialogStatusKey = &kDialogStatusKey;
 
 class DialogStatusData : public base::SupportsUserData::Data {
  public:
   DialogStatusData() : currently_shown_(false) {}
   virtual ~DialogStatusData() {}
   bool currently_shown() const { return currently_shown_; }
   void set_currently_shown(bool shown) { currently_shown_ = shown; }
  private:
   bool currently_shown_;
 };
 
 }  // namespace
 
 // static
 void DownloadFeedbackDialogView::Show(
     gfx::NativeWindow parent_window,
     Profile* profile,
+    content::PageNavigator* navigator,
     const UserDecisionCallback& callback) {
   // This dialog should only be shown if it hasn't been shown before.
   DCHECK(!profile->GetPrefs()->HasPrefPath(
       prefs::kSafeBrowsingExtendedReportingEnabled));
 
   // Only one dialog should be shown at a time, so check to see if another one
   // is open. If another one is open, treat this parallel call as if reporting
   // is disabled (to be conservative).
   DialogStatusData* data =
       static_cast<DialogStatusData*>(profile->GetUserData(kDialogStatusKey));
   if (data == NULL) {
     data = new DialogStatusData();
     profile->SetUserData(kDialogStatusKey, data);
   }
   if (data->currently_shown() == false) {
     data->set_currently_shown(true);
     DownloadFeedbackDialogView* window =
-        new DownloadFeedbackDialogView(profile, callback);
+        new DownloadFeedbackDialogView(profile, navigator, callback);
     CreateBrowserModalDialogViews(window, parent_window)->Show();
   } else {
     callback.Run(false);
   }
 }
 
 DownloadFeedbackDialogView::DownloadFeedbackDialogView(
     Profile* profile,
+    content::PageNavigator* navigator,
     const UserDecisionCallback& callback)
     : profile_(profile),
+      navigator_(navigator),
       callback_(callback),
       explanation_box_view_(new views::MessageBoxView(
           views::MessageBoxView::InitParams(l10n_util::GetStringUTF16(
               IDS_FEEDBACK_SERVICE_DIALOG_EXPLANATION)))),
+      link_view_(new views::Link(l10n_util::GetStringUTF16(
+          IDS_SAFE_BROWSING_PRIVACY_POLICY_PAGE_V2))),
       title_text_(l10n_util::GetStringUTF16(IDS_FEEDBACK_SERVICE_DIALOG_TITLE)),
       ok_button_text_(l10n_util::GetStringUTF16(
           IDS_FEEDBACK_SERVICE_DIALOG_OK_BUTTON_LABEL)),
       cancel_button_text_(l10n_util::GetStringUTF16(
           IDS_FEEDBACK_SERVICE_DIALOG_CANCEL_BUTTON_LABEL)) {
+  link_view_->set_listener(this);
 }
 
 DownloadFeedbackDialogView::~DownloadFeedbackDialogView() {}
 
 int DownloadFeedbackDialogView::GetDefaultDialogButton() const {
   return ui::DIALOG_BUTTON_CANCEL;
 }
 
 base::string16 DownloadFeedbackDialogView::GetDialogButtonLabel(
     ui::DialogButton button) const {
@@ skipping 39 matching lines @@
   return explanation_box_view_->GetWidget();
 }
 
 const views::Widget* DownloadFeedbackDialogView::GetWidget() const {
   return explanation_box_view_->GetWidget();
 }
 
 views::View* DownloadFeedbackDialogView::GetContentsView() {
   return explanation_box_view_;
 }
+
+views::View* DownloadFeedbackDialogView::CreateExtraView() {
+  return link_view_;
+}
+
+void DownloadFeedbackDialogView::LinkClicked(
+    views::Link* source, int event_flags) {
+  WindowOpenDisposition disposition =
+      ui::DispositionFromEventFlags(event_flags);
+  content::OpenURLParams params(
+      GURL(l10n_util::GetStringUTF8(IDS_SAFE_BROWSING_PRIVACY_POLICY_URL)),
+      content::Referrer(),
+      disposition == CURRENT_TAB ? NEW_FOREGROUND_TAB : disposition,
+      content::PAGE_TRANSITION_LINK, false);
+  navigator_->OpenURL(params);

[sky, 2014/06/17 19:11:28]

How do you know navigator_ is still valid by the time you get here? Is this
object guaranteed to outlive navigator_?

[felt, 2014/06/17 20:27:31]

The navigator is from the download shelf, which won't close until this dialog is
dealt with. That means that navigator_ will be valid as long as the
DownloadFeedbackDialogView object exists. I could add a DCHECK or if(navigator_)
if you feel one or the other is appropriate.

[sky, 2014/06/17 23:54:41]

DCHECK(navigator_) wouldn't tell if you the object has since been deleted. If
you say the lifetime is guaranteed that's good enough.

+}