Files.app: Separate the last search query from "lastQuery"

Files.app: Separate the last search query from "lastQuery"

The last SEARCH query and the last AUTO-COMPLETE query are different. But they were used as the single variable. This confusion caused the issue that sometimes the search result didn't disappear when the textbox was changed. This patch fixes the problem.

BUG=225959
TEST=none

NOTRY=True
# for trybots seems broken.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=192806

[yoshiki, 2013-04-08 06:47:37]

@mtomasz: could you take a look? Thanks

[mtomasz, 2013-04-08 07:59:09]

lgtm with a nit.

https://codereview.chromium.org/13771005/diff/1/chrome/browser/resources/file...
File chrome/browser/resources/file_manager/js/file_manager.js (right):

https://codereview.chromium.org/13771005/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/js/file_manager.js:3108: * @param
{function} onClearSearch Function to be called when search state
nit: Please add arguments and =. Also onClearSearch -> opt_onClearSearch.

[yoshiki, 2013-04-08 11:56:32]

Thanks!

https://codereview.chromium.org/13771005/diff/1/chrome/browser/resources/file...
File chrome/browser/resources/file_manager/js/file_manager.js (right):

https://codereview.chromium.org/13771005/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/js/file_manager.js:3108: * @param
{function} onClearSearch Function to be called when search state
On 2013/04/08 07:59:09, mtomasz wrote:
> nit: Please add arguments and =. Also onClearSearch -> opt_onClearSearch.

Done.

[commit-bot: I haz the power, 2013-04-08 11:56:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoshiki@chromium.org/13771005/5001

[commit-bot: I haz the power, 2013-04-08 12:04:21]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_chromeos_clang.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-04-08 12:16:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoshiki@chromium.org/13771005/5001

[commit-bot: I haz the power, 2013-04-08 12:16:15]

Change committed as 192806