Closed
Bug 444488
Opened 16 years ago
Closed 16 years ago
WM_ACTIVATEAPP / WM_MOUSEACTIVATE missing from WinCE
Categories
(Core :: Widget: Win32, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9.1a2
People
(Reporter: wolfe, Assigned: wolfe)
References
Details
(Keywords: mobile)
Attachments
(1 file, 2 obsolete files)
|
767 bytes,
patch
|
pavlov
:
review+
|
Details | Diff | Splinter Review |
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.
|
||
Updated•16 years ago
|
Component: General → Widget: Win32
Product: Fennec → Core
QA Contact: general → win32
Version: 1.0 → Trunk
|
||
Comment 1•16 years ago
|
||
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?
|
Assignee | |
Comment 2•16 years ago
|
||
Thanks for the good observation, Stuart.
Attachment #328795 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 3•16 years ago
|
||
New attachment id=329464 removes extra code from this attachment, and moves them into attachment for bug # 444494.
|
|
Assignee | |
Comment 4•16 years ago
|
||
Attachment #329464 -
Attachment is obsolete: true
|
|
||
Updated•16 years ago
|
Attachment #329597 -
Flags: review+
|
|
||
Updated•16 years ago
|
Assignee: nobody → wolfe
Keywords: checkin-needed
|
||
Comment 5•16 years ago
|
||
Pushed as 17102:d9870269f8fb.
Status: NEW → RESOLVED
Closed: 16 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.
Description
•