1598285 - Network Panel Search - Search never stop after a web page is refreshed

Status: VERIFIED FIXED

(DevTools :: Netmonitor, defect, P3)

Description

[Marcela (Please NI request to Brindusa Tot)]

After starting a search if a web page is refreshed searching never stoped

Affected versions
Firefox versions Nightly 72.0a1 , Beta 71.0b11

Affected platforms
Windows 10
macOS 10.14
Ubuntu 18.04

Steps to reproduce
Go to cnn.com
Open the network panel search
Start a search after "a" and during the search is performed refresh the web page

Expected Result
Search should be ended as usual

Actual Result
Search never stops. "Searching..." label stays always visible and working.

Note
If clicks on any result while "Searching" is working, no results are displayed on the left Side Panel

Comment 1

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Good catch, thanks for the report!

I can reproduce this on my machine.

Honza

Comment 2

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Attachment: image.png

Some instructions for anyone interested in fixing this bug

1. See the attached screenshot it shows the related UI

2. The search is an asynchronous operation and it's happening here:
   https://searchfox.org/mozilla-central/rev/652014ca1183c56bc5f04daf01af180d4e50a91c/devtools/client/netmonitor/src/actions/search.js#46

3. The newOngoingSearch is stored in a reducer (using addOngoingSearch action) here:
   https://searchfox.org/mozilla-central/rev/652014ca1183c56bc5f04daf01af180d4e50a91c/devtools/client/netmonitor/src/reducers/search.js#32

4. The search can be cancelled using stopOngoingSearch action
   https://searchfox.org/mozilla-central/rev/652014ca1183c56bc5f04daf01af180d4e50a91c/devtools/client/netmonitor/src/actions/search.js#250

5. The stopOngoingSearch should be called when the page is reloaded somewhere in this function:
   https://searchfox.org/mozilla-central/rev/652014ca1183c56bc5f04daf01af180d4e50a91c/devtools/client/netmonitor/src/connector/firefox-connector.js#201

Honza

Comment 3

[Duncan Dean]

I’ll take a crack at it as my first bug🙂

Comment 4

[Duncan Dean]

Attachment: Bug 1598285 - Stop ongoing search after reload. r=Honza!

Comment 5

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Awesome, thanks for the help!
Honza

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e45f7a332126
Stop ongoing search after reload. r=Honza

Comment 7

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/e45f7a332126

Comment 8

[Pascal Chevrel:pascalc]

Jan, do you want to request an uplift to our RC build or can it ship in 72?

Comment 9

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Comment on attachment 9110643 [details]
Bug 1598285 - Stop ongoing search after reload. r=Honza!

Beta/Release Uplift Approval Request

• User impact if declined: Wrong search label. Indicates that search is in progress, which might be confusing to the user.
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: Yes
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Low risk, only for developers, a few line of code.
• String changes made/needed:

Comment 10

[Pascal Chevrel:pascalc]

Comment on attachment 9110643 [details]
Bug 1598285 - Stop ongoing search after reload. r=Honza!

The patch is minimal and this bug is the reason why the feature got a yellow signoff from QA, I am taking it for RC.

Comment 11

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/releases/mozilla-release/rev/e93478316b37

Comment 12

[Cristian Fogel [:cfogel]]

Hello Marcela, please take a look when you have time.

Comment 13

[Marcela (Please NI request to Brindusa Tot)]

Verified fixed in latest Nightly 72.0a1 Build ID 20191126093448 and Firefox Release 71.0 Build ID 20191125204040 on Windows 10 x64, macOS 10.14 and Ubuntu 18.04 x64.