This test regularly times out on this particular debug bot. https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=DetailsPanel%2FFileManagerDetailsPanelBrowserTest.Test%2F1&testType=browser_tests Looks like it occasionally times out on Linux Chromium OS ASan LSan Tests (1), too.
+OWNERS
oka@, I guess that the details panel function has already been integrated to quick view, and this test does not make sense anymore. If it is the case, could you remove the test along with the details panel implementation?
Yes. Will do.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/86cc6307099df2aa19fac30eae8bbae90bbb57cb commit 86cc6307099df2aa19fac30eae8bbae90bbb57cb Author: oka <oka@chromium.org> Date: Mon Dec 12 09:20:09 2016 Remove details panel. Details panel was being developed under a flag, but has been replaced with Quick View and is no longer needed. BUG= 672767 TEST=Manually tested that FileManager successfully launches using Linux. git cl try. CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2565333002 Cr-Commit-Position: refs/heads/master@{#437836} [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/chrome/app/chromeos_strings.grdp [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/chrome/browser/about_flags.cc [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/chrome/browser/chromeos/extensions/file_manager/private_api_strings.cc [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/chrome/browser/chromeos/file_manager/file_manager_browsertest.cc [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/chromeos/chromeos_switches.cc [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/chromeos/chromeos_switches.h [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/ui/file_manager/file_manager/foreground/css/file_manager.css [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/ui/file_manager/file_manager/foreground/js/app_state_controller.js [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/ui/file_manager/file_manager/foreground/js/compiled_resources.gyp [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/ui/file_manager/file_manager/foreground/js/file_manager.js [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/ui/file_manager/file_manager/foreground/js/main_scripts.js [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/ui/file_manager/file_manager/foreground/js/main_window_component.js [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/ui/file_manager/file_manager/foreground/js/metadata_update_controller.js [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/ui/file_manager/file_manager/foreground/js/ui/compiled_resources2.gyp [delete] https://crrev.com/44e989098aaaddeb9f0ee857282bc8cdaa1cc3d1/ui/file_manager/file_manager/foreground/js/ui/details_container.js [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/ui/file_manager/file_manager/foreground/js/ui/file_manager_ui.js [delete] https://crrev.com/44e989098aaaddeb9f0ee857282bc8cdaa1cc3d1/ui/file_manager/file_manager/foreground/js/ui/multi_file_details.js [delete] https://crrev.com/44e989098aaaddeb9f0ee857282bc8cdaa1cc3d1/ui/file_manager/file_manager/foreground/js/ui/single_file_details.js [modify] https://crrev.com/86cc6307099df2aa19fac30eae8bbae90bbb57cb/ui/file_manager/file_manager/main.html [delete] https://crrev.com/44e989098aaaddeb9f0ee857282bc8cdaa1cc3d1/ui/file_manager/integration_tests/file_manager/details_panel.js
Fixed. 2016年12月12日(月) 18:24 bugdroid1 via monorail < monorail+v2.3275348242@chromium.org>:
Comment 1 by grt@chromium.org
, Dec 9 2016