New issue
Advanced search Search tips

Issue 877895 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug

Blocking:
issue 879035



Sign in to add a comment

Create separate test bundles for ChromeOS gallery/file_manager integration tests

Project Member Reported by tapted@chromium.org, Aug 27

Issue description

Chrome Version       : 70.0.3528.4

Currently they're all mixed together, which makes for some mad dependency hell.

Context: https://chromium-review.googlesource.com/c/chromium/src/+/1186295/7/ui/file_manager/file_manager/background/js/test_util_base.js#546


https://chromium-review.googlesource.com/c/chromium/src/+/1188735 documents the current dependencies.

Plan to fix
 - create //ui/file_manager/toolkit
 - move things like //ui/file_manager/file_manager/background/js:background_base (and its dependencies) to this "toolkit"
  - also shared ui components, metadata, volume manager wrapper, etc.
 - eventually purge references to file_manager/file_manager from gallery, video_player, etc.
  - depend on the toolkit instead
   - file manager also depends on the toolkit
 
Blocking: 879035

Sign in to add a comment