Files app: Enable flag files-ng by default

Add files-ng classes and attribute to main.html so files-ng styles are applied during app startup (to avoid a FOUC flash of non-files-ng blue color). Enable file-ng by default in FilesApp and its browser test harness. Bug: 992818 Change-Id: If437271c5604d36b4b0908abb663a84e52b6ae35 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2087308Reviewed-by: Alex Danilo <adanilo@chromium.org> Commit-Queue: Alex Danilo <adanilo@chromium.org> Cr-Commit-Position: refs/heads/master@{#747421}

Files app: Enable flag files-ng by default
Add files-ng classes and attribute to main.html so files-ng styles are applied during app startup (to avoid a FOUC flash of non-files-ng blue color). Enable file-ng by default in FilesApp and its browser test harness. Bug: 992818 Change-Id: If437271c5604d36b4b0908abb663a84e52b6ae35 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2087308Reviewed-by: Alex Danilo <adanilo@chromium.org> Commit-Queue: Alex Danilo <adanilo@chromium.org> Cr-Commit-Position: refs/heads/master@{#747421}
ea497580 · Luciano Pacheco · Commit Bot · 58b0e561 · ea497580 · ea497580
Commit ea497580 authored Mar 05, 2020 by Luciano Pacheco Committed by Commit Bot Mar 05, 2020
3 changed files
--- a/chrome/browser/chromeos/file_manager/file_manager_browsertest_base.cc
+++ b/chrome/browser/chromeos/file_manager/file_manager_browsertest_base.cc
@@ -1557,7 +1557,7 @@ bool FileManagerBrowserTestBase::GetIsOffline() const {
 }

 bool FileManagerBrowserTestBase::GetEnableFilesNg() const {
-  return false;
+  return true;
 }

 bool FileManagerBrowserTestBase::GetEnableNativeSmb() const {

--- a/chromeos/constants/chromeos_features.cc
+++ b/chromeos/constants/chromeos_features.cc
@@ -144,7 +144,7 @@ const base::Feature kExoPointerLock{"ExoPointerLock",
                                    base::FEATURE_DISABLED_BY_DEFAULT};

 // Enables the next generation file manager.
-const base::Feature kFilesNG{"FilesNG", base::FEATURE_DISABLED_BY_DEFAULT};
+const base::Feature kFilesNG{"FilesNG", base::FEATURE_ENABLED_BY_DEFAULT};

 // Enables the use of Mojo by Chrome-process code to communicate with Power
 // Manager. In order to use mojo, this feature must be turned on and a callsite

--- a/ui/file_manager/file_manager/main.html
+++ b/ui/file_manager/file_manager/main.html
@@ -30,7 +30,7 @@
    <script src="foreground/js/main_scripts.js" defer></script>
  </head>

-  <body aria-label="$i18n{FILEMANAGER_APP_NAME}" tabindex="-1">
+  <body aria-label="$i18n{FILEMANAGER_APP_NAME}" tabindex="-1" class="files-ng">
    <custom-style>
      <style>
        .dialog-footer cr-input {
@@ -337,7 +337,7 @@
      </cr-menu>
    </menus>

-    <div class="dialog-header" role="contentinfo">
+    <div class="dialog-header files-ng" role="contentinfo">
      <div id="location-breadcrumbs" class="breadcrumbs"></div>
      <div id="read-only-indicator" hidden>
        <span id="read-only-icon"></span>
@@ -475,7 +475,7 @@
    <div class="dialog-container">
      <div class="dialog-navigation-list">
        <div class="dialog-navigation-list-contents" role="navigation">
-          <tree id="directory-tree" role="tree" tabindex="21"></tree>
+          <tree id="directory-tree" role="tree" tabindex="21" files-ng></tree>
        </div>
        <div class="dialog-navigation-list-footer" hidden>
          <div id="progress-center" hidden>