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

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

Issue 2309083002: PlzNavigate: Fix history navigation for error pages. (Closed)
Patch Set: . Created 4 years, 3 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
« no previous file with comments | « no previous file | 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 <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 108 matching lines...) Expand 10 before | Expand all | Expand 10 after
119 class Origin; 119 class Origin;
120 } 120 }
121 121
122 namespace content { 122 namespace content {
123 123
124 class ChildFrameCompositingHelper; 124 class ChildFrameCompositingHelper;
125 class CompositorDependencies; 125 class CompositorDependencies;
126 class DevToolsAgent; 126 class DevToolsAgent;
127 class DocumentState; 127 class DocumentState;
128 class ExternalPopupMenu; 128 class ExternalPopupMenu;
129 class HistoryEntry;
129 class ManifestManager; 130 class ManifestManager;
130 class MediaInterfaceProvider; 131 class MediaInterfaceProvider;
131 class MediaStreamDispatcher; 132 class MediaStreamDispatcher;
132 class MediaStreamRendererFactory; 133 class MediaStreamRendererFactory;
133 class MediaPermissionDispatcher; 134 class MediaPermissionDispatcher;
134 class MidiDispatcher; 135 class MidiDispatcher;
135 class NavigationState; 136 class NavigationState;
136 class NotificationPermissionDispatcher; 137 class NotificationPermissionDispatcher;
137 class PageState; 138 class PageState;
138 class PepperPluginInstanceImpl; 139 class PepperPluginInstanceImpl;
(...skipping 785 matching lines...) Expand 10 before | Expand all | Expand 10 after
924 // Adds the given file chooser request to the file_chooser_completion_ queue 925 // Adds the given file chooser request to the file_chooser_completion_ queue
925 // (see that var for more) and requests the chooser be displayed if there are 926 // (see that var for more) and requests the chooser be displayed if there are
926 // no other waiting items in the queue. 927 // no other waiting items in the queue.
927 // 928 //
928 // Returns true if the chooser was successfully scheduled. False means we 929 // Returns true if the chooser was successfully scheduled. False means we
929 // didn't schedule anything. 930 // didn't schedule anything.
930 bool ScheduleFileChooser(const FileChooserParams& params, 931 bool ScheduleFileChooser(const FileChooserParams& params,
931 blink::WebFileChooserCompletion* completion); 932 blink::WebFileChooserCompletion* completion);
932 933
933 // Loads the appropriate error page for the specified failure into the frame. 934 // Loads the appropriate error page for the specified failure into the frame.
935 // |entry| is only used by PlzNavigate when navigating to a history item.
934 void LoadNavigationErrorPage(const blink::WebURLRequest& failed_request, 936 void LoadNavigationErrorPage(const blink::WebURLRequest& failed_request,
935 const blink::WebURLError& error, 937 const blink::WebURLError& error,
936 bool replace); 938 bool replace,
939 HistoryEntry* entry);
937 940
938 void HandleJavascriptExecutionResult(const base::string16& javascript, 941 void HandleJavascriptExecutionResult(const base::string16& javascript,
939 int id, 942 int id,
940 bool notify_result, 943 bool notify_result,
941 v8::Local<v8::Value> result); 944 v8::Local<v8::Value> result);
942 945
943 // Initializes |web_user_media_client_|. If this fails, because it wasn't 946 // Initializes |web_user_media_client_|. If this fails, because it wasn't
944 // possible to create a MediaStreamClient (e.g., WebRTC is disabled), then 947 // possible to create a MediaStreamClient (e.g., WebRTC is disabled), then
945 // |web_user_media_client_| will remain NULL. 948 // |web_user_media_client_| will remain NULL.
946 void InitializeUserMediaClient(); 949 void InitializeUserMediaClient();
(...skipping 334 matching lines...) Expand 10 before | Expand all | Expand 10 after
1281 mojom::FrameHostPtr frame_host_; 1284 mojom::FrameHostPtr frame_host_;
1282 1285
1283 base::WeakPtrFactory<RenderFrameImpl> weak_factory_; 1286 base::WeakPtrFactory<RenderFrameImpl> weak_factory_;
1284 1287
1285 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl); 1288 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl);
1286 }; 1289 };
1287 1290
1288 } // namespace content 1291 } // namespace content
1289 1292
1290 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 1293 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
OLDNEW
« no previous file with comments | « no previous file | content/renderer/render_frame_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698