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

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

Issue 23857004: file_manager: Unify the file task selection logic (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 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 #include "chrome/browser/chromeos/extensions/file_manager/file_tasks.h" 5 #include "chrome/browser/chromeos/extensions/file_manager/file_tasks.h"
6 6
7 #include "apps/launcher.h" 7 #include "apps/launcher.h"
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/prefs/pref_service.h" 9 #include "base/prefs/pref_service.h"
10 #include "base/strings/stringprintf.h" 10 #include "base/strings/stringprintf.h"
(...skipping 447 matching lines...) Expand 10 before | Expand all | Expand 10 after
458 void FindFileBrowserHandlerTasks( 458 void FindFileBrowserHandlerTasks(
459 Profile* profile, 459 Profile* profile,
460 const std::vector<GURL>& file_urls, 460 const std::vector<GURL>& file_urls,
461 const std::vector<base::FilePath>& file_paths, 461 const std::vector<base::FilePath>& file_paths,
462 std::vector<FullTaskDescriptor>* result_list) { 462 std::vector<FullTaskDescriptor>* result_list) {
463 DCHECK(!file_paths.empty()); 463 DCHECK(!file_paths.empty());
464 DCHECK(!file_urls.empty()); 464 DCHECK(!file_urls.empty());
465 DCHECK(result_list); 465 DCHECK(result_list);
466 466
467 file_browser_handlers::FileBrowserHandlerList common_tasks = 467 file_browser_handlers::FileBrowserHandlerList common_tasks =
468 file_browser_handlers::FindCommonFileBrowserHandlers(profile, file_urls); 468 file_browser_handlers::FindFileBrowserHandlers(profile, file_urls);
469 if (common_tasks.empty()) 469 if (common_tasks.empty())
470 return; 470 return;
471 471
472 ExtensionService* service = 472 ExtensionService* service =
473 extensions::ExtensionSystem::Get(profile)->extension_service(); 473 extensions::ExtensionSystem::Get(profile)->extension_service();
474 for (file_browser_handlers::FileBrowserHandlerList::const_iterator iter = 474 for (file_browser_handlers::FileBrowserHandlerList::const_iterator iter =
475 common_tasks.begin(); 475 common_tasks.begin();
476 iter != common_tasks.end(); 476 iter != common_tasks.end();
477 ++iter) { 477 ++iter) {
478 const FileBrowserHandler* handler = *iter; 478 const FileBrowserHandler* handler = *iter;
(...skipping 101 matching lines...) Expand 10 before | Expand all | Expand 10 after
580 if (file_browser_handlers::IsFallbackFileBrowserHandler( 580 if (file_browser_handlers::IsFallbackFileBrowserHandler(
581 task->task_descriptor())) { 581 task->task_descriptor())) {
582 task->set_is_default(true); 582 task->set_is_default(true);
583 return; 583 return;
584 } 584 }
585 } 585 }
586 } 586 }
587 587
588 } // namespace file_tasks 588 } // namespace file_tasks
589 } // namespace file_manager 589 } // namespace file_manager
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698