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

Side by Side Diff: content/browser/frame_host/render_frame_host_impl.h

Issue 464593003: Don't swap out the old RenderFrameHost until the new one commits. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Major rebase Created 6 years, 4 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 unified diff | Download patch | Annotate | Revision Log
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 #ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_ 5 #ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_
6 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_ 6 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_
7 7
8 #include <map> 8 #include <map>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 249 matching lines...) Expand 10 before | Expand all | Expand 10 after
260 // this frame, or create one if it doesn't exist yet, otherwise return 260 // this frame, or create one if it doesn't exist yet, otherwise return
261 // NULL. 261 // NULL.
262 BrowserAccessibilityManager* GetOrCreateBrowserAccessibilityManager(); 262 BrowserAccessibilityManager* GetOrCreateBrowserAccessibilityManager();
263 263
264 #if defined(OS_WIN) 264 #if defined(OS_WIN)
265 void SetParentNativeViewAccessible( 265 void SetParentNativeViewAccessible(
266 gfx::NativeViewAccessible accessible_parent); 266 gfx::NativeViewAccessible accessible_parent);
267 gfx::NativeViewAccessible GetParentNativeViewAccessible() const; 267 gfx::NativeViewAccessible GetParentNativeViewAccessible() const;
268 #endif 268 #endif
269 269
270 // Returns whether this RenderFrameHost has an outstanding cross-site request.
271 // Cleared when we hear the response and start to swap out the old
272 // RenderFrameHost, or if we hear a commit here without a network request.
273 bool HasPendingCrossSiteRequest();
274
275 // Sets whether this RenderFrameHost has an outstanding cross-site request,
276 // for which another renderer will need to run an onunload event handler.
277 // This is called before the first navigation event for this RenderFrameHost,
278 // and cleared when we hear the response or commit.
279 void SetHasPendingCrossSiteRequest(bool has_pending_request);
280
281 protected: 270 protected:
282 friend class RenderFrameHostFactory; 271 friend class RenderFrameHostFactory;
283 272
284 // TODO(nasko): Remove dependency on RenderViewHost here. RenderProcessHost 273 // TODO(nasko): Remove dependency on RenderViewHost here. RenderProcessHost
285 // should be the abstraction needed here, but we need RenderViewHost to pass 274 // should be the abstraction needed here, but we need RenderViewHost to pass
286 // into WebContentsObserver::FrameDetached for now. 275 // into WebContentsObserver::FrameDetached for now.
287 RenderFrameHostImpl(RenderViewHostImpl* render_view_host, 276 RenderFrameHostImpl(RenderViewHostImpl* render_view_host,
288 RenderFrameHostDelegate* delegate, 277 RenderFrameHostDelegate* delegate,
289 FrameTree* frame_tree, 278 FrameTree* frame_tree,
290 FrameTreeNode* frame_tree_node, 279 FrameTreeNode* frame_tree_node,
(...skipping 144 matching lines...) Expand 10 before | Expand all | Expand 10 after
435 base::Callback<void(ui::AXEvent, int)> accessibility_testing_callback_; 424 base::Callback<void(ui::AXEvent, int)> accessibility_testing_callback_;
436 // The most recently received accessibility tree - for testing only. 425 // The most recently received accessibility tree - for testing only.
437 scoped_ptr<ui::AXTree> ax_tree_for_testing_; 426 scoped_ptr<ui::AXTree> ax_tree_for_testing_;
438 427
439 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl); 428 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl);
440 }; 429 };
441 430
442 } // namespace content 431 } // namespace content
443 432
444 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_ 433 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698