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

Side by Side Diff: third_party/WebKit/Source/core/loader/FrameLoader.h

Issue 2702503002: Block renderer-initiated main frame navigations to data URLs (Closed)
Patch Set: dcheng comments - move checks to FrameLoader Created 3 years, 8 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2006, 2007, 2008, 2009, 2011 Apple Inc. All rights reserved. 2 * Copyright (C) 2006, 2007, 2008, 2009, 2011 Apple Inc. All rights reserved.
3 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. 3 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
4 * (http://www.torchmobile.com/) 4 * (http://www.torchmobile.com/)
5 * Copyright (C) Research In Motion Limited 2009. All rights reserved. 5 * Copyright (C) Research In Motion Limited 2009. All rights reserved.
6 * Copyright (C) 2011 Google Inc. All rights reserved. 6 * Copyright (C) 2011 Google Inc. All rights reserved.
7 * 7 *
8 * Redistribution and use in source and binary forms, with or without 8 * Redistribution and use in source and binary forms, with or without
9 * modification, are permitted provided that the following conditions 9 * modification, are permitted provided that the following conditions
10 * are met: 10 * are met:
(...skipping 80 matching lines...) Expand 10 before | Expand all | Expand 10 after
91 // For history navigations or reloads, an appropriate FrameLoadType should be 91 // For history navigations or reloads, an appropriate FrameLoadType should be
92 // given. Otherwise, FrameLoadTypeStandard should be used (and the final 92 // given. Otherwise, FrameLoadTypeStandard should be used (and the final
93 // FrameLoadType will be computed). For history navigations, a history item 93 // FrameLoadType will be computed). For history navigations, a history item
94 // and a HistoryLoadType should also be provided. 94 // and a HistoryLoadType should also be provided.
95 void Load(const FrameLoadRequest&, 95 void Load(const FrameLoadRequest&,
96 FrameLoadType = kFrameLoadTypeStandard, 96 FrameLoadType = kFrameLoadTypeStandard,
97 HistoryItem* = nullptr, 97 HistoryItem* = nullptr,
98 HistoryLoadType = kHistoryDifferentDocumentLoad); 98 HistoryLoadType = kHistoryDifferentDocumentLoad);
99 99
100 static void ReportLocalLoadFailed(LocalFrame*, const String& url); 100 static void ReportLocalLoadFailed(LocalFrame*, const String& url);
101 static void ReportTopLevelNavigationFailed(LocalFrame*, const String& url);
101 102
102 // Warning: stopAllLoaders can and will detach the LocalFrame out from under 103 // Warning: stopAllLoaders can and will detach the LocalFrame out from under
103 // you. All callers need to either protect the LocalFrame or guarantee they 104 // you. All callers need to either protect the LocalFrame or guarantee they
104 // won't in any way access the LocalFrame after stopAllLoaders returns. 105 // won't in any way access the LocalFrame after stopAllLoaders returns.
105 void StopAllLoaders(); 106 void StopAllLoaders();
106 107
107 // FIXME: clear() is trying to do too many things. We should break it down 108 // FIXME: clear() is trying to do too many things. We should break it down
108 // into smaller functions. 109 // into smaller functions.
109 void Clear(); 110 void Clear();
110 111
(...skipping 204 matching lines...) Expand 10 before | Expand all | Expand 10 after
315 SandboxFlags forced_sandbox_flags_; 316 SandboxFlags forced_sandbox_flags_;
316 317
317 bool dispatching_did_clear_window_object_in_main_world_; 318 bool dispatching_did_clear_window_object_in_main_world_;
318 bool protect_provisional_loader_; 319 bool protect_provisional_loader_;
319 bool detached_; 320 bool detached_;
320 }; 321 };
321 322
322 } // namespace blink 323 } // namespace blink
323 324
324 #endif // FrameLoader_h 325 #endif // FrameLoader_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698