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

Unified Diff: content/browser/renderer_host/pepper/pepper_printing_host_unittest.cc

Issue 10826072: Implement browser side of PPB_Printing resource. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 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
Index: content/browser/renderer_host/pepper/pepper_printing_host_unittest.cc
diff --git a/content/browser/renderer_host/pepper/pepper_printing_host_unittest.cc b/content/browser/renderer_host/pepper/pepper_printing_host_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..a500ba7c53a70f7ce6a441b87dee9389f17fcaa2
--- /dev/null
+++ b/content/browser/renderer_host/pepper/pepper_printing_host_unittest.cc
@@ -0,0 +1,102 @@
+// Copyright (c) 2012 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.
+
+#include "content/browser/renderer_host/pepper/browser_ppapi_host_test.h"
+#include "content/browser/renderer_host/pepper/pepper_print_settings_manager.h"
+#include "content/browser/renderer_host/pepper/pepper_printing_host.h"
+#include "ppapi/c/pp_errors.h"
+#include "ppapi/host/host_message_context.h"
+#include "ppapi/host/ppapi_host.h"
+#include "ppapi/proxy/ppapi_messages.h"
+#include "ppapi/proxy/resource_message_params.h"
+#include "ppapi/proxy/resource_message_test_sink.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace content {
+
+namespace {
+
+class PepperPrintingHostTest
+ : public testing::Test,
+ public BrowserPpapiHostTest {
+ public:
+ PepperPrintingHostTest() {
+ }
+
+ ~PepperPrintingHostTest() {
+ }
+
+ DISALLOW_COPY_AND_ASSIGN(PepperPrintingHostTest);
+};
+
+bool PP_SizeEqual(const PP_Size& lhs, const PP_Size& rhs) {
+ return lhs.width == rhs.width && lhs.height == rhs.height;
brettw 2012/09/18 21:41:38 Check indenting for these functions.
raymes 2012/09/18 23:43:57 Done.
+}
+
+bool PP_RectEqual(const PP_Rect& lhs, const PP_Rect& rhs) {
+ return lhs.point.x == rhs.point.x &&
+ lhs.point.y == rhs.point.y &&
+ PP_SizeEqual(lhs.size, rhs.size);
+}
+
+} // namespace
+
+TEST_F(PepperPrintingHostTest, GetDefaultPrintSettings) {
+ PP_Instance pp_instance = 12345;
+ PP_Resource pp_resource = 67890;
+ PP_PrintSettings_Dev expected_settings = {
+ { { 0, 0 }, { 500, 515 } },
+ { { 25, 35 }, { 300, 720 } },
+ { 600, 700 },
+ 200,
+ PP_PRINTORIENTATION_NORMAL,
+ PP_PRINTSCALINGOPTION_NONE,
+ PP_FALSE,
+ PP_PRINTOUTPUTFORMAT_PDF
+ };
+
+ // Construct the resource host.
+ scoped_ptr<PepperPrintSettingsManager> manager(
+ new MockPepperPrintSettingsManager(expected_settings));
+ PepperPrintingHost printing(GetBrowserPpapiHost()->GetPpapiHost(),
+ pp_instance, pp_resource, manager.Pass());
+
+ // Simulate a message being received.
+ ppapi::proxy::ResourceMessageCallParams call_params(pp_resource, 1);
+ ppapi::host::HostMessageContext context(call_params);
+ int32 result = printing.OnResourceMessageReceived(
+ PpapiHostMsg_Printing_GetDefaultPrintSettings(), &context);
+ EXPECT_EQ(PP_OK_COMPLETIONPENDING, result);
+
+ // This should have sent the Pepper reply to our test sink.
+ ppapi::proxy::ResourceMessageReplyParams reply_params;
+ IPC::Message reply_msg;
+ ASSERT_TRUE(sink().GetFirstResourceReplyMatching(
+ PpapiPluginMsg_Printing_GetDefaultPrintSettingsReply::ID, &reply_params,
+ &reply_msg));
+
+ // Validation of reply.
+ EXPECT_EQ(call_params.sequence(), reply_params.sequence());
+ EXPECT_EQ(PP_OK, reply_params.result());
+ PpapiPluginMsg_Printing_GetDefaultPrintSettingsReply::Schema::Param
+ reply_msg_param;
+ ASSERT_TRUE(PpapiPluginMsg_Printing_GetDefaultPrintSettingsReply::Read(
+ &reply_msg, &reply_msg_param));
+ PP_PrintSettings_Dev actual_settings = reply_msg_param.a;
+
+ EXPECT_TRUE(PP_RectEqual(expected_settings.printable_area,
+ actual_settings.printable_area));
+ EXPECT_TRUE(PP_RectEqual(expected_settings.content_area,
+ actual_settings.content_area));
+ EXPECT_TRUE(PP_SizeEqual(expected_settings.paper_size,
+ actual_settings.paper_size));
+ EXPECT_EQ(expected_settings.dpi, actual_settings.dpi);
+ EXPECT_EQ(expected_settings.orientation, actual_settings.orientation);
+ EXPECT_EQ(expected_settings.print_scaling_option,
+ actual_settings.print_scaling_option);
+ EXPECT_EQ(expected_settings.grayscale, actual_settings.grayscale);
+ EXPECT_EQ(expected_settings.format, actual_settings.format);
+}
+
+} // namespace content

Powered by Google App Engine
This is Rietveld 408576698