Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3940)

Unified Diff: chrome/browser/extensions/api/media_gallery/media_gallery_api.cc

Issue 9358076: Initial extension bindings for Media Gallery API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/media_gallery/media_gallery_api.cc
diff --git a/chrome/browser/extensions/api/media_gallery/media_gallery_api.cc b/chrome/browser/extensions/api/media_gallery/media_gallery_api.cc
new file mode 100644
index 0000000000000000000000000000000000000000..af45c6e996ccdb678aea49d3ad861461a20c9d4d
--- /dev/null
+++ b/chrome/browser/extensions/api/media_gallery/media_gallery_api.cc
@@ -0,0 +1,47 @@
+// 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 Cookies API.
+
+#include "chrome/browser/extensions/api/media_gallery/media_gallery_api.h"
+
+#include "base/values.h"
+
+namespace extensions {
+
jstritar 2012/02/16 21:34:18 Can you add an apitest for these methods? It will
vandebo (ex-Chrome) 2012/03/19 23:09:52 Done. But I had to change the "Blob" type argument
+GetMediaGalleriesFunction::~GetMediaGalleriesFunction() {}
+
+bool GetMediaGalleriesFunction::RunImpl() {
+ // TODO(vandebo) Get the list of galleries.
+ result_.reset(new ListValue);
+ SendResponse(true);
jstritar 2012/02/16 21:34:18 You won't need these SendResponses once they're Sy
vandebo (ex-Chrome) 2012/03/19 23:09:52 Done.
+ return true;
+}
+
+OpenMediaGalleryManagerFunction::~OpenMediaGalleryManagerFunction() {}
+
+bool OpenMediaGalleryManagerFunction::RunImpl() {
+ // TODO(vandebo) Open a new tab/ui surface with config UI.
+ return true;
+}
+
+AssembleMediaFileFunction::~AssembleMediaFileFunction() {}
+
+bool AssembleMediaFileFunction::RunImpl() {
+ // TODO(vandebo) Update the metadata and return the new file.
+ result_.reset(Value::CreateNullValue());
+ SendResponse(true);
+ return true;
+}
+
+ParseMediaFileMetadataFunction::~ParseMediaFileMetadataFunction() {}
+
+bool ParseMediaFileMetadataFunction::RunImpl() {
+ // TODO(vandebo) Try to parse the file.
+ result_.reset(Value::CreateNullValue());
+ SendResponse(true);
+ return true;
+}
+
+} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698