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

Unified Diff: chrome/browser/extensions/api/discovery/discovery_api.cc

Issue 10391034: Scaffolding for an experimental discovery API letting users inject links in the recommended pane of… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Answered review comments from PatchSet 4. Created 8 years, 7 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/discovery/discovery_api.cc
diff --git a/chrome/browser/extensions/api/discovery/discovery_api.cc b/chrome/browser/extensions/api/discovery/discovery_api.cc
new file mode 100644
index 0000000000000000000000000000000000000000..4392f2a8eb82cc83e666dd1ee70fff9ffadb8e3d
--- /dev/null
+++ b/chrome/browser/extensions/api/discovery/discovery_api.cc
@@ -0,0 +1,67 @@
+// 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/api/discovery/discovery_api.h"
+
+#include "base/values.h"
+#include "chrome/browser/extensions/api/discovery/suggested_link.h"
+#include "chrome/browser/extensions/api/discovery/suggested_links_registry.h"
+#include "chrome/browser/extensions/api/discovery/suggested_links_registry_factory.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/common/extensions/api/experimental_discovery.h"
+#include "chrome/common/extensions/extension_error_utils.h"
+
+namespace discovery = extensions::api::experimental_discovery;
+
+namespace {
+const char kInvalidScore[] = "Invalid score, must be between 0 and 1.";
+} // namespace
+
+namespace extensions {
+
+bool DiscoverySuggestFunction::RunImpl() {
+ scoped_ptr<discovery::Suggest::Params> params(
+ discovery::Suggest::Params::Create(*args_));
+ EXTENSION_FUNCTION_VALIDATE(params.get());
+
+ double score = 1.0;
+ if (params->details.score != NULL) {
+ score = *params->details.score;
+ if (score < 0.0 || score > 1.0) {
+ error_ = kInvalidScore;
+ return false;
+ }
+ }
+
+ extensions::SuggestedLinksRegistry* registry =
+ extensions::SuggestedLinksRegistryFactory::GetForProfile(profile());
+ scoped_ptr<extensions::SuggestedLink> suggested_link(
+ new extensions::SuggestedLink(params->details.link_url,
+ params->details.link_text, score));
+ registry->Add(extension_id(), suggested_link.Pass());
+ return true;
+}
+
+bool DiscoveryRemoveSuggestionFunction::RunImpl() {
+ scoped_ptr<discovery::RemoveSuggestion::Params> params(
+ discovery::RemoveSuggestion::Params::Create(*args_));
+ EXTENSION_FUNCTION_VALIDATE(params.get());
+
+ extensions::SuggestedLinksRegistry* registry =
+ extensions::SuggestedLinksRegistryFactory::GetForProfile(profile());
+ registry->Remove(extension_id(), params->link_url);
+
+ return true;
+}
+
+bool DiscoveryClearAllSuggestionsFunction::RunImpl() {
+ extensions::SuggestedLinksRegistry* registry =
+ extensions::SuggestedLinksRegistryFactory::GetForProfile(profile());
+ registry->ClearAll(extension_id());
+
+ return true;
+}
+
+} // namespace extensions
+
« no previous file with comments | « chrome/browser/extensions/api/discovery/discovery_api.h ('k') | chrome/browser/extensions/api/discovery/suggested_link.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698