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

Unified Diff: ppapi/proxy/pdf_resource_unittest.cc

Issue 1329213002: PDF: Use chrome.resourcesPrivate instead of Pepper to display translated strings (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@i18n_api
Patch Set: Remove test cases Created 5 years, 3 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 | « ppapi/proxy/pdf_resource.cc ('k') | ppapi/proxy/ppapi_messages.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/proxy/pdf_resource_unittest.cc
diff --git a/ppapi/proxy/pdf_resource_unittest.cc b/ppapi/proxy/pdf_resource_unittest.cc
index 0b228722ff2cfd2288e6abd2ac98a11ef9254680..7bea178e172dbf296a3e4a096c105f29c476b4ee 100644
--- a/ppapi/proxy/pdf_resource_unittest.cc
+++ b/ppapi/proxy/pdf_resource_unittest.cc
@@ -27,37 +27,6 @@ typedef PluginProxyTest PDFResourceTest;
} // namespace
-TEST_F(PDFResourceTest, GetLocalizedString) {
- const PPB_PDF* pdf_iface = thunk::GetPPB_PDF_Thunk();
-
- std::string expected_string = "hello";
- PpapiPluginMsg_PDF_GetLocalizedStringReply reply_msg(expected_string);
- ResourceSyncCallHandler handler(
- &sink(),
- PpapiHostMsg_PDF_GetLocalizedString::ID,
- PP_OK,
- reply_msg);
- sink().AddFilter(&handler);
-
- PP_Var var = pdf_iface->GetLocalizedString(
- pp_instance(), PP_RESOURCESTRING_PDFGETPASSWORD);
-
- {
- ProxyAutoLock lock;
- ScopedPPVar release_var(ScopedPPVar::PassRef(), var);
- StringVar* string_var = StringVar::FromPPVar(var);
- ASSERT_TRUE(string_var != NULL);
- std::string actual_string = string_var->value();
-
- ASSERT_EQ(PpapiHostMsg_PDF_GetLocalizedString::ID,
- handler.last_handled_msg().type());
- ASSERT_EQ(expected_string, actual_string);
- }
-
- // Remove the filter or it will be destroyed before the sink() is destroyed.
- sink().RemoveFilter(&handler);
-}
-
TEST_F(PDFResourceTest, SearchString) {
ProxyAutoLock lock;
// Instantiate a resource explicitly so we can specify the locale.
« no previous file with comments | « ppapi/proxy/pdf_resource.cc ('k') | ppapi/proxy/ppapi_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698