From 3c921389b75b82f6377775c7afc2f4af1b5b2c38 Mon Sep 17 00:00:00 2001 From: "akalin@chromium.org" Date: Fri, 6 May 2011 09:32:50 +0000 Subject: [Sync] Refactor JsArgList a bit. Make JsArgList take over the data of its constructor argument, saving a copy. BUG=None TEST=None R=atwilson@chromium.org Review URL: http://codereview.chromium.org/6933040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84423 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/ui/webui/sync_internals_ui_unittest.cc | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'chrome/browser/ui/webui/sync_internals_ui_unittest.cc') diff --git a/chrome/browser/ui/webui/sync_internals_ui_unittest.cc b/chrome/browser/ui/webui/sync_internals_ui_unittest.cc index 0ae2436..dab0b3e 100644 --- a/chrome/browser/ui/webui/sync_internals_ui_unittest.cc +++ b/chrome/browser/ui/webui/sync_internals_ui_unittest.cc @@ -144,7 +144,7 @@ TEST_F(SyncInternalsUITest, HandleJsEvent) { ListValue args; args.Append(Value::CreateIntegerValue(5)); args.Append(Value::CreateBooleanValue(true)); - GetTestSyncInternalsUI()->HandleJsEvent("testMessage", JsArgList(args)); + GetTestSyncInternalsUI()->HandleJsEvent("testMessage", JsArgList(&args)); } TEST_F(SyncInternalsUITest, HandleJsEventNullService) { @@ -159,7 +159,7 @@ TEST_F(SyncInternalsUITest, HandleJsEventNullService) { ListValue args; args.Append(Value::CreateIntegerValue(5)); args.Append(Value::CreateBooleanValue(true)); - GetTestSyncInternalsUI()->HandleJsEvent("testMessage", JsArgList(args)); + GetTestSyncInternalsUI()->HandleJsEvent("testMessage", JsArgList(&args)); } TEST_F(SyncInternalsUITest, HandleJsMessageReply) { @@ -176,7 +176,7 @@ TEST_F(SyncInternalsUITest, HandleJsMessageReply) { args.Append(Value::CreateIntegerValue(5)); args.Append(Value::CreateBooleanValue(true)); GetTestSyncInternalsUI()->HandleJsMessageReply( - "testMessage", JsArgList(args)); + "testMessage", JsArgList(&args)); } TEST_F(SyncInternalsUITest, HandleJsMessageReplyNullService) { @@ -193,7 +193,7 @@ TEST_F(SyncInternalsUITest, HandleJsMessageReplyNullService) { args.Append(Value::CreateIntegerValue(5)); args.Append(Value::CreateBooleanValue(true)); GetTestSyncInternalsUI()->HandleJsMessageReply( - "testMessage", JsArgList(args)); + "testMessage", JsArgList(&args)); } TEST_F(SyncInternalsUITest, OnWebUISendBasic) { -- cgit v1.1