Prevent an extra copy of data types that get passed into IPC messages when...

chrome/browser/renderer_host/render_widget_host_unittest.cc

 // 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.
 
 #include "app/gfx/canvas.h"
 #include "base/basictypes.h"
 #include "base/keyboard_codes.h"
 #include "base/scoped_ptr.h"
 #include "base/shared_memory.h"
 #include "build/build_config.h"
@@ skipping 257 matching lines @@
   EXPECT_TRUE(0 == memcmp(background.getPixels(),
                           view->background().getPixels(),
                           background.getSize()));
 
 #if defined(OS_WIN)
   // A message should have been dispatched telling the renderer about the new
   // background.
   const IPC::Message* set_background =
       process_->sink().GetUniqueMessageMatching(ViewMsg_SetBackground::ID);
   ASSERT_TRUE(set_background);
-  SkBitmap sent_background;
+  Tuple1<SkBitmap> sent_background;
   ViewMsg_SetBackground::Read(set_background, &sent_background);
-  EXPECT_EQ(background.getSize(), sent_background.getSize());
+  EXPECT_EQ(background.getSize(), sent_background.a.getSize());
   EXPECT_TRUE(0 == memcmp(background.getPixels(),
-                          sent_background.getPixels(),
+                          sent_background.a.getPixels(),
                           background.getSize()));
 #else
   // TODO(port): When custom backgrounds are implemented for other ports, this
   // test should work (assuming the background must still be copied into the
   // renderer -- if not, then maybe the test doesn't apply?).
 #endif
 
 #else
   // TODO(port): Mac does not have gfx::Canvas. Maybe we can just change this
   // test to use SkCanvas directly?
@@ skipping 77 matching lines @@
 
   // Now unhide.
   process_->sink().ClearMessages();
   host_->WasRestored();
   EXPECT_FALSE(host_->is_hidden_);
 
   // It should have sent out a restored message with a request to paint.
   const IPC::Message* restored = process_->sink().GetUniqueMessageMatching(
       ViewMsg_WasRestored::ID);
   ASSERT_TRUE(restored);
-  bool needs_repaint;
+  Tuple1<bool> needs_repaint;
   ViewMsg_WasRestored::Read(restored, &needs_repaint);
-  EXPECT_TRUE(needs_repaint);
+  EXPECT_TRUE(needs_repaint.a);
 }
 
 TEST_F(RenderWidgetHostTest, HandleKeyEventsWeSent) {
   NativeWebKeyboardEvent key_event;
   key_event.type = WebInputEvent::KeyDown;
   key_event.modifiers = WebInputEvent::ControlKey;
   key_event.windowsKeyCode = base::VKEY_L;  // non-null made up value.
 
   host_->ForwardKeyboardEvent(key_event);
 
@@ skipping 15 matching lines @@
 TEST_F(RenderWidgetHostTest, IgnoreKeyEventsWeDidntSend) {
   // Send a simulated, unrequested key response. We should ignore this.
   scoped_ptr<IPC::Message> response(
       new ViewHostMsg_HandleInputEvent_ACK(0));
   response->WriteInt(WebInputEvent::KeyDown);
   response->WriteBool(false);
   host_->OnMessageReceived(*response);
 
   EXPECT_FALSE(host_->unhandled_keyboard_event_called());
 }