[Extensions] Add support for reporting abuse on extension uninstallation

chrome/browser/extensions/extension_uninstall_dialog.cc

 // 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.
 
 #include "chrome/browser/extensions/extension_uninstall_dialog.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_util.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/browser_navigator.h"
 #include "chrome/grit/generated_resources.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/browser/image_loader.h"
 #include "extensions/common/constants.h"
 #include "extensions/common/extension.h"
 #include "extensions/common/extension_icon_set.h"
 #include "extensions/common/extension_resource.h"
+#include "extensions/common/extension_urls.h"
 #include "extensions/common/manifest_handlers/icons_handler.h"
 #include "ui/base/l10n/l10n_util.h"
+#include "ui/base/page_transition_types.h"
+#include "ui/base/window_open_disposition.h"
 #include "ui/gfx/image/image.h"
 #include "ui/gfx/image/image_skia.h"
 
 namespace extensions {
 
 namespace {
 
 // Returns bitmap for the default icon with size equal to the default icon's
 // pixel size under maximal supported scale factor.
 SkBitmap GetDefaultIconBitmapForMaxScaleFactor(bool is_app) {
@@ skipping 84 matching lines @@
   if (triggering_extension_) {
     return l10n_util::GetStringFUTF8(
         IDS_EXTENSION_PROGRAMMATIC_UNINSTALL_PROMPT_HEADING,
         base::UTF8ToUTF16(triggering_extension_->name()),
         base::UTF8ToUTF16(extension_->name()));
   }
   return l10n_util::GetStringFUTF8(IDS_EXTENSION_UNINSTALL_PROMPT_HEADING,
                                    base::UTF8ToUTF16(extension_->name()));
 }
 
+bool ExtensionUninstallDialog::ShouldShowReportAbuseCheckbox() const {
+  // TODO(devlin): Add a field trial for reporting abuse on uninstallation.
+  // See crbug.com/441377.
+  return false;

[Finnur, 2015/03/19 10:43:26]

There was a concern with logged-in vs. not-logged-in. Was the resolution on that
to require non-logged-in users to log in?

[Devlin, 2015/03/19 15:50:42]

I think Jake and I are hoping that we can just drop the logged-in restriction
for the site, but that would be a webstore change.  If that can't happen (maybe
there's a concern about spam reports?), I assume we still want the option for
not-logged-in users to report abuse, so we might just have to have them log in. 
I'll try and follow up in the bug.  Shouldn't block this yet, since this is just
to get the UI ready (doesn't show it at all yet).

+}
+
+void ExtensionUninstallDialog::HandleReportAbuse() {
+  chrome::NavigateParams params(
+      profile_,
+      extension_urls::GetWebstoreReportAbuseUrl(extension_->id()),
+      ui::PAGE_TRANSITION_LINK);
+  params.disposition = NEW_FOREGROUND_TAB;
+  chrome::Navigate(&params);
+}
+
 }  // namespace extensions