Instant Evaluation (aka eager evaluation) not showing correct values after using Space key on different function while autocomplete is opened
Categories
(DevTools :: Console, defect, P1)
Tracking
(firefox-esr68 unaffected, firefox73 disabled, firefox74 disabled, firefox75- verified, firefox76 verified)
People
(Reporter: atrif, Assigned: nchevobbe)
References
(Blocks 2 open bugs, Regression)
Details
(Keywords: regression)
Attachments
(2 files)
243.23 KB,
image/gif
|
Details | |
47 bytes,
text/x-phabricator-request
|
jcristau
:
approval-mozilla-beta+
|
Details | Review |
Affected versions
- 75.0a1 (20200303214128)
Affected platforms
- Ubuntu 18.04
- Windows 10x64
- macOS 10.15
Steps to reproduce
- Open Firefox and Web Console.
- Set
a=1
andb=2
. - Write
a*b
and use the down arrow key to go to the “blur” item on the autocomplete popup. - Hit Space key.
Expected result
- The result for evaluation is
2
.
Actual result
- The result is
NaN
.
Regression Range
- I will search for one ASAP if there is one
Notes
- Attached a screen recording.
Reporter | ||
Comment 1•4 years ago
•
|
||
Hello! Attaching here the pushlog from regression range:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=4312c783d4346cfbe0ca92266036abc76f38df1f&tochange=23688b6ac3dedfc1388094418b0b097beea1534b
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 2•4 years ago
|
||
There was a bug if you had a given item selected in the
autocomplete popup, but would then type a character that
would dismiss the popup.
This was because of an extra check that was made in the
reducer to ensure that the result of an eager evaluation
was matching the expression that requested it.
This check was here from the beginning of eager evaluation,
but isn't needed anymore as we have some guarantee of order
for the eager evaluation.
A test case is added to ensure this is working as expected.
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/311a5dc0cc71 Fix instant evaluation result when dismissing popup suggestion. r=Honza.
Comment 4•4 years ago
|
||
[Tracking Requested - why for this release]: Can degrade the experience for the newly added instant eval feature.
Comment 5•4 years ago
|
||
bugherder |
Assignee | ||
Comment 6•4 years ago
|
||
Comment on attachment 9133508 [details]
Bug 1619924 - Fix instant evaluation result when dismissing popup suggestion. r=Honza.
Beta/Release Uplift Approval Request
- User impact if declined: Erroneous instant evaluation result in the webconsole
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: 1. Open Firefox and Web Console.
- Execute
a=1; b=2
. - Write
a*b
and use the down arrow key to go to the “blur” item on the autocomplete popup. - Hit Space key.
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): DevTools only, covered by automated test
- String changes made/needed:
Assignee | ||
Updated•4 years ago
|
Updated•4 years ago
|
Comment 7•4 years ago
|
||
Comment on attachment 9133508 [details]
Bug 1619924 - Fix instant evaluation result when dismissing popup suggestion. r=Honza.
approved for 75.0b5
Updated•4 years ago
|
Updated•4 years ago
|
Comment 8•4 years ago
|
||
I got a conflict when trying to uplift this to beta.
devtools/client/webconsole/actions/input.js
conflict with https://bugzilla.mozilla.org/show_bug.cgi?id=1619302 which didn't land on beta
Assignee | ||
Comment 9•4 years ago
|
||
we should put this block in the uplifted patch: https://searchfox.org/mozilla-central/rev/d6f957415cf009995ecb539ef1425316d82164a9/devtools/client/webconsole/actions/input.js#244-251
(we can't uplift Bug 1619302 because there are too many dependencies)
Reporter | ||
Comment 10•4 years ago
|
||
Fix verified with Firefox 76.0a1 (20200318041411) on Windos 10x64, macOS 10.15 and Ubuntu 18.04.
Comment 11•4 years ago
|
||
bugherder uplift |
Reporter | ||
Comment 12•4 years ago
|
||
Verified with Firefox 75.0b6 (20200318131458) from comment 11 on Windows 10x64, macOS 10.15 and Ubuntu 18.04.
Description
•