MacViews: Enable views based Download recovery dialog behind secondary-ui-md flag.

chrome/browser/download/download_danger_prompt.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_DOWNLOAD_DOWNLOAD_DANGER_PROMPT_H_
 #define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_DANGER_PROMPT_H_
 
 #include "base/callback_forward.h"
 #include "chrome/common/safe_browsing/csd.pb.h"
 
@@ skipping 47 matching lines @@
   // information in this report.
   static void SendSafeBrowsingDownloadReport(
       safe_browsing::ClientSafeBrowsingReportRequest::ReportType report_type,
       bool did_proceed,
       const content::DownloadItem& download);
 
  protected:
   // Records UMA stats for a download danger prompt event.
   static void RecordDownloadDangerPrompt(bool did_proceed,
                                          const content::DownloadItem& download);
+
+ private:
+  // Returns a toolkit-views based download danger prompt.
+  static DownloadDangerPrompt* CreateDownloadDangerPromptViews(

[karandeepb, 2017/01/23 06:51:40]

Should this be in a #ifdef (TOOLKIT_VIEWS)? This file is not included on
android.

Also, I see some functions in browser_dialogs.h which are declared on android
but not defined. There is no linker error because they are not called on
android. Is this considered ok?

+      content::DownloadItem* item,
+      content::WebContents* web_contents,
+      bool show_context,
+      const OnDone& done);
 };
 
 #endif  // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_DANGER_PROMPT_H_