Fix the failed case AndroidProviderBackendTest.UpdateFavicon on Android

chrome/browser/history/android/android_provider_backend_unittest.cc

Index: chrome/browser/history/android/android_provider_backend_unittest.cc
diff --git a/chrome/browser/history/android/android_provider_backend_unittest.cc b/chrome/browser/history/android/android_provider_backend_unittest.cc
index 31a84620498684387a3db5916ee3bea32a08d3de..edb2b84250472735403ce2583d6ca732aabfc79e 100644
--- a/chrome/browser/history/android/android_provider_backend_unittest.cc
+++ b/chrome/browser/history/android/android_provider_backend_unittest.cc
@@ -1139,6 +1139,9 @@ TEST_F(AndroidProviderBackendTest, UpdateFavicon) {
   std::vector<unsigned char> data;
   data.push_back('1');
   update_row1.set_favicon(base::RefCountedBytes::TakeVector(&data));
+  // the content of data is swapped, so reset it to compare with
+  // the result of GetFavicon
+  data.push_back('1');

[pkotwicz, 2012/08/20 14:17:29]

You can actually use new RefCountedBytes(data) instead.

   update_args.push_back(UTF8ToUTF16(row1.raw_url()));
   delegate_.ResetDetails();
   ASSERT_TRUE(backend->UpdateHistoryAndBookmarks(update_row1, "url = ?",