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

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

Issue 1910153002: Reland of DocumentLoader should commit the load before creating a DocumentWriter (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 Apple Inc. All rights reserved. 2 * Copyright (C) 2006, 2007, 2008, 2009 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 131 matching lines...) Expand 10 before | Expand all | Expand 10 after
142 Resource* startPreload(Resource::Type, FetchRequest&); 142 Resource* startPreload(Resource::Type, FetchRequest&);
143 143
144 DECLARE_VIRTUAL_TRACE(); 144 DECLARE_VIRTUAL_TRACE();
145 145
146 protected: 146 protected:
147 DocumentLoader(LocalFrame*, const ResourceRequest&, const SubstituteData&); 147 DocumentLoader(LocalFrame*, const ResourceRequest&, const SubstituteData&);
148 148
149 Vector<KURL> m_redirectChain; 149 Vector<KURL> m_redirectChain;
150 150
151 private: 151 private:
152 static DocumentWriter* createWriterFor(const DocumentInit&, const AtomicStri ng& mimeType, const AtomicString& encoding, bool dispatch, ParserSynchronization Policy); 152 static DocumentWriter* createWriterFor(const DocumentInit&, const AtomicStri ng& mimeType, const AtomicString& encoding, bool dispatchWindowObjectAvailable, ParserSynchronizationPolicy, const KURL& overridingURL = KURL());
153 153
154 void ensureWriter(const AtomicString& mimeType, const KURL& overridingURL = KURL()); 154 void ensureWriter(const AtomicString& mimeType, const KURL& overridingURL = KURL());
155 void endWriting(DocumentWriter*); 155 void endWriting(DocumentWriter*);
156 156
157 FrameLoader* frameLoader() const; 157 FrameLoader* frameLoader() const;
158 158
159 void commitIfReady(); 159 void commitIfReady();
160 void commitData(const char* bytes, size_t length); 160 void commitData(const char* bytes, size_t length);
161 ResourceLoader* mainResourceLoader() const; 161 ResourceLoader* mainResourceLoader() const;
162 void clearMainResourceHandle(); 162 void clearMainResourceHandle();
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
229 // Used to protect against reentrancy into dataReceived(). 229 // Used to protect against reentrancy into dataReceived().
230 bool m_inDataReceived; 230 bool m_inDataReceived;
231 RefPtr<SharedBuffer> m_dataBuffer; 231 RefPtr<SharedBuffer> m_dataBuffer;
232 }; 232 };
233 233
234 DECLARE_WEAK_IDENTIFIER_MAP(DocumentLoader); 234 DECLARE_WEAK_IDENTIFIER_MAP(DocumentLoader);
235 235
236 } // namespace blink 236 } // namespace blink
237 237
238 #endif // DocumentLoader_h 238 #endif // DocumentLoader_h
OLDNEW
« no previous file with comments | « chrome/renderer/content_settings_observer_browsertest.cc ('k') | third_party/WebKit/Source/core/loader/DocumentLoader.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698