Remove partial blockfile v3 disk_cache implementation.

Remove partial blockfile v3 disk_cache implementation.

This isn't being worked on; so it's been sad seeing this code updated
with various refactorings and such.  Also, it's a bit scary how we
were just relying on linker dead code elimination to protect our
binary size.

R=davidben@chromium.org
BUG=241277
Committed: https://crrev.com/1b45006a139d227d5570e2bd77943ebfc0bf96b3
Cr-Commit-Position: refs/heads/master@{#388976}

[gavinp, 2016-04-21 18:16:23]

mmenke: WDYT?

[mmenke, 2016-04-21 18:31:25]

On 2016/04/21 18:16:23, gavinp wrote:
> mmenke: WDYT?

Could you find someone else to review this?  Took two days off, and came back to
14 codereviews waiting.  I'm happy to do it, just not sure when I'll get to it.

[gavinp, 2016-04-21 18:32:52]

Description was changed from

==========
Remove partial blockfile v3 disk_cache implementation.

This isn't being worked on; so it's been sad seeing this code updated
with various refactorings and such.  Also, it's a bit scary how we
were just relying on linker dead code elimination to protect our
binary size.

R=mmenke@chromium.org
BUG=None
==========

to

==========
Remove partial blockfile v3 disk_cache implementation.

This isn't being worked on; so it's been sad seeing this code updated
with various refactorings and such.  Also, it's a bit scary how we
were just relying on linker dead code elimination to protect our
binary size.

R=mmenke@chromium.org
BUG=None
==========

[gavinp, 2016-04-21 18:32:52]

gavinp@chromium.org changed reviewers:
+ davidben@chromium.org

[gavinp, 2016-04-21 18:33:11]

mmenke: your wish is my command. Now FYI.

davidben: PTAL.

[davidben, 2016-04-21 18:38:08]

Yay deleting code. lgtm.

Should we close https://crbug.com/241277?

[mmenke, 2016-04-21 18:56:11]

mmenke@chromium.org changed reviewers:
- mmenke@chromium.org

[gavinp, 2016-04-21 23:25:38]

The CQ bit was checked by gavinp@chromium.org

[commit-bot: I haz the power, 2016-04-21 23:26:08]

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

[gavinp, 2016-04-21 23:27:16]

On 2016/04/21 18:38:08, davidben wrote:
> Yay deleting code. lgtm.
> 
> Should we close https://crbug.com/241277

Yes.

[gavinp, 2016-04-21 23:50:18]

On 2016/04/21 23:27:16, gavinp wrote:
> On 2016/04/21 18:38:08, davidben wrote:
> > Yay deleting code. lgtm.
> > 
> > Should we close https://crbug.com/241277
> 
> Yes.

(And done, I went and did that.)

[gavinp, 2016-04-21 23:50:26]

The CQ bit was unchecked by gavinp@chromium.org

[gavinp, 2016-04-21 23:50:45]

Description was changed from

==========
Remove partial blockfile v3 disk_cache implementation.

This isn't being worked on; so it's been sad seeing this code updated
with various refactorings and such.  Also, it's a bit scary how we
were just relying on linker dead code elimination to protect our
binary size.

R=mmenke@chromium.org
BUG=None
==========

to

==========
Remove partial blockfile v3 disk_cache implementation.

This isn't being worked on; so it's been sad seeing this code updated
with various refactorings and such.  Also, it's a bit scary how we
were just relying on linker dead code elimination to protect our
binary size.

R=davidben@chromium.org
BUG=241277
==========

[gavinp, 2016-04-21 23:51:00]

The CQ bit was checked by gavinp@chromium.org

[commit-bot: I haz the power, 2016-04-21 23:51:29]

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

[commit-bot: I haz the power, 2016-04-22 00:33:35]

Description was changed from

==========
Remove partial blockfile v3 disk_cache implementation.

This isn't being worked on; so it's been sad seeing this code updated
with various refactorings and such.  Also, it's a bit scary how we
were just relying on linker dead code elimination to protect our
binary size.

R=davidben@chromium.org
BUG=241277
==========

to

==========
Remove partial blockfile v3 disk_cache implementation.

This isn't being worked on; so it's been sad seeing this code updated
with various refactorings and such.  Also, it's a bit scary how we
were just relying on linker dead code elimination to protect our
binary size.

R=davidben@chromium.org
BUG=241277
==========

[commit-bot: I haz the power, 2016-04-22 00:33:36]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 19:42:25]

Description was changed from

==========
Remove partial blockfile v3 disk_cache implementation.

This isn't being worked on; so it's been sad seeing this code updated
with various refactorings and such.  Also, it's a bit scary how we
were just relying on linker dead code elimination to protect our
binary size.

R=davidben@chromium.org
BUG=241277
==========

to

==========
Remove partial blockfile v3 disk_cache implementation.

This isn't being worked on; so it's been sad seeing this code updated
with various refactorings and such.  Also, it's a bit scary how we
were just relying on linker dead code elimination to protect our
binary size.

R=davidben@chromium.org
BUG=241277

Committed: https://crrev.com/1b45006a139d227d5570e2bd77943ebfc0bf96b3
Cr-Commit-Position: refs/heads/master@{#388976}
==========

[commit-bot: I haz the power, 2016-04-22 19:42:26]

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