[HBD] Intercept navigation to Flash download page

chrome/browser/plugins/flash_download_interception.cc

+// 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.
+
+#include "chrome/browser/plugins/flash_download_interception.h"
+
+#include <string>
+
+#include "base/bind.h"
+#include "base/memory/ptr_util.h"
+#include "chrome/common/chrome_features.h"
+#include "components/navigation_interception/intercept_navigation_throttle.h"
+#include "components/navigation_interception/navigation_params.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/navigation_handle.h"
+#include "content/public/browser/web_contents.h"
+
+using content::BrowserThread;
+using content::WebContents;

[Lei Zhang, 2016/08/25 00:55:06]

not needed

[trizzofo, 2016/08/25 01:50:36]

Done.

+using content::NavigationHandle;
+using content::NavigationThrottle;
+
+namespace {
+
+const char kFlashDownloadURL[] = "get.adobe.com/flashplayer";
+
+bool ShouldInterceptNavigation(

[Lei Zhang, 2016/08/25 00:55:06]

Does this need a comment or a TODO?

[trizzofo, 2016/08/25 01:50:36]

Added a TODO to body.

+    content::WebContents* source,
+    const navigation_interception::NavigationParams& params) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  return true;
+}
+
+}  // namespace
+
+// static
+std::unique_ptr<NavigationThrottle>
+FlashDownloadInterception::MaybeCreateThrottleFor(NavigationHandle* handle) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+
+  if (!base::FeatureList::IsEnabled(features::kPreferHtmlOverPlugins))
+    return nullptr;
+
+  if (!handle->HasUserGesture())
+    return nullptr;
+
+  if (handle->GetURL().GetContent().compare(0, strlen(kFlashDownloadURL),

[Lei Zhang, 2016/08/25 00:55:07]

Use base::StartsWith() ?

[trizzofo, 2016/08/25 01:50:36]

Done.

+                                            kFlashDownloadURL)) {
+    return nullptr;
+  }
+
+  return base::MakeUnique<navigation_interception::InterceptNavigationThrottle>(
+      handle, base::Bind(&ShouldInterceptNavigation), true);
+}