Commit 2a8d287c authored by dcheng@chromium.org's avatar dcheng@chromium.org

Fix SyncNotifierRegistrarHelper test in official builds.

CHECK() messages get stripped out in official builds.

BUG=none


Review URL: https://chromiumcodereview.appspot.com/10826290

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151457 0039d316-1c4b-4281-b951-d872f2087c98
parent b632675b
...@@ -185,8 +185,9 @@ TEST_F(SyncNotifierRegistrarTest, MultipleRegistration) { ...@@ -185,8 +185,9 @@ TEST_F(SyncNotifierRegistrarTest, MultipleRegistration) {
registrar.UpdateRegisteredIds(&handler1, ids); registrar.UpdateRegisteredIds(&handler1, ids);
registrar.DetachFromThreadForTest(); registrar.DetachFromThreadForTest();
EXPECT_DEATH({ registrar.UpdateRegisteredIds(&handler2, ids); }, // We expect a death via CHECK(). We can't match against the CHECK() message
"Duplicate registration: .*"); // though since they are removed in official builds.
EXPECT_DEATH({ registrar.UpdateRegisteredIds(&handler2, ids); }, "");
} }
// Make sure that passing an empty set to UpdateRegisteredIds clears the // Make sure that passing an empty set to UpdateRegisteredIds clears the
......
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