| Index: chrome/browser/ui/webui/downloads_dom_handler_browsertest.cc
|
| diff --git a/chrome/browser/ui/webui/downloads_dom_handler_browsertest.cc b/chrome/browser/ui/webui/downloads_dom_handler_browsertest.cc
|
| index b02542bc30b0199c836865fa9d51a9554d41926f..804e65184e57f119f19d73ceed9a2bf9aac8fc4e 100644
|
| --- a/chrome/browser/ui/webui/downloads_dom_handler_browsertest.cc
|
| +++ b/chrome/browser/ui/webui/downloads_dom_handler_browsertest.cc
|
| @@ -37,7 +37,8 @@ bool ListMatches(base::ListValue* left_list, const std::string& right_json) {
|
| if (left_dict->HasKey(iter.key()) &&
|
| left_dict->Get(iter.key(), &left_value) &&
|
| !iter.value().Equals(left_value)) {
|
| - LOG(WARNING) << iter.key();
|
| + LOG(WARNING) << "key \"" << iter.key() << "\" doesn't match ("
|
| + << iter.value() << " vs. " << *left_value << ")";
|
| return false;
|
| }
|
| }
|
| @@ -161,7 +162,7 @@ class DownloadsDOMHandlerTest : public InProcessBrowserTest {
|
| mock_handler_->downloads_list(),
|
| "[{\"file_externally_removed\": false,"
|
| " \"file_name\": \"file\","
|
| - " \"id\": 1,"
|
| + " \"id\": \"1\","
|
| " \"otr\": false,"
|
| " \"since_string\": \"Today\","
|
| " \"state\": \"COMPLETE\","
|
| @@ -200,7 +201,7 @@ IN_PROC_BROWSER_TEST_F(DownloadsDOMHandlerTest, RemoveAll) {
|
| IN_PROC_BROWSER_TEST_F(DownloadsDOMHandlerTest, RemoveOneItem) {
|
| DownloadAnItem();
|
| base::ListValue item;
|
| - item.AppendInteger(1);
|
| + item.AppendString("1");
|
|
|
| mock_handler_->reset_downloads_list();
|
| browser()->profile()->GetPrefs()->SetBoolean(
|
| @@ -231,7 +232,7 @@ IN_PROC_BROWSER_TEST_F(DownloadsDOMHandlerTest, DownloadsRelayed) {
|
| EXPECT_TRUE(ListMatches(
|
| mock_handler_->download_updated(),
|
| "[{\"file_externally_removed\": true,"
|
| - " \"id\": 1}]"));
|
| + " \"id\": \"1\"}]"));
|
|
|
| mock_handler_->reset_downloads_list();
|
| browser()->profile()->GetPrefs()->SetBoolean(
|
|
|