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

Issue 2976004: Http Cache: chain GetLoadState from transactions waiting for... (Closed)

Created:
10 years, 5 months ago by rvargas (doing something else)
Modified:
9 years, 6 months ago
Reviewers:
eroman
CC:
chromium-reviews, cbentzel+watch_chromium.org, darin-cc_chromium.org, Paweł Hajdan Jr.
Visibility:
Public.

Description

Http Cache: chain GetLoadState from transactions waiting for access to an entry to the transaction that has exclusive access to that entry. This means that requests that are blocked on the single writer lock of the http cache will report the load state of the request that has the lock, and WAITING_FOR_CACHE is reserved for requests that are actually waiting for the entry to be created by the cache. BUG=43582 TEST=net_unittests Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=52334

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+55 lines, -6 lines) Patch
M net/http/http_cache.h View 1 2 chunks +4 lines, -0 lines 0 comments Download
M net/http/http_cache.cc View 1 chunk +13 lines, -0 lines 0 comments Download
M net/http/http_cache_transaction.h View 1 chunk +6 lines, -0 lines 0 comments Download
M net/http/http_cache_transaction.cc View 2 chunks +16 lines, -5 lines 0 comments Download
M net/http/http_cache_unittest.cc View 2 chunks +14 lines, -0 lines 0 comments Download
M net/http/http_transaction_unittest.h View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
rvargas (doing something else)
10 years, 5 months ago (2010-07-13 01:42:21 UTC) #1
eroman
10 years, 5 months ago (2010-07-13 22:16:26 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698