Service Workers: Allow empty string as cache name

Service Workers: Allow empty string as cache name

The spec doesn't restrict the namespace of cache names, and the
implementation uses hashes to generate filenames from cache names,
so just delete the code.

BUG=408359
R=gavinp@chromium.org,jkarlin@chromium.org,michaeln@chromium.org

Committed: https://crrev.com/6309c874f2e36c7765d7b5e0483e241af9065ccd
Cr-Commit-Position: refs/heads/master@{#293422}

[jsbell, 2014-09-03 23:20:33]

PTAL?

[michaeln, 2014-09-03 23:32:00]

> The spec doesn't restrict the namespace of cache names

oh, good catch! i think this is just an artifact of an earlier version of this
code where a directory name was based on the cache name?

[jkarlin, 2014-09-04 00:01:55]

On 2014/09/03 23:32:00, michaeln wrote:
> > The spec doesn't restrict the namespace of cache names
> 
> oh, good catch! i think this is just an artifact of an earlier version of this
> code where a directory name was based on the cache name?

Yes, that's correct.  Thanks for writing this jsbell.  lgtm but I don't have
owners.

[jsbell, 2014-09-04 00:30:12]

Thanks!

michaeln - r? cq?

[michaeln, 2014-09-04 00:59:23]

oh, lgtm2

[michaeln, 2014-09-04 01:00:14]

gah... since when does lgtm2 not count as lgtm?

[jsbell, 2014-09-04 23:07:20]

The CQ bit was checked by jsbell@chromium.org

[commit-bot: I haz the power, 2014-09-04 23:09:44]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jsbell@chromium.org/539673003/20001

[commit-bot: I haz the power, 2014-09-05 03:35:47]

Committed patchset #2 (id:20001) as d31515847940b31c24dc94a2544484945def6507

[commit-bot: I haz the power, 2014-09-10 03:36:18]

Patchset 2 (id:??) landed as
https://crrev.com/6309c874f2e36c7765d7b5e0483e241af9065ccd
Cr-Commit-Position: refs/heads/master@{#293422}