Revert "Run more FilesApp tests with DriveFS enabled."

This reverts commit 0780e76b.

Reason for revert:

Findit (https://goo.gl/kROfz5) identified CL at revision 596048 as the
culprit for flakes in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/flake/flake-culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyQwsSDEZsYWtlQ3VscHJpdCIxY2hyb21pdW0vMDc4MGU3NmIwMzg4MmQ5MmVjOGI5YjczNDIyNzgzMzA5YzZjZWM1Nww

Sample Failed Build: https://ci.chromium.org/buildbot/chromium.memory/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/29332

Sample Failed Step: viz_browser_tests

Sample Flaky Test: FolderShortcuts/FilesAppBrowserTest.Test/traverseFolderShortcuts_DriveFs

Original change's description:
> Run more FilesApp tests with DriveFS enabled.
> 
> Run with DriveFS enabled:
> - Drive-related context_menu.js tests.
> - Drive-related metadata.js tests.
> - folder_shortcuts.js tests.
> - fileDisplayDrive in tablet mode with DriveFS enabled.
> 
> In order to make those tests pass:
> - Pass capabilities metadata to FakeDriveFs in SetMetadata() and return
>   it in GetMetadata().
> - Populate croppedThumbnailUrl so metadata cache doesn't retry fetching
>   metadata, trying to get a value for it.
> - Remove two unused and incomplete tests in context_menu.js.
> - Only perform early-init for the downloads volume for the "user"
>   profile.
> 
> Bug: 891197
> Change-Id: I8ddcbd75b74258b6988ad5c70c4e70c707d18f70
> Reviewed-on: https://chromium-review.googlesource.com/c/1255086
> Reviewed-by: Noel Gordon <noel@chromium.org>
> Commit-Queue: Sam McNally <sammc@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#596048}

Change-Id: I918211f80347a93388965b5278c2e3b55d4bfca5
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 891197, 891641
Reviewed-on: https://chromium-review.googlesource.com/c/1258616
Cr-Commit-Position: refs/heads/master@{#596172}

chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc

@@ -707,8 +707,6 @@ class SingleEntryPropertiesGetterForDriveFs {
     if (metadata->type != drivefs::mojom::FileMetadata::Type::kDirectory) {
       properties_->thumbnail_url = std::make_unique<std::string>(
           base::StrCat({"drivefs:", file_system_url_.ToGURL().spec()}));
-      properties_->cropped_thumbnail_url =
-          std::make_unique<std::string>(*properties_->thumbnail_url);
     }
 
     CompleteGetEntryProperties(drive::FILE_ERROR_OK);

chrome/browser/chromeos/file_manager/file_manager_browsertest.cc

@@ -180,7 +180,6 @@ WRAPPED_INSTANTIATE_TEST_CASE_P(
         TestCase("fileDisplayDrive"),
         TestCase("fileDisplayDrive").TabletMode(),
         TestCase("fileDisplayDrive").EnableDriveFs(),
-        TestCase("fileDisplayDrive").TabletMode().EnableDriveFs(),
         TestCase("fileDisplayDriveOffline").Offline().EnableDriveFs(),
         TestCase("fileDisplayDriveOnline").EnableDriveFs(),
         TestCase("fileDisplayDriveOnline"),
@@ -323,48 +322,6 @@ WRAPPED_INSTANTIATE_TEST_CASE_P(
                       TestCase("checkPasteDisabledForReadOnlyFolderInTree"),
                       TestCase("checkContextMenuForTeamDriveRoot")));
 
-WRAPPED_INSTANTIATE_TEST_CASE_P(
-    ContextMenu2, /* context_menu.js */
-    FilesAppBrowserTest,
-    ::testing::Values(
-        TestCase("checkDeleteEnabledForReadWriteFile").EnableDriveFs(),
-        TestCase("checkDeleteDisabledForReadOnlyDocument").EnableDriveFs(),
-        TestCase("checkDeleteDisabledForReadOnlyFile").EnableDriveFs(),
-        TestCase("checkDeleteDisabledForReadOnlyFolder").EnableDriveFs(),
-        TestCase("checkRenameEnabledForReadWriteFile").EnableDriveFs(),
-        TestCase("checkRenameDisabledForReadOnlyDocument").EnableDriveFs(),
-        TestCase("checkRenameDisabledForReadOnlyFile").EnableDriveFs(),
-        TestCase("checkRenameDisabledForReadOnlyFolder").EnableDriveFs(),
-        TestCase("checkShareEnabledForReadWriteFile").EnableDriveFs(),
-        TestCase("checkShareEnabledForReadOnlyDocument").EnableDriveFs(),
-        TestCase("checkShareDisabledForStrictReadOnlyDocument").EnableDriveFs(),
-        TestCase("checkShareEnabledForReadOnlyFile").EnableDriveFs(),
-        TestCase("checkShareEnabledForReadOnlyFolder").EnableDriveFs(),
-        TestCase("checkCopyEnabledForReadWriteFile").EnableDriveFs(),
-        TestCase("checkCopyEnabledForReadOnlyDocument").EnableDriveFs(),
-        TestCase("checkCopyDisabledForStrictReadOnlyDocument").EnableDriveFs(),
-        TestCase("checkCopyEnabledForReadOnlyFile").EnableDriveFs(),
-        TestCase("checkCopyEnabledForReadOnlyFolder").EnableDriveFs(),
-        TestCase("checkCutEnabledForReadWriteFile").EnableDriveFs(),
-        TestCase("checkCutDisabledForReadOnlyDocument").EnableDriveFs(),
-        TestCase("checkCutDisabledForReadOnlyFile").EnableDriveFs(),
-        TestCase("checkCutDisabledForReadOnlyFolder").EnableDriveFs(),
-        TestCase("checkPasteIntoFolderEnabledForReadWriteFolder")
-            .EnableDriveFs(),
-        TestCase("checkPasteIntoFolderDisabledForReadOnlyFolder")
-            .EnableDriveFs(),
-        TestCase("checkNewFolderEnabledInsideReadWriteFolder").EnableDriveFs(),
-        TestCase("checkNewFolderDisabledInsideReadOnlyFolder").EnableDriveFs(),
-        TestCase("checkPasteEnabledInsideReadWriteFolder").EnableDriveFs(),
-        TestCase("checkPasteDisabledInsideReadOnlyFolder").EnableDriveFs(),
-        TestCase("checkCopyEnabledForReadWriteFolderInTree").EnableDriveFs(),
-        TestCase("checkCopyEnabledForReadOnlyFolderInTree").EnableDriveFs(),
-        TestCase("checkCutEnabledForReadWriteFolderInTree").EnableDriveFs(),
-        TestCase("checkCutDisabledForReadOnlyFolderInTree").EnableDriveFs(),
-        TestCase("checkPasteEnabledForReadWriteFolderInTree").EnableDriveFs(),
-        TestCase("checkPasteDisabledForReadOnlyFolderInTree").EnableDriveFs(),
-        TestCase("checkContextMenuForTeamDriveRoot").EnableDriveFs()));
-
 WRAPPED_INSTANTIATE_TEST_CASE_P(
     Delete, /* delete.js */
     FilesAppBrowserTest,
@@ -524,9 +481,7 @@ WRAPPED_INSTANTIATE_TEST_CASE_P(
     FolderShortcuts, /* folder_shortcuts.js */
     FilesAppBrowserTest,
     ::testing::Values(TestCase("traverseFolderShortcuts"),
-                      TestCase("traverseFolderShortcuts").EnableDriveFs(),
-                      TestCase("addRemoveFolderShortcuts"),
-                      TestCase("addRemoveFolderShortcuts").EnableDriveFs()));
+                      TestCase("addRemoveFolderShortcuts")));
 
 WRAPPED_INSTANTIATE_TEST_CASE_P(
     SortColumns, /* sort_columns.js */
@@ -672,11 +627,8 @@ WRAPPED_INSTANTIATE_TEST_CASE_P(
     FilesAppBrowserTest,
     ::testing::Values(TestCase("metadataDownloads"),
                       TestCase("metadataDrive"),
-                      TestCase("metadataDrive").EnableDriveFs(),
                       TestCase("metadataTeamDrives"),
-                      TestCase("metadataTeamDrives").EnableDriveFs(),
-                      TestCase("metadataLargeDrive"),
-                      TestCase("metadataLargeDrive").EnableDriveFs()));
+                      TestCase("metadataLargeDrive")));
 
 // Structure to describe an account info.
 struct TestAccountInfo {

chrome/browser/chromeos/file_manager/file_manager_browsertest_base.cc

@@ -856,10 +856,7 @@ class DriveFsTestVolume : public DriveTestVolume {
         base::FilePath(entry.target_path).BaseName().value(), entry.pinned,
         entry.shared_option == AddEntriesMessage::SharedOption::SHARED ||
             entry.shared_option ==
-                AddEntriesMessage::SharedOption::SHARED_WITH_ME,
-        {entry.capabilities.can_share, entry.capabilities.can_copy,
-         entry.capabilities.can_delete, entry.capabilities.can_rename,
-         entry.capabilities.can_add_children});
+                AddEntriesMessage::SharedOption::SHARED_WITH_ME);
 
     switch (entry.type) {
       case AddEntriesMessage::FILE: {
@@ -1404,8 +1401,7 @@ FileManagerBrowserTestBase::CreateDriveIntegrationService(Profile* profile) {
                                    kPredefinedProfileSalt);
     drive_volumes_[profile->GetOriginalProfile()] =
         std::make_unique<DriveFsTestVolume>(profile->GetOriginalProfile());
-    if (!IsIncognitoModeTest() &&
-        profile->GetPath().BaseName().value() == "user") {
+    if (!IsIncognitoModeTest()) {
       base::ThreadTaskRunnerHandle::Get()->PostTask(
           FROM_HERE,
           base::BindOnce(base::IgnoreResult(&LocalTestVolume::Mount),

chromeos/components/drivefs/fake_drivefs.cc

@@ -89,7 +89,6 @@ struct FakeDriveFs::FileMetadata {
   bool hosted = false;
   bool shared = false;
   std::string original_name;
-  mojom::Capabilities capabilities;
 };
 
 class FakeDriveFs::SearchQuery : public mojom::SearchQuery {
@@ -236,13 +235,11 @@ void FakeDriveFs::SetMetadata(const base::FilePath& path,
                               const std::string& mime_type,
                               const std::string& original_name,
                               bool pinned,
-                              bool shared,
-                              const mojom::Capabilities& capabilities) {
+                              bool shared) {
   auto& stored_metadata = metadata_[path];
   stored_metadata.mime_type = mime_type;
   stored_metadata.original_name = original_name;
   stored_metadata.hosted = (original_name != path.BaseName().value());
-  stored_metadata.capabilities = capabilities;
   if (pinned) {
     stored_metadata.pinned = true;
   }
@@ -297,7 +294,7 @@ void FakeDriveFs::GetMetadata(const base::FilePath& path,
                            ? path.BaseName().value()
                            : stored_metadata.original_name;
   metadata->alternate_url = GURL(base::StrCat({prefix, suffix})).spec();
-  metadata->capabilities = stored_metadata.capabilities.Clone();
+  metadata->capabilities = mojom::Capabilities::New();
 
   std::move(callback).Run(drive::FILE_ERROR_OK, std::move(metadata));
 }

chromeos/components/drivefs/fake_drivefs.h

@@ -34,8 +34,7 @@ class FakeDriveFs : public drivefs::mojom::DriveFs,
                    const std::string& mime_type,
                    const std::string& original_name,
                    bool pinned,
-                   bool shared,
-                   const mojom::Capabilities& capabilities);
+                   bool shared);
 
   const base::FilePath& mount_path() { return mount_path_; }
 

ui/file_manager/integration_tests/file_manager/context_menu.js

@@ -275,6 +275,22 @@ testcase.checkPasteIntoFolderDisabledForReadOnlyFolder = function() {
   checkContextMenu('paste-into-folder', 'Read-Only Folder', false);
 };
 
+/**
+ * Tests that the New Folder menu item is enabled if a read-write folder is
+ * selected.
+ */
+testcase.checkNewFolderEnabledForReadWriteFolder = function() {
+  checkContextMenu('new-folder', 'photos', true);
+};
+
+/**
+ * Tests that the New Folder menu item is disabled if a read-only folder is
+ * selected.
+ */
+testcase.checkNewFolderDisabledForReadOnlyFolder = function() {
+  checkContextMenu('new-folder', 'Read-Only Folder', false);
+};
+
 /**
  * Tests that text selection context menus are disabled in tablet mode.
  */