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

Side by Side Diff: third_party/WebKit/Source/web/WebLocalFrameImpl.cpp

Issue 1475863005: [Async][WIP] Call FrameLoader::checkCompleted() asynchronously to avoid sync body.onload() Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 4 years, 10 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) 2009 Google Inc. All rights reserved. 2 * Copyright (C) 2009 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 946 matching lines...) Expand 10 before | Expand all | Expand 10 after
957 957
958 void WebLocalFrameImpl::stopLoading() 958 void WebLocalFrameImpl::stopLoading()
959 { 959 {
960 if (!frame()) 960 if (!frame())
961 return; 961 return;
962 // FIXME: Figure out what we should really do here. It seems like a bug 962 // FIXME: Figure out what we should really do here. It seems like a bug
963 // that FrameLoader::stopLoading doesn't call stopAllLoaders. 963 // that FrameLoader::stopLoading doesn't call stopAllLoaders.
964 frame()->loader().stopAllLoaders(); 964 frame()->loader().stopAllLoaders();
965 } 965 }
966 966
967 void WebLocalFrameImpl::stopLoadingCheckCompletedNow()
968 {
969 if (!frame())
970 return;
971 frame()->loader().checkCompletedNow(-1);
972 }
973
967 WebDataSource* WebLocalFrameImpl::provisionalDataSource() const 974 WebDataSource* WebLocalFrameImpl::provisionalDataSource() const
968 { 975 {
969 ASSERT(frame()); 976 ASSERT(frame());
970 return DataSourceForDocLoader(frame()->loader().provisionalDocumentLoader()) ; 977 return DataSourceForDocLoader(frame()->loader().provisionalDocumentLoader()) ;
971 } 978 }
972 979
973 WebDataSource* WebLocalFrameImpl::dataSource() const 980 WebDataSource* WebLocalFrameImpl::dataSource() const
974 { 981 {
975 ASSERT(frame()); 982 ASSERT(frame());
976 return DataSourceForDocLoader(frame()->loader().documentLoader()); 983 return DataSourceForDocLoader(frame()->loader().documentLoader());
(...skipping 1207 matching lines...) Expand 10 before | Expand all | Expand 10 after
2184 return WebSandboxFlags::None; 2191 return WebSandboxFlags::None;
2185 return static_cast<WebSandboxFlags>(frame()->loader().effectiveSandboxFlags( )); 2192 return static_cast<WebSandboxFlags>(frame()->loader().effectiveSandboxFlags( ));
2186 } 2193 }
2187 2194
2188 void WebLocalFrameImpl::forceSandboxFlags(WebSandboxFlags flags) 2195 void WebLocalFrameImpl::forceSandboxFlags(WebSandboxFlags flags)
2189 { 2196 {
2190 frame()->loader().forceSandboxFlags(static_cast<SandboxFlags>(flags)); 2197 frame()->loader().forceSandboxFlags(static_cast<SandboxFlags>(flags));
2191 } 2198 }
2192 2199
2193 } // namespace blink 2200 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/web/WebLocalFrameImpl.h ('k') | third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698