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

Side by Side Diff: net/base/load_states_list.h

Issue 2519473002: Fixes the cache lock issue. (Closed)
Patch Set: Feedback addressed Created 3 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
« no previous file with comments | « net/BUILD.gn ('k') | net/http/http_cache.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // This is the list of load states and their values. For the enum values, 5 // This is the list of load states and their values. For the enum values,
6 // include the file "net/base/load_states.h". 6 // include the file "net/base/load_states.h".
7 // 7 //
8 // Here we define the values using a macro LOAD_STATE, so it can be 8 // Here we define the values using a macro LOAD_STATE, so it can be
9 // expanded differently in some places (for example, to automatically 9 // expanded differently in some places (for example, to automatically
10 // map a load flag value to its symbolic name). 10 // map a load flag value to its symbolic name).
(...skipping 21 matching lines...) Expand all
32 32
33 // This state indicates that the URLRequest delegate has chosen to block this 33 // This state indicates that the URLRequest delegate has chosen to block this
34 // request before it was sent over the network. When in this state, the 34 // request before it was sent over the network. When in this state, the
35 // delegate should set a load state parameter on the URLRequest describing 35 // delegate should set a load state parameter on the URLRequest describing
36 // the nature of the delay (i.e. "Waiting for <description given by 36 // the nature of the delay (i.e. "Waiting for <description given by
37 // delegate>"). 37 // delegate>").
38 LOAD_STATE(WAITING_FOR_DELEGATE, 4) 38 LOAD_STATE(WAITING_FOR_DELEGATE, 4)
39 39
40 // This state corresponds to a resource load that is blocked waiting for 40 // This state corresponds to a resource load that is blocked waiting for
41 // access to a resource in the cache. If multiple requests are made for the 41 // access to a resource in the cache. If multiple requests are made for the
42 // same resource, the first request will be responsible for writing (or 42 // same resource, they'll typically run in parallel but in some circumstances
43 // updating) the cache entry and the second request will be deferred until 43 // they may have to defer until a previous request has completed.
44 // the first completes. This may be done to optimize for cache reuse.
45 LOAD_STATE(WAITING_FOR_CACHE, 5) 44 LOAD_STATE(WAITING_FOR_CACHE, 5)
46 45
47 // This state corresponds to a resource load that is blocked waiting for 46 // This state corresponds to a resource load that is blocked waiting for
48 // access to a resource in the AppCache. 47 // access to a resource in the AppCache.
49 // Note: This is a layering violation, but being the only one it's not that 48 // Note: This is a layering violation, but being the only one it's not that
50 // bad. TODO(rvargas): Reconsider what to do if we need to add more. 49 // bad. TODO(rvargas): Reconsider what to do if we need to add more.
51 LOAD_STATE(WAITING_FOR_APPCACHE, 6) 50 LOAD_STATE(WAITING_FOR_APPCACHE, 6)
52 51
53 // This state corresponds to a resource being blocked waiting for the 52 // This state corresponds to a resource being blocked waiting for the
54 // PAC script to be downloaded. 53 // PAC script to be downloaded.
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
93 // corresponds to the period after the request is sent and before all of the 92 // corresponds to the period after the request is sent and before all of the
94 // response headers have been received. 93 // response headers have been received.
95 LOAD_STATE(WAITING_FOR_RESPONSE, 15) 94 LOAD_STATE(WAITING_FOR_RESPONSE, 15)
96 95
97 // This state corresponds to a resource load that is blocked waiting for a 96 // This state corresponds to a resource load that is blocked waiting for a
98 // read to complete. In the case of a HTTP transaction, this corresponds to 97 // read to complete. In the case of a HTTP transaction, this corresponds to
99 // the period after the response headers have been received and before all of 98 // the period after the response headers have been received and before all of
100 // the response body has been downloaded. (NOTE: This state only applies for 99 // the response body has been downloaded. (NOTE: This state only applies for
101 // an URLRequest while there is an outstanding Read operation.) 100 // an URLRequest while there is an outstanding Read operation.)
102 LOAD_STATE(READING_RESPONSE, 16) 101 LOAD_STATE(READING_RESPONSE, 16)
OLDNEW
« no previous file with comments | « net/BUILD.gn ('k') | net/http/http_cache.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698