Move the rest of the content browser->renderer messages to content. Also move drag related messa...

chrome/common/render_messages.h

Index: chrome/common/render_messages.h
===================================================================
--- chrome/common/render_messages.h	(revision 79031)
+++ chrome/common/render_messages.h	(working copy)
@@ -53,7 +53,6 @@
 #include "webkit/glue/webaccessibility.h"
 #include "webkit/glue/webcookie.h"
 #include "webkit/glue/webcursor.h"
-#include "webkit/glue/webdropdata.h"
 #include "webkit/plugins/npapi/webplugin.h"
 #include "webkit/plugins/npapi/webplugininfo.h"
 
@@ -136,15 +135,6 @@
   static void Log(const param_type& p, std::string* l);
 };
 
-// Traits for WebDropData
-template <>
-struct ParamTraits<WebDropData> {
-  typedef WebDropData 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::string* l);
-};
-
 #if defined(OS_POSIX)
 
 // TODO(port): this shouldn't exist. However, the plugin stuff is really using
@@ -322,88 +312,6 @@
                      GURL /* url */,
                      ContentSettings /* content_settings */)
 
-// Change encoding of page in the renderer.
-IPC_MESSAGE_ROUTED1(ViewMsg_SetPageEncoding,
-                    std::string /*new encoding name*/)
-
-// Reset encoding of page in the renderer back to default.
-IPC_MESSAGE_ROUTED0(ViewMsg_ResetPageEncodingToDefault)
-
-// Requests the renderer to reserve a range of page ids.
-IPC_MESSAGE_ROUTED1(ViewMsg_ReservePageIDRange,
-                    int  /* size_of_range */)
-
-// D&d drop target messages.
-IPC_MESSAGE_ROUTED4(ViewMsg_DragTargetDragEnter,
-                    WebDropData /* drop_data */,
-                    gfx::Point /* client_pt */,
-                    gfx::Point /* screen_pt */,
-                    WebKit::WebDragOperationsMask /* ops_allowed */)
-IPC_MESSAGE_ROUTED3(ViewMsg_DragTargetDragOver,
-                    gfx::Point /* client_pt */,
-                    gfx::Point /* screen_pt */,
-                    WebKit::WebDragOperationsMask /* ops_allowed */)
-IPC_MESSAGE_ROUTED0(ViewMsg_DragTargetDragLeave)
-IPC_MESSAGE_ROUTED2(ViewMsg_DragTargetDrop,
-                    gfx::Point /* client_pt */,
-                    gfx::Point /* screen_pt */)
-
-// Notifies the renderer of updates in mouse position of an in-progress
-// drag.  if |ended| is true, then the user has ended the drag operation.
-IPC_MESSAGE_ROUTED4(ViewMsg_DragSourceEndedOrMoved,
-                    gfx::Point /* client_pt */,
-                    gfx::Point /* screen_pt */,
-                    bool /* ended */,
-                    WebKit::WebDragOperation /* drag_operation */)
-
-// Notifies the renderer that the system DoDragDrop call has ended.
-IPC_MESSAGE_ROUTED0(ViewMsg_DragSourceSystemDragEnded)
-
-// Used to tell a render view whether it should expose various bindings
-// that allow JS content extended privileges.  See BindingsPolicy for valid
-// flag values.
-IPC_MESSAGE_ROUTED1(ViewMsg_AllowBindings,
-                    int /* enabled_bindings_flags */)
-
-// Tell the renderer to add a property to the WebUI binding object.  This
-// only works if we allowed WebUI bindings.
-IPC_MESSAGE_ROUTED2(ViewMsg_SetWebUIProperty,
-                    std::string /* property_name */,
-                    std::string /* property_value_json */)
-
-// This message starts/stop monitoring the input method status of the focused
-// edit control of a renderer process.
-// Parameters
-// * is_active (bool)
-//   Indicates if an input method is active in the browser process.
-//   The possible actions when a renderer process receives this message are
-//   listed below:
-//     Value Action
-//     true  Start sending IPC message ViewHostMsg_ImeUpdateTextInputState
-//           to notify the input method status of the focused edit control.
-//     false Stop sending IPC message ViewHostMsg_ImeUpdateTextInputState.
-IPC_MESSAGE_ROUTED1(ViewMsg_SetInputMethodActive,
-                    bool /* is_active */)
-
-// This message sends a string being composed with an input method.
-IPC_MESSAGE_ROUTED4(
-    ViewMsg_ImeSetComposition,
-    string16, /* text */
-    std::vector<WebKit::WebCompositionUnderline>, /* underlines */
-    int, /* selectiont_start */
-    int /* selection_end */)
-
-// This message confirms an ongoing composition.
-IPC_MESSAGE_ROUTED1(ViewMsg_ImeConfirmComposition,
-                    string16 /* text */)
-
-// Used to notify the render-view that we have received a target URL. Used
-// to prevent target URLs spamming the browser.
-IPC_MESSAGE_ROUTED0(ViewMsg_UpdateTargetURL_ACK)
-
-// Sets the alternate error page URL (link doctor) for the renderer process.
-IPC_MESSAGE_ROUTED1(ViewMsg_SetAltErrorPageURL, GURL)
-
 // Install the first missing pluign.
 IPC_MESSAGE_ROUTED0(ViewMsg_InstallMissingPlugin)
 
