Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1430)

Unified Diff: chrome/browser/history/text_database_manager_unittest.cc

Issue 2802010: Revert 49594 - Convert page contents grabbing from wide to UTF16. The current... (Closed) Base URL: svn://svn.chromium.org/chrome/branches/437/src/
Patch Set: Created 10 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/history/text_database_manager.cc ('k') | chrome/browser/renderer_host/render_view_host.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/history/text_database_manager_unittest.cc
===================================================================
--- chrome/browser/history/text_database_manager_unittest.cc (revision 50158)
+++ chrome/browser/history/text_database_manager_unittest.cc (working copy)
@@ -1,4 +1,4 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Copyright (c) 2009 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
@@ -6,7 +6,6 @@
#include "base/file_path.h"
#include "base/file_util.h"
#include "base/message_loop.h"
-#include "base/utf_string_conversions.h"
#include "chrome/browser/history/text_database_manager.h"
#include "chrome/browser/history/visit_database.h"
#include "testing/gtest/include/gtest/gtest.h"
@@ -21,23 +20,23 @@
const char* kURL1 = "http://www.google.com/asdf";
const wchar_t* kTitle1 = L"Google A";
-const char* kBody1 = "FOO page one.";
+const wchar_t* kBody1 = L"FOO page one.";
const char* kURL2 = "http://www.google.com/qwer";
const wchar_t* kTitle2 = L"Google B";
-const char* kBody2 = "FOO two.";
+const wchar_t* kBody2 = L"FOO two.";
const char* kURL3 = "http://www.google.com/zxcv";
const wchar_t* kTitle3 = L"Google C";
-const char* kBody3 = "FOO drei";
+const wchar_t* kBody3 = L"FOO drei";
const char* kURL4 = "http://www.google.com/hjkl";
const wchar_t* kTitle4 = L"Google D";
-const char* kBody4 = "FOO lalala four.";
+const wchar_t* kBody4 = L"FOO lalala four.";
const char* kURL5 = "http://www.google.com/uiop";
const wchar_t* kTitle5 = L"Google cinq";
-const char* kBody5 = "FOO page one.";
+const wchar_t* kBody5 = L"FOO page one.";
// This provides a simple implementation of a URL+VisitDatabase using an
// in-memory sqlite connection. The text database manager expects to be able to
@@ -83,7 +82,7 @@
times->push_back(visit_row.visit_time);
manager.AddPageData(GURL(kURL1), visit_row.url_id, visit_row.visit_id,
- visit_row.visit_time, kTitle1, UTF8ToUTF16(kBody1));
+ visit_row.visit_time, kTitle1, kBody1);
exploded.day_of_month++;
visit_row.url_id = 2;
@@ -91,7 +90,7 @@
visit_id = visit_db->AddVisit(&visit_row);
times->push_back(visit_row.visit_time);
manager.AddPageData(GURL(kURL2), visit_row.url_id, visit_row.visit_id,
- visit_row.visit_time, kTitle2, UTF8ToUTF16(kBody2));
+ visit_row.visit_time, kTitle2, kBody2);
exploded.day_of_month++;
visit_row.url_id = 2;
@@ -99,7 +98,7 @@
visit_id = visit_db->AddVisit(&visit_row);
times->push_back(visit_row.visit_time);
manager.AddPageData(GURL(kURL3), visit_row.url_id, visit_row.visit_id,
- visit_row.visit_time, kTitle3, UTF8ToUTF16(kBody3));
+ visit_row.visit_time, kTitle3, kBody3);
// Put the next ones in the next month.
exploded.month++;
@@ -108,7 +107,7 @@
visit_id = visit_db->AddVisit(&visit_row);
times->push_back(visit_row.visit_time);
manager.AddPageData(GURL(kURL4), visit_row.url_id, visit_row.visit_id,
- visit_row.visit_time, kTitle4, UTF8ToUTF16(kBody4));
+ visit_row.visit_time, kTitle4, kBody4);
exploded.day_of_month++;
visit_row.url_id = 2;
@@ -116,7 +115,7 @@
visit_id = visit_db->AddVisit(&visit_row);
times->push_back(visit_row.visit_time);
manager.AddPageData(GURL(kURL5), visit_row.url_id, visit_row.visit_id,
- visit_row.visit_time, kTitle5, UTF8ToUTF16(kBody5));
+ visit_row.visit_time, kTitle5, kBody5);
// Put the first one in again in the second month.
exploded.day_of_month++;
@@ -125,7 +124,7 @@
visit_id = visit_db->AddVisit(&visit_row);
times->push_back(visit_row.visit_time);
manager.AddPageData(GURL(kURL1), visit_row.url_id, visit_row.visit_id,
- visit_row.visit_time, kTitle1, UTF8ToUTF16(kBody1));
+ visit_row.visit_time, kTitle1, kBody1);
}
bool ResultsHaveURL(const std::vector<TextDatabase::Match>& results,
@@ -205,7 +204,7 @@
const GURL url(kURL1);
manager.AddPageURL(url, 0, 0, Time::Now());
manager.AddPageTitle(url, kTitle1);
- manager.AddPageContents(url, UTF8ToUTF16(kBody1));
+ manager.AddPageContents(url, kBody1);
// Check that the page got added.
QueryOptions options;
@@ -239,7 +238,7 @@
// Add a full text indexed entry for that visit.
const GURL url(kURL2);
manager.AddPageURL(url, visit.url_id, visit.visit_id, visit.visit_time);
- manager.AddPageContents(url, UTF8ToUTF16(kBody2));
+ manager.AddPageContents(url, kBody2);
manager.AddPageTitle(url, kTitle2);
// Check that the page got added.
@@ -276,7 +275,7 @@
// The third one has a URL and body.
GURL url3(kURL3);
manager.AddPageURL(url3, 0, 0, Time::Now());
- manager.AddPageContents(url3, UTF8ToUTF16(kBody3));
+ manager.AddPageContents(url3, kBody3);
// Expire stuff very fast. This assumes that the time between the first
// AddPageURL and this line is less than the expiration time (20 seconds).
@@ -343,7 +342,7 @@
EXPECT_EQ(0U, results.size());
// Now add the body, which should be queryable.
- manager.AddPageContents(url, UTF8ToUTF16("Very awesome body"));
+ manager.AddPageContents(url, L"Very awesome body");
manager.GetTextMatches(L"awesome", options, &results, &first_time_searched);
EXPECT_EQ(1U, results.size());
« no previous file with comments | « chrome/browser/history/text_database_manager.cc ('k') | chrome/browser/renderer_host/render_view_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698