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

Side by Side Diff: chrome/browser/chromeos/extensions/file_manager/file_browser_handler_api.cc

Issue 257333002: Drive extension functions from ExtensionFunction::Run. The (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix comment Created 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // The file contains the implementation of 5 // The file contains the implementation of
6 // fileBrowserHandlerInternal.selectFile extension function. 6 // fileBrowserHandlerInternal.selectFile extension function.
7 // When invoked, the function does the following: 7 // When invoked, the function does the following:
8 // - Verifies that the extension function was invoked as a result of user 8 // - Verifies that the extension function was invoked as a result of user
9 // gesture. 9 // gesture.
10 // - Display 'save as' dialog using FileSelectorImpl which waits for the user 10 // - Display 'save as' dialog using FileSelectorImpl which waits for the user
(...skipping 266 matching lines...) Expand 10 before | Expand all | Expand 10 after
277 FileSelectorFactory* file_selector_factory, 277 FileSelectorFactory* file_selector_factory,
278 bool enable_user_gesture_check) 278 bool enable_user_gesture_check)
279 : file_selector_factory_(file_selector_factory), 279 : file_selector_factory_(file_selector_factory),
280 user_gesture_check_enabled_(enable_user_gesture_check) { 280 user_gesture_check_enabled_(enable_user_gesture_check) {
281 DCHECK(file_selector_factory); 281 DCHECK(file_selector_factory);
282 } 282 }
283 283
284 FileBrowserHandlerInternalSelectFileFunction:: 284 FileBrowserHandlerInternalSelectFileFunction::
285 ~FileBrowserHandlerInternalSelectFileFunction() {} 285 ~FileBrowserHandlerInternalSelectFileFunction() {}
286 286
287 bool FileBrowserHandlerInternalSelectFileFunction::RunImpl() { 287 bool FileBrowserHandlerInternalSelectFileFunction::RunAsync() {
288 scoped_ptr<SelectFile::Params> params(SelectFile::Params::Create(*args_)); 288 scoped_ptr<SelectFile::Params> params(SelectFile::Params::Create(*args_));
289 289
290 base::FilePath suggested_name(params->selection_params.suggested_name); 290 base::FilePath suggested_name(params->selection_params.suggested_name);
291 std::vector<std::string> allowed_extensions; 291 std::vector<std::string> allowed_extensions;
292 if (params->selection_params.allowed_file_extensions.get()) 292 if (params->selection_params.allowed_file_extensions.get())
293 allowed_extensions = *params->selection_params.allowed_file_extensions; 293 allowed_extensions = *params->selection_params.allowed_file_extensions;
294 294
295 if (!user_gesture() && user_gesture_check_enabled_) { 295 if (!user_gesture() && user_gesture_check_enabled_) {
296 SetError(kNoUserGestureError); 296 SetError(kNoUserGestureError);
297 return false; 297 return false;
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
365 result->entry->file_system_name = entry_definition.file_system_name; 365 result->entry->file_system_name = entry_definition.file_system_name;
366 result->entry->file_system_root = entry_definition.file_system_root_url; 366 result->entry->file_system_root = entry_definition.file_system_root_url;
367 result->entry->file_full_path = 367 result->entry->file_full_path =
368 "/" + entry_definition.full_path.AsUTF8Unsafe(); 368 "/" + entry_definition.full_path.AsUTF8Unsafe();
369 result->entry->file_is_directory = entry_definition.is_directory; 369 result->entry->file_is_directory = entry_definition.is_directory;
370 } 370 }
371 371
372 results_ = SelectFile::Results::Create(*result); 372 results_ = SelectFile::Results::Create(*result);
373 SendResponse(true); 373 SendResponse(true);
374 } 374 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698