@@ -418,9 +326,6 @@
 // Tells the render view a prerendered page is about to be displayed.
 IPC_MESSAGE_ROUTED0(ViewMsg_DisplayPrerenderedPage)
 
-IPC_MESSAGE_ROUTED1(ViewMsg_RunFileChooserResponse,
-                    std::vector<FilePath> /* selected files */)
-
 // Used to instruct the RenderView to go into "view source" mode.
 IPC_MESSAGE_ROUTED0(ViewMsg_EnableViewSourceMode)
 
@@ -449,24 +354,6 @@
                            a preferred image size to request, such as when
                            downloading the favicon */)
 
-// When a renderer sends a ViewHostMsg_Focus to the browser process,
-// the browser has the option of sending a ViewMsg_CantFocus back to
-// the renderer.
-IPC_MESSAGE_ROUTED0(ViewMsg_CantFocus)
-
-// Instructs the renderer to invoke the frame's shouldClose method, which
-// runs the onbeforeunload event handler.  Expects the result to be returned
-// via ViewHostMsg_ShouldClose.
-IPC_MESSAGE_ROUTED0(ViewMsg_ShouldClose)
-
-// Instructs the renderer to close the current page, including running the
-// onunload event handler. See the struct in render_messages.h for more.
-//
-// Expects a ClosePage_ACK message when finished, where the parameters are
-// echoed back.
-IPC_MESSAGE_ROUTED1(ViewMsg_ClosePage,
-                    ViewMsg_ClosePage_Params)
-
 // Asks the renderer to send back stats on the WebCore cache broken down by
 // resource types.
 IPC_MESSAGE_CONTROL0(ViewMsg_GetCacheResourceStats)
@@ -483,14 +370,6 @@
 // Asks the renderer to send back V8 heap stats.
 IPC_MESSAGE_CONTROL0(ViewMsg_GetV8HeapStats)
 
