Bug 86202 – Web Inspector: Can't type spaces in "Override User Agent" field

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

[Alexander Pavlov (apavlov)]

What steps will reproduce the problem?
1. On any web page, open the developer tools (either from "View", "Developer" or by pressing Command+Option+J).
2. Click the gear icon in the lower right corner of the window.
3. Check the checkbox near where it says "Override User Agent".
4. Pick "Other..." from the dropdown menu to enter your own user agent string.
5. Try to type "Mozilla/5.0 (whatever)".

What is the expected result?
I should be able to enter the user agent string I am trying to type.

What happens instead?
Pressing the space bar closes the settings popup.

Please provide any additional information below. Attach a screenshot if
possible.
I am able to work around the problem by copying a space and pasting it into the text field.

Upstreaming http://code.google.com/p/chromium/issues/detail?id=125796

[Alexander Pavlov (apavlov)]

Created attachment 141394 [details]
Patch

[Yury Semikhatsky]

Comment on attachment 141394 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=141394&action=review

> Source/WebCore/inspector/front-end/HelpScreen.js:50
> +    this.closeKeys = [

Can we have a virtual method instead and override it in SettingsScreen or pass the key set as a parameter to the constructor? We don't override fields in the front-end code.

[Alexander Pavlov (apavlov)]

Created attachment 141398 [details]
Patch

[Alexander Pavlov (apavlov)]

Committed r116762: <http://trac.webkit.org/changeset/116762>