Closed Bug 581508 Opened 10 years ago Closed 7 years ago

Implement nsIUserInfo on Win32, particularly fullname

Categories

(Toolkit :: Startup and Profile System, defect, minor)

x86
Windows 7
defect
Not set
minor

Tracking

()

RESOLVED FIXED
mozilla19

People

(Reporter: BenB, Assigned: bbondy)

References

Details

Attachments

(1 file, 1 obsolete file)

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.
Assignee: nobody → netzen
Depends on: 29596
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.
Keywords: checkin-needed
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.
Keywords: checkin-needed
Attached patch Patch v2.Splinter Review
- 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 #556417 - Attachment is obsolete: true
Attachment #678001 - Flags: review+
Attachment #678001 - Attachment description: Patch v1. → Patch v2.
https://hg.mozilla.org/mozilla-central/rev/8ee857e02861
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Thanks, everybody!
You need to log in before you can comment on or make changes to this bug.