If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

WARNING: GetScanCodeWithExtendedFlat() returns without extended flag

RESOLVED FIXED in mozilla18

Status

()

Core
Widget: Win32
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: vlad, Assigned: vlad)

Tracking

unspecified
mozilla18
x86_64
Windows 7
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

WARNING: GetScanCodeWithExtendedFlat() returns without extended flag: file c:/proj/firefox/widget/windows/../../../mozilla-central/widget/windows/KeyboardLayout.cpp, line 554

spam in debug builds when I just hold down the control key.  Fairly clean Windows 7 install with latest updates, debug m-c build.
Created attachment 663246 [details] [diff] [review]
remove this warning

Pretty sure this can just go away; if the NativeKey isn't extended, we don't need to set the flag, right?  So we shouldn't print the warning... and is it useful to print it on < VISTA if mIsExtended?
Assignee: nobody → vladimir
Attachment #663246 - Flags: review?(jmathies)

Comment 2

5 years ago
Comment on attachment 663246 [details] [diff] [review]
remove this warning

Masayuki a better reviewer for this. He's authored all our overhauled keyboard handling.
Attachment #663246 - Flags: review?(jmathies) → review?(masayuki)
Comment on attachment 663246 [details] [diff] [review]
remove this warning

Ah, it's my mistake. I'd like to add NS_ASSERTION(), but it's not so urgent. I'll do it later. So, just removing it is okay for now.
Attachment #663246 - Flags: review?(masayuki) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/02e75979ee3b
https://hg.mozilla.org/mozilla-central/rev/02e75979ee3b

Should this have a test?
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
Don't think so, it just nukes an assertion that was incorrectly-written in the first place (there's no reason to assert the condition as written, so nothing to test).
Flags: in-testsuite? → in-testsuite-
You need to log in before you can comment on or make changes to this bug.