-// Notifies the renderer about ui theme changes
-IPC_MESSAGE_ROUTED0(ViewMsg_ThemeChanged)
-
-// Notifies the renderer that a paint is to be generated for the rectangle
-// passed in.
-IPC_MESSAGE_ROUTED1(ViewMsg_Repaint,
-                    gfx::Size /* The view size to be repainted */)
-
 // Posts a message to the renderer.
 IPC_MESSAGE_ROUTED3(ViewMsg_HandleMessageFromExternalHost,
                     std::string /* The message */,
@@ -509,10 +388,6 @@
 IPC_MESSAGE_ROUTED1(ViewMsg_AllowScriptToClose,
                     bool /* script_can_close */)
 
-// Notification that a move or resize renderer's containing window has
-// started.
-IPC_MESSAGE_ROUTED0(ViewMsg_MoveOrResizeStarted)
-
 // The browser sends this message in response to all extension api calls.
 IPC_MESSAGE_ROUTED4(ViewMsg_ExtensionResponse,
                     int /* request_id */,
@@ -570,28 +445,6 @@
 IPC_MESSAGE_CONTROL1(ViewMsg_Extension_SetScriptingWhitelist,
                      Extension::ScriptingWhitelist /* extenison ids */)
 
-// Changes the text direction of the currently selected input field (if any).
-IPC_MESSAGE_ROUTED1(ViewMsg_SetTextDirection,
-                    WebKit::WebTextDirection /* direction */)
-
-// Tells the renderer to clear the focused node (if any).
-IPC_MESSAGE_ROUTED0(ViewMsg_ClearFocusedNode)
-
-// Make the RenderView transparent and render it onto a custom background. The
-// background will be tiled in both directions if it is not large enough.
-IPC_MESSAGE_ROUTED1(ViewMsg_SetBackground,
-                    SkBitmap /* background */)
-
-// Reply to ViewHostMsg_RequestMove, ViewHostMsg_ShowView, and
-// ViewHostMsg_ShowWidget to inform the renderer that the browser has
-// processed the move.  The browser may have ignored the move, but it finished
-// processing.  This is used because the renderer keeps a temporary cache of
-// the widget position while these asynchronous operations are in progress.
-IPC_MESSAGE_ROUTED0(ViewMsg_Move_ACK)
-
-// Used to instruct the RenderView to send back updates to the preferred size.
-IPC_MESSAGE_ROUTED1(ViewMsg_EnablePreferredSizeChangedMode, int /*flags*/)
-
 IPC_MESSAGE_ROUTED4(ViewMsg_SearchBoxChange,
                     string16 /* value */,
                     bool /* verbatim */,
@@ -609,36 +462,6 @@
                     int /* selection_start */,
                     int /* selection_end */)
 
-// Used to tell the renderer not to add scrollbars with height and
-// width below a threshold.
-IPC_MESSAGE_ROUTED1(ViewMsg_DisableScrollbarsForSmallWindows,
-                    gfx::Size /* disable_scrollbar_size_limit */)
-
-// Activate/deactivate the RenderView (i.e., set its controls' tint
-// accordingly, etc.).
-IPC_MESSAGE_ROUTED1(ViewMsg_SetActive,
-                    bool /* active */)
-
-#if defined(OS_MACOSX)
-// Let the RenderView know its window has changed visibility.
-IPC_MESSAGE_ROUTED1(ViewMsg_SetWindowVisibility,
-                    bool /* visibile */)
-
-// Let the RenderView know its window's frame has changed.
-IPC_MESSAGE_ROUTED2(ViewMsg_WindowFrameChanged,
-                    gfx::Rect /* window frame */,
-                    gfx::Rect /* content view frame */)
-
-// Tell the renderer that plugin IME has completed.
-IPC_MESSAGE_ROUTED2(ViewMsg_PluginImeCompositionCompleted,
-                    string16 /* text */,
-                    int /* plugin_id */)
-#endif
-
-// Response message to ViewHostMsg_CreateShared/DedicatedWorker.
-// Sent when the worker has started.
-IPC_MESSAGE_ROUTED0(ViewMsg_WorkerCreated)
-
 // Tell the renderer which browser window it's being attached to.
 IPC_MESSAGE_ROUTED1(ViewMsg_UpdateBrowserWindowId,
                     int /* id of browser window */)
@@ -719,12 +542,6 @@
 // message was processed and it can send addition notifications.
 IPC_MESSAGE_ROUTED0(ViewMsg_AccessibilityNotifications_ACK)
 
-// The response to ViewHostMsg_AsyncOpenFile.
-IPC_MESSAGE_ROUTED3(ViewMsg_AsyncOpenFile_ACK,
-                    base::PlatformFileError /* error_code */,
-                    IPC::PlatformFileForTransit /* file descriptor */,
-                    int /* message_id */)
-
 // A classification model for client-side phishing detection.
 // The given file contains an encoded safe_browsing::ClientSideModel
 // protocol buffer.
@@ -738,11 +555,6 @@
 // which it has started loading.
 IPC_MESSAGE_ROUTED1(ViewMsg_StartPhishingDetection, GURL)
 
-// Tells the renderer that the network state has changed and that
-// window.navigator.onLine should be updated for all WebViews.
-IPC_MESSAGE_ROUTED1(ViewMsg_NetworkStateChanged,
-                    bool /* online */)
-
 //-----------------------------------------------------------------------------
 // TabContents messages
 // These are messages sent from the renderer to the browser process.
@@ -1231,21 +1043,6 @@
 IPC_MESSAGE_ROUTED1(ViewHostMsg_RunFileChooser,
                     ViewHostMsg_RunFileChooser_Params)
 
-// Used to tell the parent the user started dragging in the content area. The
-// WebDropData struct contains contextual information about the pieces of the
-// page the user dragged. The parent uses this notification to initiate a
-// drag session at the OS level.
-IPC_MESSAGE_ROUTED4(ViewHostMsg_StartDragging,
-                    WebDropData /* drop_data */,
-                    WebKit::WebDragOperationsMask /* ops_allowed */,
-                    SkBitmap /* image */,
-                    gfx::Point /* image_offset */)
-
-// The page wants to update the mouse cursor during a drag & drop operation.
-// |is_drop_target| is true if the mouse is over a valid drop target.
-IPC_MESSAGE_ROUTED1(ViewHostMsg_UpdateDragCursor,
-                    WebKit::WebDragOperation /* drag_operation */)
-
 // Tells the browser to move the focus to the next (previous if reverse is
 // true) focusable element.
 IPC_MESSAGE_ROUTED1(ViewHostMsg_TakeFocus, bool /* reverse */)
@@ -1415,18 +1212,6 @@
 IPC_MESSAGE_ROUTED1(ViewHostMsg_InstallApplication,
                     WebApplicationInfo)
 
-// Provides the result from running OnMsgShouldClose.  |proceed| matches the
-// return value of the the frame's shouldClose method (which includes the
-// onbeforeunload handler): true if the user decided to proceed with leaving
-// the page.
-IPC_MESSAGE_ROUTED1(ViewHostMsg_ShouldClose_ACK,
-                    bool /* proceed */)
-
-// Indicates that the current page has been closed, after a ClosePage
-// message. The parameters are just echoed from the ClosePage request.
-IPC_MESSAGE_ROUTED1(ViewHostMsg_ClosePage_ACK,
-                    ViewMsg_ClosePage_Params)
-
 IPC_MESSAGE_ROUTED4(ViewHostMsg_DidDownloadFavicon,
                     int /* Identifier of the request */,
                     GURL /* URL of the image */,
@@ -1609,47 +1394,6 @@
                     uint64 /* surface_id */)
 #endif
 
-// A renderer sends this to the browser process when it wants to create a
-// worker.  The browser will create the worker process if necessary, and
-// will return the route id on success.  On error returns MSG_ROUTING_NONE.
-IPC_SYNC_MESSAGE_CONTROL1_1(ViewHostMsg_CreateWorker,
-                            ViewHostMsg_CreateWorker_Params,
-                            int /* route_id */)
-
-// This message is sent to the browser to see if an instance of this shared
-// worker already exists. If so, it returns exists == true. If a
-// non-empty name is passed, also validates that the url matches the url of
-// the existing worker. If a matching worker is found, the passed-in
-// document_id is associated with that worker, to ensure that the worker
-// stays alive until the document is detached.
-// The route_id returned can be used to forward messages to the worker via
-// ForwardToWorker if it exists, otherwise it should be passed in to any
-// future call to CreateWorker to avoid creating duplicate workers.
-IPC_SYNC_MESSAGE_CONTROL1_3(ViewHostMsg_LookupSharedWorker,
-                            ViewHostMsg_CreateWorker_Params,
-                            bool /* exists */,
-                            int /* route_id */,
-                            bool /* url_mismatch */)
-
-// A renderer sends this to the browser process when a document has been
-// detached. The browser will use this to constrain the lifecycle of worker
-// processes (SharedWorkers are shut down when their last associated document
-// is detached).
-IPC_MESSAGE_CONTROL1(ViewHostMsg_DocumentDetached,
-                     uint64 /* document_id */)
-
-// Sent if the worker object has sent a ViewHostMsg_CreateDedicatedWorker
-// message and not received a ViewMsg_WorkerCreated reply, but in the
-// mean time it's destroyed.  This tells the browser to not create the queued
-// worker.
-IPC_MESSAGE_CONTROL1(ViewHostMsg_CancelCreateDedicatedWorker,
-                     int /* route_id */)
-
-// Wraps an IPC message that's destined to the worker on the renderer->browser
-// hop.
-IPC_MESSAGE_CONTROL1(ViewHostMsg_ForwardToWorker,
-                     IPC::Message /* message */)
-
 // Open a channel to all listening contexts owned by the extension with
 // the given ID.  This always returns a valid port ID which can be used for
 // sending messages.  If an error occurred, the opener will be notified