Completely revert all my IPC work to see if this was what regressed the page cycler.

chrome/common/common_param_traits.h

 // Copyright (c) 2010 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.
 
 // This file is used to define IPC::ParamTraits<> specializations for a number
 // of types so that they can be serialized over IPC.  IPC::ParamTraits<>
 // specializations for basic types (like int and std::string) and types in the
 // 'base' project can be found in ipc/ipc_message_utils.h.  This file contains
 // specializations for types that are shared by more than one child process.
 
 #ifndef CHROME_COMMON_COMMON_PARAM_TRAITS_H_
 #define CHROME_COMMON_COMMON_PARAM_TRAITS_H_
 #pragma once
 
 #include <vector>
 
 #include "app/surface/transport_dib.h"
 #include "chrome/common/content_settings.h"
 #include "chrome/common/geoposition.h"
 #include "chrome/common/page_zoom.h"
 #include "gfx/native_widget_types.h"
 #include "ipc/ipc_message_utils.h"
 #include "net/base/upload_data.h"
 #include "net/url_request/url_request_status.h"
 #include "printing/native_metafile.h"
+#include "webkit/glue/password_form.h"
 #include "webkit/glue/webcursor.h"
 #include "webkit/glue/window_open_disposition.h"
 
 // Forward declarations.
 class GURL;
 class SkBitmap;
 class DictionaryValue;
 class ListValue;
 struct ThumbnailScore;
 class URLRequestStatus;
-class WebCursor;
 
 namespace gfx {
 class Point;
 class Rect;
 class Size;
 }  // namespace gfx
 
 namespace printing {
 struct PageRange;
 }  // namespace printing
@@ skipping 156 matching lines @@
   }
 };
 
 
 template <>
 struct ParamTraits<WebCursor> {
   typedef WebCursor param_type;
   static void Write(Message* m, const param_type& p) {
     p.Serialize(m);
   }
-  static bool Read(const Message* m, void** iter, param_type* r)  {
+  static bool Read(const Message* m, void** iter, param_type* r) {
     return r->Deserialize(m, iter);
   }
   static void Log(const param_type& p, std::wstring* l) {
     l->append(L"<WebCursor>");
   }
 };
 
 
 template <>
 struct ParamTraits<webkit_glue::WebApplicationInfo> {
@@ skipping 137 matching lines @@
 struct ParamTraits<Geoposition::ErrorCode> {
   typedef Geoposition::ErrorCode param_type;
   static void Write(Message* m, const param_type& p);
   static bool Read(const Message* m, void** iter, param_type* p);
   static void Log(const param_type& p, std::wstring* l);
 };
 
 template <>
 struct ParamTraits<webkit_glue::PasswordForm> {
   typedef webkit_glue::PasswordForm param_type;
-  static void Write(Message* m, const param_type& p);
-  static bool Read(const Message* m, void** iter, param_type* p);
-  static void Log(const param_type& p, std::wstring* l);
+  static void Write(Message* m, const param_type& p) {
+    WriteParam(m, p.signon_realm);
+    WriteParam(m, p.origin);
+    WriteParam(m, p.action);
+    WriteParam(m, p.submit_element);
+    WriteParam(m, p.username_element);
+    WriteParam(m, p.username_value);
+    WriteParam(m, p.password_element);
+    WriteParam(m, p.password_value);
+    WriteParam(m, p.old_password_element);
+    WriteParam(m, p.old_password_value);
+    WriteParam(m, p.ssl_valid);
+    WriteParam(m, p.preferred);
+    WriteParam(m, p.blacklisted_by_user);
+  }
+  static bool Read(const Message* m, void** iter, param_type* p) {
+    return
+        ReadParam(m, iter, &p->signon_realm) &&
+        ReadParam(m, iter, &p->origin) &&
+        ReadParam(m, iter, &p->action) &&
+        ReadParam(m, iter, &p->submit_element) &&
+        ReadParam(m, iter, &p->username_element) &&
+        ReadParam(m, iter, &p->username_value) &&
+        ReadParam(m, iter, &p->password_element) &&
+        ReadParam(m, iter, &p->password_value) &&
+        ReadParam(m, iter, &p->old_password_element) &&
+        ReadParam(m, iter, &p->old_password_value) &&
+        ReadParam(m, iter, &p->ssl_valid) &&
+        ReadParam(m, iter, &p->preferred) &&
+        ReadParam(m, iter, &p->blacklisted_by_user);
+  }
+  static void Log(const param_type& p, std::wstring* l) {
+    l->append(L"<PasswordForm>");
+  }
 };
 
 template <>
 struct ParamTraits<printing::PageRange> {
   typedef printing::PageRange param_type;
   static void Write(Message* m, const param_type& p);
+
   static bool Read(const Message* m, void** iter, param_type* r);
+
   static void Log(const param_type& p, std::wstring* l);
 };
 
 template <>
 struct ParamTraits<printing::NativeMetafile> {
   typedef printing::NativeMetafile param_type;
   static void Write(Message* m, const param_type& p);
+
   static bool Read(const Message* m, void** iter, param_type* r);
+
   static void Log(const param_type& p, std::wstring* l);
 };
 
 }  // namespace IPC
 
 #endif  // CHROME_COMMON_COMMON_PARAM_TRAITS_H_