This opens the door to have more then a single instance of CacheFileIOManager (a.k.a physical storage location) with all its capabilities easily.

- CacheIndex::gInstance is gone - API changed to be non-static - CacheIndex now member of CacheFileIOManager (refptred) - CacheFileIOManager has now direct and lambda (to prevent cumbersome non-null check all over the code) accessors to its CacheIndex instance - these accessors will update in bug 1032254 to recognize which index to use (gen or pinning) - CacheFileHandle(s) now also carry reference to its CacheFileIOManager (took off the current patch for bug 1032254, made some code much simpler to refactor for this bug and will be needed anyway) - roughly tested https://treeherder.mozilla.org/#/jobs?repo=try&revision=cce974417a48 apply above patch from bug 1082735 to build.

(with fixed CacheIndex::QueryInterface impl) https://treeherder.mozilla.org/#/jobs?repo=try&revision=4173d16dcbf8

Comment on attachment 8646399 [details] [diff] [review] v1 Seems like this work is unfortunately lost and not needed. It's bad, since this is a good change that we may need in the future anyway.

No one needs this right now. We can reopen and take the patches if needed once again.