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

Side by Side Diff: content/renderer/render_frame_impl.h

Issue 743803002: Avoid stale navigation requests without excessive page id knowledge in the renderer process. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: update Created 6 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/public/test/render_view_test.cc ('k') | content/renderer/render_frame_impl.cc » ('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 #ifndef CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 5 #ifndef CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
6 #define CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 6 #define CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 644 matching lines...) Expand 10 before | Expand all | Expand 10 after
655 // Creates a factory object used for creating audio and video renderers. 655 // Creates a factory object used for creating audio and video renderers.
656 // The method is virtual so that layouttests can override it. 656 // The method is virtual so that layouttests can override it.
657 virtual scoped_ptr<MediaStreamRendererFactory> CreateRendererFactory(); 657 virtual scoped_ptr<MediaStreamRendererFactory> CreateRendererFactory();
658 658
659 // Checks that the RenderView is ready to display the navigation to |url|. If 659 // Checks that the RenderView is ready to display the navigation to |url|. If
660 // the return value is false, the navigation should be abandoned. 660 // the return value is false, the navigation should be abandoned.
661 bool PrepareRenderViewForNavigation( 661 bool PrepareRenderViewForNavigation(
662 const GURL& url, 662 const GURL& url,
663 FrameMsg_Navigate_Type::Value navigate_type, 663 FrameMsg_Navigate_Type::Value navigate_type,
664 const PageState& state, 664 const PageState& state,
665 bool check_history, 665 bool check_for_stale_navigation,
666 int pending_history_list_offset, 666 bool is_history_navigation,
667 int current_history_list_offset,
667 int32 page_id, 668 int32 page_id,
668 bool* is_reload, 669 bool* is_reload,
669 blink::WebURLRequest::CachePolicy* cache_policy); 670 blink::WebURLRequest::CachePolicy* cache_policy);
670 671
671 // PlzNavigate 672 // PlzNavigate
672 // Sends a FrameHostMsg_BeginNavigation to the browser based on the contents 673 // Sends a FrameHostMsg_BeginNavigation to the browser based on the contents
673 // of the WebURLRequest. 674 // of the WebURLRequest.
674 void BeginNavigation(blink::WebURLRequest* request); 675 void BeginNavigation(blink::WebURLRequest* request);
675 676
676 // Returns the URL being loaded by the |frame_|'s request. 677 // Returns the URL being loaded by the |frame_|'s request.
(...skipping 138 matching lines...) Expand 10 before | Expand all | Expand 10 after
815 #endif 816 #endif
816 817
817 base::WeakPtrFactory<RenderFrameImpl> weak_factory_; 818 base::WeakPtrFactory<RenderFrameImpl> weak_factory_;
818 819
819 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl); 820 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl);
820 }; 821 };
821 822
822 } // namespace content 823 } // namespace content
823 824
824 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 825 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
OLDNEW
« no previous file with comments | « content/public/test/render_view_test.cc ('k') | content/renderer/render_frame_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698