Closed Bug 581508 Opened 10 years ago Closed 7 years ago
IUser Info on Win32, particularly fullname
nsIUserInfo.fullname gives error "not implemented" on Win32, see <http://mxr.mozilla.org/comm-central/source/mozilla/toolkit/components/startup/src/nsUserInfoWin.cpp>. I'd be useful for e.g. bug 29593.
If you don't have your full name filled out it will return NS_ERROR_NOT_AVAILABLE. If there is any other type of error it will return NS_ERROR_FAILURE. If you don't have your full name filled out, you can fill it in by following these steps if you are not part of a domain: 1) Go to Control Panel\System and Security\Administrative Tools 2) Open Computer Management 3) Expand Local Users and Groups on the left 4) Click on Users on the left 5) Double click on your username on the right If you are part of a domain you will need to use Active Directory on the domain controller.
Attachment #556417 - Flags: review?(robert.bugzilla)
Everything used by the way, I believe, is supported on Windows 2000 and up which I think is our minimum supported version.
Comment on attachment 556417 [details] [diff] [review] Patch for implementation of nsIUserInfo Moving review request because rs has a bunch of silent software update reviews from me which are higher priority than this task.
Attachment #556417 - Flags: review?(robert.bugzilla) → review?(dtownsend)
Comment on attachment 556417 [details] [diff] [review] Patch for implementation of nsIUserInfo Robert Strong mentioned that you could do this review Jim. (No rush this is pretty low priority)
Attachment #556417 - Flags: review?(dtownsend) → review?(jmathies)
Comment on attachment 556417 [details] [diff] [review] Patch for implementation of nsIUserInfo grabbing this info feels a little creepy. I hope we are treating it appropriately!
Attachment #556417 - Flags: review?(jmathies) → review+
I think the intent is to use it for Thunderbird when composing new emails on Windows. Pushed to try: https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=97bc44057367
Try results look good, but before I push this I just need to setup a mingw build environment first and test a build.
Thanks, Brian, for the work. Appreciated. Is a Windows 7 user, when buying a pre-installed notebook, prompted to enter his name when using the computer the first time? (What about Windows XP, used by 40%? Windows 7 is now the majority, though.)
> Is a Windows 7 user, when buying a pre-installed notebook, prompted to enter his name when using the computer the first time? (What about Windows XP, used by 40%? Windows 7 is now the majority, though.) Sorry I don't know the answer to that question.
> - LIBS="$LIBS -luuid -lgdi32 -lwinmm -lwsock32" > + LIBS="$LIBS -luuid -lgdi32 -lwinmm -lwsock32 -lsecur32 -lnetapi32" Is it possible to make an autoconfig test for this, set an #define when found, and only then add the dependencies, and protect the C++ code with that define? Or are these deps uncritical (any supported Windows would have them)?
Should be supported by all since Windows 2000 and up, just want to try with mingw to be sure though. I have another task I need to get mingw up for anyway so might as well do this at the same time.
Do we still want this patch? If not we should WONTFIX this bug. Just cleaning up some old bugs in my list.
yes. Seems like the patch is fine, has review, and ready for commit.
OK thanks. I'm not sure if it ever ran through try so I'll do that now and then I'll push it myself. I never did get mingw builds to work to try it out, but if it doesn't we can fix that in a follow-up once someone complains.
- Rebased patch to m-c tip - Changed to newer mozilla::ArrayLength form NS_ARRAY_LENGTH - Added NS_ENSURE_ARG_POINTER to the start of every function Carrying forward r+. Pushed to try, will push to mozilla-inbound if that succeeds.
Attachment #678001 - Attachment description: Patch v1. → Patch v2.
Target Milestone: --- → mozilla19
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.