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

Side by Side Diff: third_party/WebKit/Source/core/loader/DocumentLoader.cpp

Issue 2844913002: Drop GetFrame()->GetDocument() usage in FrameFetchContext (Closed)
Patch Set: Fix last crash Created 3 years, 7 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 Apple Inc. All rights reserved. 2 * Copyright (C) 2006, 2007, 2008 Apple Inc. All rights reserved.
3 * Copyright (C) 2011 Google Inc. All rights reserved. 3 * Copyright (C) 2011 Google Inc. All rights reserved.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions 6 * modification, are permitted provided that the following conditions
7 * are met: 7 * are met:
8 * 8 *
9 * 1. Redistributions of source code must retain the above copyright 9 * 1. Redistributions of source code must retain the above copyright
10 * notice, this list of conditions and the following disclaimer. 10 * notice, this list of conditions and the following disclaimer.
(...skipping 224 matching lines...) Expand 10 before | Expand all | Expand 10 after
235 source_location_ = std::move(source_location); 235 source_location_ = std::move(source_location);
236 } 236 }
237 237
238 std::unique_ptr<SourceLocation> DocumentLoader::CopySourceLocation() const { 238 std::unique_ptr<SourceLocation> DocumentLoader::CopySourceLocation() const {
239 return source_location_ ? source_location_->Clone() : nullptr; 239 return source_location_ ? source_location_->Clone() : nullptr;
240 } 240 }
241 241
242 void DocumentLoader::DispatchLinkHeaderPreloads( 242 void DocumentLoader::DispatchLinkHeaderPreloads(
243 ViewportDescriptionWrapper* viewport, 243 ViewportDescriptionWrapper* viewport,
244 LinkLoader::MediaPreloadPolicy media_policy) { 244 LinkLoader::MediaPreloadPolicy media_policy) {
245 DCHECK_GE(state_, kCommitted);
245 LinkLoader::LoadLinksFromHeader( 246 LinkLoader::LoadLinksFromHeader(
246 GetResponse().HttpHeaderField(HTTPNames::Link), GetResponse().Url(), 247 GetResponse().HttpHeaderField(HTTPNames::Link), GetResponse().Url(),
247 frame_->GetDocument(), NetworkHintsInterfaceImpl(), 248 *frame_, frame_->GetDocument(), NetworkHintsInterfaceImpl(),
248 LinkLoader::kOnlyLoadResources, media_policy, viewport); 249 LinkLoader::kOnlyLoadResources, media_policy, viewport);
249 } 250 }
250 251
251 void DocumentLoader::DidChangePerformanceTiming() { 252 void DocumentLoader::DidChangePerformanceTiming() {
252 if (frame_ && state_ >= kCommitted) { 253 if (frame_ && state_ >= kCommitted) {
253 GetLocalFrameClient().DidChangePerformanceTiming(); 254 GetLocalFrameClient().DidChangePerformanceTiming();
254 } 255 }
255 } 256 }
256 257
257 void DocumentLoader::DidObserveLoadingBehavior( 258 void DocumentLoader::DidObserveLoadingBehavior(
(...skipping 834 matching lines...) Expand 10 before | Expand all | Expand 10 after
1092 InstallNewDocumentReason::kJavascriptURL, 1093 InstallNewDocumentReason::kJavascriptURL,
1093 kForceSynchronousParsing, KURL()); 1094 kForceSynchronousParsing, KURL());
1094 if (!source.IsNull()) 1095 if (!source.IsNull())
1095 writer_->AppendReplacingData(source); 1096 writer_->AppendReplacingData(source);
1096 EndWriting(); 1097 EndWriting();
1097 } 1098 }
1098 1099
1099 DEFINE_WEAK_IDENTIFIER_MAP(DocumentLoader); 1100 DEFINE_WEAK_IDENTIFIER_MAP(DocumentLoader);
1100 1101
1101 } // namespace blink 1102 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698