Implement chrome.downloads.onDeterminingFilename()

chrome/browser/extensions/api/downloads_internal/downloads_internal_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.
+
+#include "chrome/browser/extensions/api/downloads_internal/downloads_internal_api.h"
+
+#include "chrome/browser/extensions/api/downloads/downloads_api.h"
+#include "chrome/common/extensions/api/downloads_internal.h"
+
+DownloadsInternalAddFilenameDeterminerFunction::
+    DownloadsInternalAddFilenameDeterminerFunction() {}
+
+DownloadsInternalAddFilenameDeterminerFunction::
+    ~DownloadsInternalAddFilenameDeterminerFunction() {}
+
+typedef extensions::api::downloads_internal::AddFilenameDeterminer::Params
+    AddFilenameDeterminerParams;
+
+bool DownloadsInternalAddFilenameDeterminerFunction::RunImpl() {
+  scoped_ptr<AddFilenameDeterminerParams> params(
+      AddFilenameDeterminerParams::Create(*args_));
+  EXTENSION_FUNCTION_VALIDATE(params.get());
+  return ExtensionDownloadsEventRouter::AddFilenameDeterminer(
+      profile(),
+      GetExtension()->id(),
+      params->sub_event_id);
+}
+
+DownloadsInternalRemoveFilenameDeterminerFunction::
+    DownloadsInternalRemoveFilenameDeterminerFunction() {}
+
+DownloadsInternalRemoveFilenameDeterminerFunction::
+    ~DownloadsInternalRemoveFilenameDeterminerFunction() {}
+
+typedef extensions::api::downloads_internal::RemoveFilenameDeterminer::Params
+    RemoveFilenameDeterminerParams;
+
+bool DownloadsInternalRemoveFilenameDeterminerFunction::RunImpl() {

[Randy Smith (Not in Mondays), 2013/01/22 19:43:08]

What is the flow back into the extensions on return from these functions?  I.e.
if a false is returned, what happens?

[benjhayden, 2013/01/23 21:30:10]

The "error_" string is printed in red text in the dev tools console. For the way
the code is now, that string is empty, so a generic error message is printed.

+  scoped_ptr<RemoveFilenameDeterminerParams> params(
+      RemoveFilenameDeterminerParams::Create(*args_));
+  EXTENSION_FUNCTION_VALIDATE(params.get());
+  return ExtensionDownloadsEventRouter::RemoveFilenameDeterminer(
+      profile(),
+      GetExtension()->id(),
+      params->sub_event_id);
+}
+
+DownloadsInternalDetermineFilenameFunction::
+    DownloadsInternalDetermineFilenameFunction() {}
+
+DownloadsInternalDetermineFilenameFunction::
+    ~DownloadsInternalDetermineFilenameFunction() {}
+
+typedef extensions::api::downloads_internal::DetermineFilename::Params
+    DetermineFilenameParams;
+
+bool DownloadsInternalDetermineFilenameFunction::RunImpl() {
+  scoped_ptr<DetermineFilenameParams> params(
+      DetermineFilenameParams::Create(*args_));
+  EXTENSION_FUNCTION_VALIDATE(params.get());
+  FilePath::StringType filename;
+  if (params->filename.get()) {
+    EXTENSION_FUNCTION_VALIDATE(args_->GetString(2, &filename));
+  }
+  return ExtensionDownloadsEventRouter::DetermineFilename(
+      profile(),
+      include_incognito(),
+      GetExtension()->id(),
+      params->sub_event_id,
+      params->download_id,
+      FilePath(filename),
+      params->overwrite);
+}