Show local files within search results |
|||||||
Issue descriptionRight now launcher shows Drive files, but it does not show local ones, this feels like a bug. Weifang, what do you think?
,
Oct 11 2017
,
Nov 30 2017
,
Dec 5 2017
,
Dec 14 2017
,
Dec 26 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/571c8566c2eb0aa945c5b44bef061c26c953f7f6 commit 571c8566c2eb0aa945c5b44bef061c26c953f7f6 Author: Naoki Fukino <fukino@chromium.org> Date: Tue Dec 26 10:14:58 2017 Show files/directories from Downloads volume in launcher search resutls. This change adds local entries to the result of launcher search. Finding local entries is done by recursive reads for the Downloads volume in JS side. Now that both foreground and background do the similar recursive reads, the common implementation is moved from LocalSearchContentScanner to a common util function. As a part of this change, I added a logic to prefer files which have the search query as their prefixes in reply to omri's request. Bug: 773497 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I22921e6c141aa9014198446ea8a562c6b5154fbf Reviewed-on: https://chromium-review.googlesource.com/842365 Commit-Queue: Naoki Fukino <fukino@chromium.org> Reviewed-by: Tatsuhisa Yamaguchi <yamaguchi@chromium.org> Cr-Commit-Position: refs/heads/master@{#526189} [modify] https://crrev.com/571c8566c2eb0aa945c5b44bef061c26c953f7f6/ui/file_manager/file_manager/background/js/launcher_search.js [modify] https://crrev.com/571c8566c2eb0aa945c5b44bef061c26c953f7f6/ui/file_manager/file_manager/common/js/util.js [modify] https://crrev.com/571c8566c2eb0aa945c5b44bef061c26c953f7f6/ui/file_manager/file_manager/foreground/js/directory_contents.js
,
Jan 10 2018
,
Jan 22 2018
Issue 753970 has been merged into this issue.
,
Jan 24 2018
Verified on M65 dev build (10323.2.0, 65.0.3325.11) cave device. |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by fukino@chromium.org
, Oct 11 2017Owner: fukino@chromium.org
Status: Assigned (was: Untriaged)