Closed Bug 444488 Opened 12 years ago Closed 12 years ago

WM_ACTIVATEAPP / WM_MOUSEACTIVATE missing from WinCE

Categories

(Core :: Widget: Win32, defect)

x86
Windows CE
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla1.9.1a2

People

(Reporter: wolfe, Assigned: wolfe)

References

Details

(Keywords: mobile)

Attachments

(1 file, 2 obsolete files)

Removed hack to make WM_ACTIVATEAPP be WM_ACTIVATE on WinCE

Removed hack to make WM_MOUSEACTIVATE be WM_ACTIVATE on WinCE

Both of these defines were messing up windows message switch statement, and these three windows messages communicate very different events.
Component: General → Widget: Win32
Product: Fennec → Core
QA Contact: general → win32
Version: 1.0 → Trunk
Keywords: mobile
Comment on attachment 328795 [details] [diff] [review]
Comment out the bad equates for WinCE builds

Is there a purpose for just commenting the ifdefs out rather than just removing them?
Thanks for the good observation, Stuart.
Attachment #328795 - Attachment is obsolete: true
New attachment id=329464 removes extra code from this attachment, and moves them into attachment for bug # 444494.
Attachment #329464 - Attachment is obsolete: true
Attachment #329597 - Flags: review+
Assignee: nobody → wolfe
Keywords: checkin-needed
Blocks: 432792
Pushed as 17102:d9870269f8fb.
Status: NEW → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1a2
You need to log in before you can comment on or make changes to this bug.