Bug 1003091 - HTTP cache v2: Remove temporary fix in CacheFileContextEvictor::GetContextFile() due to CacheFileUtils::ParseKey() limitation, r=honzab
authorMichal Novotny <michal.novotny@gmail.com>
Mon, 05 May 2014 15:04:09 +0200
changeset 201851 aaf6342baaa3f488b4d32b365150544750e3bcab
parent 201850 fbe9c7cc085db57289ad30e29699aacf4f274813
child 201852 4d2b72314d24436caabd39eef4eeffb6dad7f40c
push id494
push userraliiev@mozilla.com
push dateMon, 25 Aug 2014 18:42:16 +0000
treeherdermozilla-release@a3cc3e46b571 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewershonzab
bugs1003091
milestone32.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1003091 - HTTP cache v2: Remove temporary fix in CacheFileContextEvictor::GetContextFile() due to CacheFileUtils::ParseKey() limitation, r=honzab
netwerk/cache2/CacheFileContextEvictor.cpp
--- a/netwerk/cache2/CacheFileContextEvictor.cpp
+++ b/netwerk/cache2/CacheFileContextEvictor.cpp
@@ -399,22 +399,16 @@ CacheFileContextEvictor::GetContextFile(
   nsresult rv;
 
   nsAutoCString leafName;
   leafName.Assign(NS_LITERAL_CSTRING(kContextEvictionPrefix));
 
   nsAutoCString keyPrefix;
   CacheFileUtils::AppendKeyPrefix(aLoadContextInfo, keyPrefix);
 
-  // TODO: This hack is needed because current CacheFileUtils::ParseKey() can
-  // parse only the whole key and not just the key prefix generated by
-  // CacheFileUtils::CreateKeyPrefix(). This should be removed once bug #968593
-  // is fixed.
-  keyPrefix.Append(":foo");
-
   nsAutoCString data64;
   rv = Base64Encode(keyPrefix, data64);
   if (NS_WARN_IF(NS_FAILED(rv))) {
     return rv;
   }
 
   // Replace '/' with '-' since '/' cannot be part of the filename.
   data64.ReplaceChar('/', '-');