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

Unified Diff: chrome/browser/extensions/component_loader.cc

Issue 210623003: [VideoPlayer] Remove the old video player from Files.app (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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/component_loader.cc
diff --git a/chrome/browser/extensions/component_loader.cc b/chrome/browser/extensions/component_loader.cc
index e361d6800a52399b80379485365ea82d7253cabf..b4025c9102aa900dbb2318d4c8adbeb6394d5706 100644
--- a/chrome/browser/extensions/component_loader.cc
+++ b/chrome/browser/extensions/component_loader.cc
@@ -440,8 +440,8 @@ void ComponentLoader::AddDefaultComponentExtensionsForKioskMode(
return;
// Component extensions needed for kiosk apps.
- AddFileManagerExtension();
AddVideoPlayerExtension();
+ AddFileManagerExtension();
// Add virtual keyboard.
AddKeyboardApp();
@@ -471,8 +471,11 @@ void ComponentLoader::AddDefaultComponentExtensionsWithBackgroundPages(
#endif
if (!skip_session_components) {
- AddFileManagerExtension();
+ // Video player should be avove the file manager, since the video player
+ // is the default app for video files by default.
hashimoto 2014/03/25 07:25:19 I'm a bit confused here. Does this mean FileManage
yoshiki 2014/03/25 07:45:46 Yes, not only the video player but the galley can
AddVideoPlayerExtension();
+ AddFileManagerExtension();
+
AddHangoutServicesExtension();
AddHotwordHelperExtension();
AddImageLoaderExtension();

Powered by Google App Engine
This is Rietveld 408576698