Media Galleries: Add a dropPermissionForMediaFileSystem() API.

chrome/browser/extensions/api/media_galleries/media_galleries_api.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.
 
 // Implements the Chrome Extensions Media Galleries API.
 
 #include "chrome/browser/extensions/api/media_galleries/media_galleries_api.h"
 
 #include <set>
 #include <string>
@@ skipping 44 matching lines @@
 #include "ui/base/l10n/l10n_util.h"
 
 using content::WebContents;
 using storage_monitor::MediaStorageUtil;
 using storage_monitor::StorageInfo;
 using web_modal::WebContentsModalDialogManager;
 
 namespace extensions {
 
 namespace MediaGalleries = api::media_galleries;
+namespace DropPermissionForMediaFileSystem =
+    MediaGalleries::DropPermissionForMediaFileSystem;
 namespace GetMediaFileSystems = MediaGalleries::GetMediaFileSystems;
 
 namespace {
 
 const char kDisallowedByPolicy[] =
     "Media Galleries API is disallowed by policy: ";
 const char kMissingEventListener[] =
     "Missing event listener registration.";
 const char kNoScanPermission[] =
     "No permission to scan.";
@@ skipping 561 matching lines @@
   if (!render_view_host()) {
     cb.Run(std::vector<MediaFileSystemInfo>());
     return;
   }
   MediaFileSystemRegistry* registry = media_file_system_registry();
   DCHECK(registry->GetPreferences(GetProfile())->IsInitialized());
   registry->GetMediaFileSystemsForExtension(
       render_view_host(), GetExtension(), cb);
 }
 
+MediaGalleriesDropPermissionForMediaFileSystemFunction::
+    ~MediaGalleriesDropPermissionForMediaFileSystemFunction() {}
+
+bool MediaGalleriesDropPermissionForMediaFileSystemFunction::RunImpl() {
+  media_galleries::UsageCount(
+      media_galleries::DROP_PERMISSION_FOR_MEDIA_FILE_SYSTEM);
+
+  scoped_ptr<base::ListValue> default_results(new base::ListValue());
+  default_results->AppendString(base::string16());
+  default_results->AppendBoolean(false);
+  SetResult(default_results.release());
+
+  scoped_ptr<DropPermissionForMediaFileSystem::Params> params(
+      DropPermissionForMediaFileSystem::Params::Create(*args_));
+  EXTENSION_FUNCTION_VALIDATE(params.get());
+  MediaGalleryPrefId pref_id;
+  if (!base::StringToUint64(params->gallery_id, &pref_id))
+    return false;
+
+  // The name of this function is too darn long!
+#define DROP_PERMISSION_FUNCTION \
+  &MediaGalleriesDropPermissionForMediaFileSystemFunction::OnPreferencesInit

[vandebo (ex-Chrome), 2014/04/08 14:58:45]

Would wrapping at the :: be better than a macro?

[Lei Zhang, 2014/04/08 17:10:18]

Done.

+  base::Closure callback = base::Bind(DROP_PERMISSION_FUNCTION, this, pref_id);
+#undef DROP_PERMISSION_FUNCTION
+  return Setup(GetProfile(), &error_, callback);

[vandebo (ex-Chrome), 2014/04/08 14:58:45]

Consider putting this in the conditional on line 664 so that you can only set
the result if returning false.

[Lei Zhang, 2014/04/08 17:10:18]

Code shuffled.

+}
+
+void MediaGalleriesDropPermissionForMediaFileSystemFunction::OnPreferencesInit(
+    MediaGalleryPrefId pref_id) {
+  MediaGalleriesPreferences* preferences =
+      media_file_system_registry()->GetPreferences(GetProfile());
+  bool dropped = preferences->SetGalleryPermissionForExtension(
+      *GetExtension(), pref_id, false);
+  scoped_ptr<base::ListValue> results(new base::ListValue());
+  results->AppendString(base::Uint64ToString(pref_id));
+  results->AppendBoolean(dropped);
+  SetResult(results.release());
+  SendResponse(true);
+}
+
 MediaGalleriesStartMediaScanFunction::~MediaGalleriesStartMediaScanFunction() {}
 
 bool MediaGalleriesStartMediaScanFunction::RunImpl() {
   media_galleries::UsageCount(media_galleries::START_MEDIA_SCAN);
   if (!CheckScanPermission(GetExtension(), &error_)) {
     MediaGalleriesEventRouter::Get(GetProfile())->OnScanError(
         GetExtension()->id());
     return false;
   }
   return Setup(GetProfile(), &error_, base::Bind(
@@ skipping 179 matching lines @@
   if (!parse_success) {
     SendResponse(false);
     return;
   }
 
   SetResult(metadata_dictionary->DeepCopy());
   SendResponse(true);
 }
 
 }  // namespace extensions