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

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: more saving 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
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 635 matching lines...) Expand 10 before | Expand all | Expand 10 after
646 646
647 blink::WebMediaPlayer* CreateWebMediaPlayerForMediaStream( 647 blink::WebMediaPlayer* CreateWebMediaPlayerForMediaStream(
648 const blink::WebURL& url, 648 const blink::WebURL& url,
649 blink::WebMediaPlayerClient* client); 649 blink::WebMediaPlayerClient* client);
650 650
651 // Creates a factory object used for creating audio and video renderers. 651 // Creates a factory object used for creating audio and video renderers.
652 // The method is virtual so that layouttests can override it. 652 // The method is virtual so that layouttests can override it.
653 virtual scoped_ptr<MediaStreamRendererFactory> CreateRendererFactory(); 653 virtual scoped_ptr<MediaStreamRendererFactory> CreateRendererFactory();
654 654
655 // Checks that the RenderView is ready to display the navigation to |url|. If 655 // Checks that the RenderView is ready to display the navigation to |url|. If
656 // the return value is false, the navigation should be abandonned. 656 // the return value is false, the navigation should be abandoned.
657 bool PrepareRenderViewForNavigation( 657 bool PrepareRenderViewForNavigation(
658 const GURL& url, 658 const GURL& url,
659 FrameMsg_Navigate_Type::Value navigate_type, 659 FrameMsg_Navigate_Type::Value navigate_type,
660 const PageState& state, 660 const PageState& state,
661 bool check_history,
662 int pending_history_list_offset,
663 int32 page_id, 661 int32 page_id,
664 bool* is_reload, 662 bool* is_reload,
665 blink::WebURLRequest::CachePolicy* cache_policy); 663 blink::WebURLRequest::CachePolicy* cache_policy);
666 664
667 // PlzNavigate 665 // PlzNavigate
668 // Sends a FrameHostMsg_BeginNavigation to the browser based on the contents 666 // Sends a FrameHostMsg_BeginNavigation to the browser based on the contents
669 // of the WebURLRequest. 667 // of the WebURLRequest.
670 void BeginNavigation(blink::WebURLRequest* request); 668 void BeginNavigation(blink::WebURLRequest* request);
671 669
672 // Returns the URL being loaded by the |frame_|'s request. 670 // Returns the URL being loaded by the |frame_|'s request.
(...skipping 138 matching lines...) Expand 10 before | Expand all | Expand 10 after
811 #endif 809 #endif
812 810
813 base::WeakPtrFactory<RenderFrameImpl> weak_factory_; 811 base::WeakPtrFactory<RenderFrameImpl> weak_factory_;
814 812
815 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl); 813 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl);
816 }; 814 };
817 815
818 } // namespace content 816 } // namespace content
819 817
820 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 818 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698