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

Side by Side Diff: content/common/frame_messages.h

Issue 1409693009: Fix leaking of RenderFrames. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased on ToT and couple of more nits fixes. Created 5 years 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 unified diff | Download patch
« no previous file with comments | « content/browser/site_per_process_browsertest.cc ('k') | content/renderer/render_frame_impl.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // IPC messages for interacting with frames. 5 // IPC messages for interacting with frames.
6 // Multiply-included message file, hence no include guard. 6 // Multiply-included message file, hence no include guard.
7 7
8 #include "cc/surfaces/surface_id.h" 8 #include "cc/surfaces/surface_id.h"
9 #include "cc/surfaces/surface_sequence.h" 9 #include "cc/surfaces/surface_sequence.h"
10 #include "content/common/content_export.h" 10 #include "content/common/content_export.h"
(...skipping 505 matching lines...) Expand 10 before | Expand all | Expand 10 after
516 // disowned. 516 // disowned.
517 IPC_MESSAGE_ROUTED1(FrameMsg_UpdateOpener, int /* opener_routing_id */) 517 IPC_MESSAGE_ROUTED1(FrameMsg_UpdateOpener, int /* opener_routing_id */)
518 518
519 // Requests that the RenderFrame send back a response after waiting for the 519 // Requests that the RenderFrame send back a response after waiting for the
520 // commit, activation and frame swap of the current DOM tree in blink. 520 // commit, activation and frame swap of the current DOM tree in blink.
521 IPC_MESSAGE_ROUTED1(FrameMsg_VisualStateRequest, uint64 /* id */) 521 IPC_MESSAGE_ROUTED1(FrameMsg_VisualStateRequest, uint64 /* id */)
522 522
523 // Instructs the renderer to create a new RenderFrame object. 523 // Instructs the renderer to create a new RenderFrame object.
524 IPC_MESSAGE_CONTROL1(FrameMsg_NewFrame, FrameMsg_NewFrame_Params /* params */) 524 IPC_MESSAGE_CONTROL1(FrameMsg_NewFrame, FrameMsg_NewFrame_Params /* params */)
525 525
526 // Instructs the renderer to delete the RenderFrame.
527 IPC_MESSAGE_ROUTED0(FrameMsg_Delete)
528
526 // Instructs the renderer to create a new RenderFrameProxy object with 529 // Instructs the renderer to create a new RenderFrameProxy object with
527 // |routing_id|. |render_view_routing_id| identifies the 530 // |routing_id|. |render_view_routing_id| identifies the
528 // RenderView to be associated with this proxy. The new proxy's opener should 531 // RenderView to be associated with this proxy. The new proxy's opener should
529 // be set to the object identified by |opener_routing_id|, or to null if that 532 // be set to the object identified by |opener_routing_id|, or to null if that
530 // is MSG_ROUTING_NONE. The new proxy should be created as a child of the 533 // is MSG_ROUTING_NONE. The new proxy should be created as a child of the
531 // object identified by |parent_routing_id| or as top level if that is 534 // object identified by |parent_routing_id| or as top level if that is
532 // MSG_ROUTING_NONE. 535 // MSG_ROUTING_NONE.
533 IPC_MESSAGE_CONTROL5(FrameMsg_NewFrameProxy, 536 IPC_MESSAGE_CONTROL5(FrameMsg_NewFrameProxy,
534 int /* routing_id */, 537 int /* routing_id */,
535 int /* render_view_routing_id */, 538 int /* render_view_routing_id */,
(...skipping 780 matching lines...) Expand 10 before | Expand all | Expand 10 after
1316 IPC_MESSAGE_ROUTED1(FrameHostMsg_ShowPopup, 1319 IPC_MESSAGE_ROUTED1(FrameHostMsg_ShowPopup,
1317 FrameHostMsg_ShowPopup_Params) 1320 FrameHostMsg_ShowPopup_Params)
1318 IPC_MESSAGE_ROUTED0(FrameHostMsg_HidePopup) 1321 IPC_MESSAGE_ROUTED0(FrameHostMsg_HidePopup)
1319 1322
1320 #endif 1323 #endif
1321 1324
1322 // Adding a new message? Stick to the sort order above: first platform 1325 // Adding a new message? Stick to the sort order above: first platform
1323 // independent FrameMsg, then ifdefs for platform specific FrameMsg, then 1326 // independent FrameMsg, then ifdefs for platform specific FrameMsg, then
1324 // platform independent FrameHostMsg, then ifdefs for platform specific 1327 // platform independent FrameHostMsg, then ifdefs for platform specific
1325 // FrameHostMsg. 1328 // FrameHostMsg.
OLDNEW
« no previous file with comments | « content/browser/site_per_process_browsertest.cc ('k') | content/renderer/render_frame_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698