Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 410889

Summary: Search Files dialog behaves poorly when correcting text.
Product: [ECD] Orion Reporter: Mike Wilson <Mike_Wilson>
Component: EditorAssignee: libing wang <libingw>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: libingw, mamacdon, simon_kaegi
Version: unspecifiedFlags: mamacdon: review+
simon_kaegi: review+
Target Milestone: 3.0 RC3   
Hardware: PC   
OS: Mac OS X   
Whiteboard:
Attachments:
Description Flags
Search Files dialog in bogus state none

Description Mike Wilson CLA 2013-06-16 12:09:37 EDT
Created attachment 232417 [details]
Search Files dialog in bogus state

1) Hit Cmd-H to open new Search Files dialog
2) Type some characters that won't match files in your workspace
3) Delete characters, then type correct text.

It seems there are timing issues with the threads that search for results. I have seen more than one "Searching for..." message displayed simultaneously, and sometimes this leaves you in a state where the previous search results replace the correct ones. 

So for example, I typed "Syntaax", then quickly corrected it to "Syntax". There should have been several matches found for Syntax, but instead the dialog displayed "No matches found for Syntaax". See the attached screen shot.

Note this doesn't always happen, but I was able to cause the error several times in 10 minutes of testing.

(Tried in both Safari and Chrome)
Comment 1 libing wang CLA 2013-06-17 08:45:45 EDT
I could always reproduce the multiple "Search for..." part and sometimes(quite often) "syntaax" part.

The file name search(CTRL+SHIFT+F) is much faster than keyword search so we did not notice the lagging issues before the keyword search was put into the find dialog. 

I remember that for the file name search(CTRL+SHIFT+F) in a non-index file system, I put an waiting gesture on the right of the search field, but it seems now the gesture is gone for that. Maybe a regression?

I will roll that back and I think we can use the same waiting/timer mechanism that was used for the crawling file name search.
Comment 2 libing wang CLA 2013-06-17 09:07:50 EDT
Tested the CTRL+H on an HTML5 local file system:
1.When the dialog opens, it builds the file skelton. But this should be only for the file name search. 
2.It always searches for file name, not the keyword.

Issue 1 is easy to fix but issue 2 will be really hard. We can't expect the on-the-fly crawler search. So I think we should disable the on the fly search.
Comment 4 Mark Macdonald CLA 2013-06-19 19:38:56 EDT
Tested. +
Comment 5 Simon Kaegi CLA 2013-06-19 23:42:16 EDT
Pushed.