Use of base::StringPairs appropriately in disk_cache

Use of base::StringPairs appropriately in disk_cache

Bescause base/strings/string_split.h defines:
typedef std::vector<std::pair<std::string, std::string> > StringPairs;

BUG=412250
Committed: https://crrev.com/b256c74a4dc38cd5bf579bb87909b19a1cee9374
Cr-Commit-Position: refs/heads/master@{#327675}

[payal.pandey, 2015-04-29 10:37:19]

payal.pandey@samsung.com changed reviewers:
+ eroman@chromium.org

[payal.pandey, 2015-04-29 10:37:19]

Please have a look, Thanks.

[eroman, 2015-04-29 19:09:06]

Please also update the overrides of this method. Notably:

https://code.google.com/p/chromium/codesearch#chromium/src/net/disk_cache/dis...

https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/ca...

https://code.google.com/p/chromium/codesearch#chromium/src/net/disk_cache/mem...

https://code.google.com/p/chromium/codesearch#chromium/src/net/disk_cache/sim...

https://code.google.com/p/chromium/codesearch#chromium/src/net/disk_cache/sim...

https://code.google.com/p/chromium/codesearch#chromium/src/net/http/mock_http...

https://code.google.com/p/chromium/codesearch#chromium/src/net/http/mock_http...

[payal.pandey, 2015-04-29 19:24:09]

On 2015/04/29 19:09:06, eroman wrote:
> Please also update the overrides of this method. Notably:
> 
>
https://code.google.com/p/chromium/codesearch#chromium/src/net/disk_cache/dis...
> 
>
https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/ca...
> 
>
https://code.google.com/p/chromium/codesearch#chromium/src/net/disk_cache/mem...
> 
>
https://code.google.com/p/chromium/codesearch#chromium/src/net/disk_cache/sim...
> 
>
https://code.google.com/p/chromium/codesearch#chromium/src/net/disk_cache/sim...
> 
>
https://code.google.com/p/chromium/codesearch#chromium/src/net/http/mock_http...
> 
>
https://code.google.com/p/chromium/codesearch#chromium/src/net/http/mock_http...

Already raised review for following changes in other CL
CL 1111203002: simple_backend_impl.cc,simple_backend_impl.h 
CL 1112083002: mock_http_cache.cc, mock_http_cache.h

And will update for cache_storage_cache_unittest.cc, mem_backend_impl.h.
@eroman : Please review for raised CL, Thanks

[eroman, 2015-04-29 19:31:23]

lgtm

[payal.pandey, 2015-04-30 06:26:09]

The CQ bit was checked by payal.pandey@samsung.com

[commit-bot: I haz the power, 2015-04-30 06:26:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1114883002/1

[commit-bot: I haz the power, 2015-04-30 07:36:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-04-30 07:37:03]

Patchset 1 (id:??) landed as
https://crrev.com/b256c74a4dc38cd5bf579bb87909b19a1cee9374
Cr-Commit-Position: refs/heads/master@{#327675}