Closed Bug 507338 Opened 10 years ago Closed 10 years ago
CE] updater .exe update in progress dialog isn't displayed
It hasn't been implemented as of yet.
10 years ago
Shows that the update dialog is centered
Still needs to handle device screen dimensions smaller than the dialog. There is also a strange bug when running this on the nv device in that ResizeDialogToFit subsequent code to fail though it does work if there is a MessageBox just after the call to DrawText while this works fine in an emulator. *scratches head*
Attachment #393124 - Attachment is obsolete: true
After getting everything working on an emulator I tested on an nv device and ended up running into a couple of idiosyncrasies with it. SendDlgItemMessage wasn't working so I switched to using GetDlgItem and SendMessage. Nothing after the ResizeDialogToFit function was called. Turned out that even though the call to DrawText was succeeding CE would bail later on. Using szwInfo fixed this. I went with a smaller dialog in updater.rc for WINCE since the screen real estate is smaller... it is resized as needed. I still need to handle resizing using multiple lines when there isn't enough screen real estate but I'd prefer to do that in another bug.
Comment on attachment 394150 [details] [diff] [review] patch rev1 Brad, could you also review this? Thanks
Attachment #394150 - Flags: review?(bugmail)
Comment on attachment 394150 [details] [diff] [review] patch rev1 >+OS_LIBS += $(call EXPAND_LIBNAME,commctrl ws2) >+DEFINES += -DUNICODE -D_UNICODE -DWINCE We should have WINCE defined in autoconf.mk. Also, arm-wince-gcc defines UNICODE. Is this needed for RC?
Attachment #394150 - Flags: review?(bugmail) → review+
It was needed for the RC but since the RC had so many ifdefs I went with a separate one for WINCE to clean it up a bit.
forgot to add the fix for bug 508905
Filed bug 510162 for using multiple lines for the info text when appropriate.
Attachment #394218 - Flags: review?(vladimir) → review+
Requesting blocking... we need this to show the updater ui for nv.
Pushed to mozilla-central http://hg.mozilla.org/mozilla-central/rev/546a7d596966
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
10 years ago
Flags: blocking1.9.2? → blocking1.9.2+
10 years ago
pushed to mozilla-1.9.2 http://hg.mozilla.org/releases/mozilla-1.9.2/rev/b660ab28fc27
Target Milestone: --- → mozilla1.9.2b1
Verified fixed on 1.9.2 using Mozilla/5.0 (Windows; U; WindowsCE 6.0; en-US; rv:1.9.2a2pre) Gecko/20090905 Firefox/3.6a2pre. Adding keyword.
You need to log in before you can comment on or make changes to this bug.