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

Issue 2921010: Revert 52334 - 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:
rvargas1
CC:
chromium-reviews, cbentzel+watch_chromium.org, darin-cc_chromium.org, Paweł Hajdan Jr.
Visibility:
Public.

Description

Revert 52334 - 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 Review URL: http://codereview.chromium.org/2976004 TBR=rvargas@google.com Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=52405

Patch Set 1 #

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

Messages

Total messages: 1 (0 generated)
rvargas (doing something else)
10 years, 5 months ago (2010-07-14 21:28:47 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698