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

Unified Diff: content/common/view_messages.h

Issue 208793005: Remove JavaScript execution from RenderViewHost. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: rebase Created 6 years, 9 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 | « content/browser/renderer_host/render_view_host_impl.cc ('k') | content/public/browser/render_view_host.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/common/view_messages.h
diff --git a/content/common/view_messages.h b/content/common/view_messages.h
index 7846859785027242539c7e35fe05ca7a1dcb567f..ea76d7dc26f12ba04237fc497cc58162a4139715 100644
--- a/content/common/view_messages.h
+++ b/content/common/view_messages.h
@@ -699,35 +699,6 @@ IPC_MESSAGE_ROUTED2(ViewMsg_PluginActionAt,
gfx::Point, /* location */
blink::WebPluginAction)
-// Request for the renderer to evaluate an xpath to a frame and execute a
-// javascript: url in that frame's context. The message is completely
-// asynchronous and no corresponding response message is sent back.
-//
-// frame_xpath contains the modified xpath notation to identify an inner
-// subframe (starting from the root frame). It is a concatenation of
-// number of smaller xpaths delimited by '\n'. Each chunk in the string can
-// be evaluated to a frame in its parent-frame's context.
-//
-// Example: /html/body/iframe/\n/html/body/div/iframe/\n/frameset/frame[0]
-// can be broken into 3 xpaths
-// /html/body/iframe evaluates to an iframe within the root frame
-// /html/body/div/iframe evaluates to an iframe within the level-1 iframe
-// /frameset/frame[0] evaluates to first frame within the level-2 iframe
-//
-// jscript_url is the string containing the javascript: url to be executed
-// in the target frame's context. The string should start with "javascript:"
-// and continue with a valid JS text.
-//
-// If the fourth parameter is true the result is sent back to the renderer
-// using the message ViewHostMsg_ScriptEvalResponse.
-// ViewHostMsg_ScriptEvalResponse is passed the ID parameter so that the
-// client can uniquely identify the request.
-IPC_MESSAGE_ROUTED4(ViewMsg_ScriptEvalRequest,
- base::string16, /* frame_xpath */
- base::string16, /* jscript_url */
- int, /* ID */
- bool /* If true, result is sent back. */)
-
// Posts a message from a frame in another process to the current renderer.
IPC_MESSAGE_ROUTED1(ViewMsg_PostMessageEvent,
ViewMsg_PostMessage_Params)
@@ -1177,14 +1148,6 @@ IPC_MESSAGE_ROUTED1(ViewHostMsg_ShowPopup,
IPC_MESSAGE_ROUTED0(ViewHostMsg_HidePopup)
#endif
-// Response from ViewMsg_ScriptEvalRequest. The ID is the parameter supplied
-// to ViewMsg_ScriptEvalRequest. The result has the value returned by the
-// script as its only element, one of Null, Boolean, Integer, Real, Date, or
-// String.
-IPC_MESSAGE_ROUTED2(ViewHostMsg_ScriptEvalResponse,
- int /* id */,
- base::ListValue /* result */)
-
// Result of string search in the page.
// Response to ViewMsg_Find with the results of the requested find-in-page
// search, the number of matches found and the selection rect (in screen
« no previous file with comments | « content/browser/renderer_host/render_view_host_impl.cc ('k') | content/public/browser/render_view_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698