805759 - Relax global search results auto-expand rules

Status: RESOLVED FIXED

(DevTools :: Debugger, defect, P3)

Description

[Victor Porof [:vporof][:vp]]

In the initial phase of implementing global sources search in the debugger, the decision of having only the first per-file group of matches expanded by default was taken because of performance issues. These could be lifted in case of files with a small set of matches. It would be more convenient to have this rule a bit more relaxed.

Comment 1

[Victor Porof [:vporof][:vp]]

FILTER ON PUMPKINS.

Comment 2

[Victor Porof [:vporof][:vp]]

Attachment: v1

Comment 3

[Panos Astithas (he/him) [:past] (please ni?)]

Comment on attachment 683566 [details] [diff] [review]
v1

Review of attachment 683566 [details] [diff] [review]:
-----------------------------------------------------------------

It doesn't feel noticeably slower on my laptop, even for github and gmail, so let's do it. One problem, unrelated to this patch (although it probably makes things slightly worse than they were), is that after searching for "!test", if I try to search for "!click" on a site like gmail, the moment I have left a single character match in the input box ("!t") things slow down to a crawl. We should add a 3-character threshold to make sure the debugger stays responsive at all times.

Comment 4

[Victor Porof [:vporof][:vp]]

(In reply to Panos Astithas [:past] from comment #3)
> Comment on attachment 683566 [details] [diff] [review]
> v1
> 
> Review of attachment 683566 [details] [diff] [review]:
> -----------------------------------------------------------------
> We should add a 3-character threshold to make sure the debugger
> stays responsive at all times.

Agreed.

Comment 5

[Victor Porof [:vporof][:vp]]

(In reply to Victor Porof [:vp] from comment #4)
> (In reply to Panos Astithas [:past] from comment #3)
> 
> Agreed.

Or maybe increase the search timeout based on the number of characters? It's currently always set at a comfortable 150ms, but I can't imagine any perceived slowdowns by raising it to 500ms when there are less than 3 characters.

Comment 6

[Victor Porof [:vporof][:vp]]

Filed bug 813587.

Comment 7

[Victor Porof [:vporof][:vp]]

https://hg.mozilla.org/integration/fx-team/rev/3d52cb78cebd

Comment 8

[Victor Porof [:vporof][:vp]]

https://hg.mozilla.org/mozilla-central/rev/3d52cb78cebd

Comment 9

[Victor Porof [:vporof][:vp]]

Comment on attachment 683566 [details] [diff] [review]
v1

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Searching for a string in all the source files usually shows the results in collapsed menus, which isn't very streamlined.
User impact if declined: Almost none, this patch affects a developer tool, however global search operations are common and this fix fluidizes the experience.
Testing completed (on m-c, etc.): fx-team and m-c
Risk to taking this patch (and alternatives if risky): None.
String or UUID changes made by this patch: None.

Comment 10

[Alex Keybl [:akeybl]]

Comment on attachment 683566 [details] [diff] [review]
v1

Early enough in the cycle to take a minor feature fix here with little to no risk.

Comment 11

[Victor Porof [:vporof][:vp]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/ee4104aba1cb