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

Unified Diff: Source/core/xml/XMLHttpRequest.h

Issue 490083002: [XHR] Move the code to clear variables to internalAbort() (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Reverted removal of cancelLoader() in handleDidTimeout Created 6 years, 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | Source/core/xml/XMLHttpRequest.cpp » ('j') | Source/core/xml/XMLHttpRequest.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/xml/XMLHttpRequest.h
diff --git a/Source/core/xml/XMLHttpRequest.h b/Source/core/xml/XMLHttpRequest.h
index 8124e332bbb1172e7ff50cadcb3c2a13002fa641..6ca199b99ac3ed764b5eb5ceddc92f02dd940993 100644
--- a/Source/core/xml/XMLHttpRequest.h
+++ b/Source/core/xml/XMLHttpRequest.h
@@ -201,8 +201,10 @@ private:
// Clears variables used only while the resource is being loaded.
void clearVariablesForLoading();
- // Returns false iff reentry happened and a new load is started.
- bool internalAbort();
+ void internalAbort();
+ // Detaches m_loader and calls cancel() on it. Returns false iff reentry
+ // happened and a new load is started.
+ bool cancelLoader();
// Clears variables holding response header and body data.
void clearResponse();
void clearRequest();
« no previous file with comments | « no previous file | Source/core/xml/XMLHttpRequest.cpp » ('j') | Source/core/xml/XMLHttpRequest.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698