Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(34)

Issue 10810021: Register Chrome in Active Setup to be called by Windows at user login. (Closed)

Created:
8 years, 5 months ago by gab
Modified:
8 years, 5 months ago
CC:
chromium-reviews, grt+watch_chromium.org, grt (UTC plus 2)
Visibility:
Public.

Description

Register Chrome in Active Setup to be called by Windows at user login. This commit also adds --configure-user-settings flag to setup.exe which Active Setup will invoke at user login on system-level installs. For now this flag doesn't do anything (it will be implemented in a follow up CL). Active Setup design doc: https://docs.google.com/a/chromium.org/document/d/1yQdG1wMDKi_lf0i8bk6P2_BWqudRb-ap6Y2C5f1RO2w/edit This CL also introduces ScopedTokenPrivilege, a class meant to give the current process's token privileges for a short period of time while some critical operations (loading other users' registry hives) are happening. BUG=132825 TEST=From an admin prompt: setup_unittests.exe --gtest_filter=SetupUtilTest.ScopedTokenPrivilege* Windows only calls Active Setup's StubPath (i.e. setup.exe --configure-user-settings) if: 1) It never called it before (i.e. the HKCU equivalence of the Chrome Active Setup doesn't exist for this user). or 2) The version in HKLM is greater than the version in HKCU (i.e. Chrome has been updated since the last user login). Uninstalling Chrome deletes the HKCU entries for all users (logged in or not). Later reinstalling an older version of Chrome (just reinstall and manually hack the HKLM Active Setup version) has every user go through Active Setup again (if we didn't delete the values ourselves it wouldn't... which is why I had to introduce 75% of this CL simply to delete the HKCU Active Setup entries created by Windows for every user...) Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=147667

Patch Set 1 : #

Total comments: 18

Patch Set 2 : address tommi's comments #

Patch Set 3 : rebase onto 147656 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+375 lines, -5 lines) Patch
M chrome/installer/setup/install_worker.h View 1 2 1 chunk +10 lines, -0 lines 0 comments Download
M chrome/installer/setup/install_worker.cc View 1 2 4 chunks +51 lines, -1 line 0 comments Download
M chrome/installer/setup/setup_main.cc View 1 2 1 chunk +8 lines, -0 lines 0 comments Download
M chrome/installer/setup/setup_util.h View 2 chunks +39 lines, -0 lines 0 comments Download
M chrome/installer/setup/setup_util.cc View 1 2 chunks +44 lines, -0 lines 0 comments Download
M chrome/installer/setup/setup_util_unittest.cc View 1 5 chunks +89 lines, -4 lines 0 comments Download
M chrome/installer/setup/uninstall.cc View 1 2 3 chunks +127 lines, -0 lines 0 comments Download
M chrome/installer/util/util_constants.h View 1 2 1 chunk +1 line, -0 lines 0 comments Download
M chrome/installer/util/util_constants.cc View 1 2 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
gab
Hey Tommi, could you please take a look at this. Although it looks like a ...
8 years, 5 months ago (2012-07-19 20:57:08 UTC) #1
tommi (sloooow) - chröme
lgtm with a few comments. http://codereview.chromium.org/10810021/diff/11001/chrome/installer/setup/install_worker.cc File chrome/installer/setup/install_worker.cc (right): http://codereview.chromium.org/10810021/diff/11001/chrome/installer/setup/install_worker.cc#newcode1230 chrome/installer/setup/install_worker.cc:1230: DWORD(1U), true); static_cast<DWORD>(1U). Is ...
8 years, 5 months ago (2012-07-20 09:59:25 UTC) #2
gab
Thanks! http://codereview.chromium.org/10810021/diff/11001/chrome/installer/setup/install_worker.cc File chrome/installer/setup/install_worker.cc (right): http://codereview.chromium.org/10810021/diff/11001/chrome/installer/setup/install_worker.cc#newcode1230 chrome/installer/setup/install_worker.cc:1230: DWORD(1U), true); On 2012/07/20 09:59:25, tommi wrote: > ...
8 years, 5 months ago (2012-07-20 14:45:47 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gab@chromium.org/10810021/7002
8 years, 5 months ago (2012-07-20 14:47:08 UTC) #4
commit-bot: I haz the power
Failed to apply patch for chrome/installer/setup/uninstall.cc: While running patch -p1 --forward --force; patching file chrome/installer/setup/uninstall.cc ...
8 years, 5 months ago (2012-07-20 14:47:15 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gab@chromium.org/10810021/11003
8 years, 5 months ago (2012-07-20 14:50:51 UTC) #6
commit-bot: I haz the power
Failed to apply patch for chrome/installer/setup/uninstall.cc: While running patch -p1 --forward --force; patching file chrome/installer/setup/uninstall.cc ...
8 years, 5 months ago (2012-07-20 14:51:00 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gab@chromium.org/10810021/5014
8 years, 5 months ago (2012-07-20 15:16:53 UTC) #8
commit-bot: I haz the power
8 years, 5 months ago (2012-07-20 16:53:21 UTC) #9
Change committed as 147667

Powered by Google App Engine
This is Rietveld 408576698