1366800 - Two search fields are displayed on the new debugger

Status: VERIFIED FIXED

(DevTools :: Debugger, defect, P1)

Description

[Hani Yacoub, Desktop QA]

Attachment: Recording #7.mp4

[Affected versions]: 
Nightly 55.0a1

[Affected platforms]:
Platforms: Windows 10 x 64, Ubuntu 16.10 and Mac OS X 10.12

[Steps to reproduce]:
1. Go to https://devtools-html.github.io/debugger-examples/examples/todomvc/
2. Open Debugger.
3. From "Sources" section select a file that has the ".js" termination (e.g todo-view.js).
4. (On Windows - Press CTRL + F OR on Mac - Press CMD + F )
   Search for a term that can be found (e.g listenTo).
5. Click on any code line.
6. Press (On Windows - Press CTRL + F OR on Mac - Press CMD + F )

[Expected result]:
Pressing (On Windows - Press CTRL + F OR on Mac - Press CMD + F) should highlight or close the search field.

[Actual result]:
Two search fields are displayed on the new debugger

Comment 1

[Jason Laster [:jlast]]

This is being tracked here: https://github.com/devtools-html/debugger.html/issues/3006

Comment 2

[Bryan Clark (DevTools PM) [:clarkbw]]

Fixed here: https://github.com/devtools-html/debugger.html/pull/3135
And cannot be reproduced: https://github.com/devtools-html/debugger.html/issues/3006#issuecomment-309403791

Comment 3

[Hani Yacoub, Desktop QA]

I verified this bug on Mac OS 12.12 and it is fixed, but it's still reproducible on Windows 10 x 64 and Ubuntu 16.04 x64.

Comment 4

[Bryan Clark (DevTools PM) [:clarkbw]]

Ok, lets have Jason take a look at this on Windows only.  Thanks!

Comment 5

[Jason Laster [:jlast]]

Thanks. I'll take a look

Comment 6

[Jason Laster [:jlast]]

This was fixed on nightly last week. I tested it today on a windows laptop and could not reproduce.

Comment 7

[Hani Yacoub, Desktop QA]

Build ID: 20170706060058
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:56.0) Gecko/20100101 Firefox/56.0

Verified as fixed on Firefox Nightly 56.0a1 on Windows 10 x 64, Windows 7 x32, Mac OS X 10.12 and Ubuntu 16.04 x64.

Comment 8

[Julien Cristau [:jcristau] (back August 18)]

the new debugger is disabled in 55.