The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b4e92b237ed276982f0518387280d6df9d79fb4a

commit b4e92b237ed276982f0518387280d6df9d79fb4a
Author: Timothy Loh <timloh@chromium.org>
Date: Mon Jul 30 04:48:12 2018

Add tests for InstallLinuxPackageDialog

This CL adds some tests for the linux package install flow in the File
Manager. It also fixes a bug in the CrostiniManager where package
progress observers were not correctly removed, as discovered by these
tests.

Bug:  868221 
Change-Id: I2b6adf2530ff43a0b22fe2b552bb877f98fd61c7
Reviewed-on: https://chromium-review.googlesource.com/1152643
Reviewed-by: Joel Hockey <joelhockey@chromium.org>
Commit-Queue: Timothy Loh <timloh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578975}
[modify] https://crrev.com/b4e92b237ed276982f0518387280d6df9d79fb4a/chrome/browser/chromeos/crostini/crostini_manager.cc
[modify] https://crrev.com/b4e92b237ed276982f0518387280d6df9d79fb4a/chrome/browser/chromeos/file_manager/file_manager_browsertest.cc
[add] https://crrev.com/b4e92b237ed276982f0518387280d6df9d79fb4a/chrome/test/data/chromeos/file_manager/package.deb
[modify] https://crrev.com/b4e92b237ed276982f0518387280d6df9d79fb4a/ui/file_manager/file_manager/foreground/js/file_tasks_unittest.js
[add] https://crrev.com/b4e92b237ed276982f0518387280d6df9d79fb4a/ui/file_manager/integration_tests/file_manager/install_linux_package_dialog.js
[modify] https://crrev.com/b4e92b237ed276982f0518387280d6df9d79fb4a/ui/file_manager/integration_tests/file_manager_test_manifest.json
[modify] https://crrev.com/b4e92b237ed276982f0518387280d6df9d79fb4a/ui/file_manager/integration_tests/test_util.js

Requesting a merge to 69 for the above CL. There's a one character typo fix for I think a crash or maybe UAF, the rest of the CL is adding tests.

Your change meets the bar and is auto-approved for M69. Please go ahead and merge the CL to branch 3497 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), kariahda@(iOS), cindyb@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2837e5635ef0799879d1c2ccc5b5a64c7ca60b4b

commit 2837e5635ef0799879d1c2ccc5b5a64c7ca60b4b
Author: Timothy Loh <timloh@chromium.org>
Date: Wed Aug 01 04:06:10 2018

[merge to 69] Add tests for InstallLinuxPackageDialog

This CL adds some tests for the linux package install flow in the File
Manager. It also fixes a bug in the CrostiniManager where package
progress observers were not correctly removed, as discovered by these
tests.

Bug:  868221 
Change-Id: I2b6adf2530ff43a0b22fe2b552bb877f98fd61c7
Reviewed-on: https://chromium-review.googlesource.com/1152643
Reviewed-by: Joel Hockey <joelhockey@chromium.org>
Commit-Queue: Timothy Loh <timloh@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#578975}(cherry picked from commit b4e92b237ed276982f0518387280d6df9d79fb4a)
Reviewed-on: https://chromium-review.googlesource.com/1157964
Reviewed-by: Timothy Loh <timloh@chromium.org>
Cr-Commit-Position: refs/branch-heads/3497@{#305}
Cr-Branched-From: 271eaf50594eb818c9295dc78d364aea18c82ea8-refs/heads/master@{#576753}
[modify] https://crrev.com/2837e5635ef0799879d1c2ccc5b5a64c7ca60b4b/chrome/browser/chromeos/crostini/crostini_manager.cc
[modify] https://crrev.com/2837e5635ef0799879d1c2ccc5b5a64c7ca60b4b/chrome/browser/chromeos/file_manager/file_manager_browsertest.cc
[add] https://crrev.com/2837e5635ef0799879d1c2ccc5b5a64c7ca60b4b/chrome/test/data/chromeos/file_manager/package.deb
[modify] https://crrev.com/2837e5635ef0799879d1c2ccc5b5a64c7ca60b4b/ui/file_manager/file_manager/foreground/js/file_tasks_unittest.js
[add] https://crrev.com/2837e5635ef0799879d1c2ccc5b5a64c7ca60b4b/ui/file_manager/integration_tests/file_manager/install_linux_package_dialog.js
[modify] https://crrev.com/2837e5635ef0799879d1c2ccc5b5a64c7ca60b4b/ui/file_manager/integration_tests/file_manager_test_manifest.json
[modify] https://crrev.com/2837e5635ef0799879d1c2ccc5b5a64c7ca60b4b/ui/file_manager/integration_tests/test_util.js

 timloh@ Please provide verification steps. Thanks!