Commit ea173bd3 authored by S. Ganesh's avatar S. Ganesh Committed by Commit Bot

Fix InstallServiceWorkItemTest deletes Chrome key under ClientState.

Greg discovered that InstallServiceWorkItemTest.Do_ServiceName will
create Google\Update\ClientState\{8A69D345-D564-463c-AFF1-A69D9E530F96}
if it does not exist and then delete it. If the test is run on a
development machine, it will corrupt the stable Chrome install.

Bug: 1057233
Change-Id: I5e71c6ab9776f376dcffe0a415bfb084baec8e26
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086353
Auto-Submit: S. Ganesh <ganesh@chromium.org>
Reviewed-by: default avatarGreg Thompson <grt@chromium.org>
Commit-Queue: Greg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#746738}
parent 9e5b7a04
...@@ -154,7 +154,7 @@ TEST_F(InstallServiceWorkItemTest, Do_ServiceName) { ...@@ -154,7 +154,7 @@ TEST_F(InstallServiceWorkItemTest, Do_ServiceName) {
EXPECT_EQ(0UL, EXPECT_EQ(0UL,
GetImpl(item.get())->GetCurrentServiceName().find(kServiceName)); GetImpl(item.get())->GetCurrentServiceName().find(kServiceName));
EXPECT_EQ(ERROR_SUCCESS, key.DeleteKey(L"")); EXPECT_EQ(ERROR_SUCCESS, key.DeleteValue(kServiceName));
} }
} // namespace installer } // namespace installer
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment