fix($cookies): remove the deprecated $cookieStore factory

[mgol]

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Bug fix - a removal of a deprecated factory.

What is the current behavior? (You can also link to an open issue here)

The $cookieStore factory exists.

What is the new behavior (if this is a feature change)?

It doesn't.

Does this PR introduce a breaking change?

Yes.

Please check if the PR fulfills these requirements

• The commit message follows our guidelines
• Fix/Feature: Docs have been added/updated
• Fix/Feature: Tests have been added; existing tests pass

Other information:

BREAKING CHANGE:

The $cookieStore has been removed. Migrate to the $cookie service. For simple
values it's enough to rename $cookieStore to $cookie; for object values you need
to use the putObject & getObject methods.

Before:

$cookieStore.put('name1', 'simple value');
$cookieStore.get('name1'); // 'simple value'
$cookieStore.remove('name1');
$cookieStore.put('name2', {key: 'value'});
$cookieStore.get('name2'); // {key: 'value'}
$cookieStore.remove('name2');

After:

$cookie.put('name1', 'simple value');
$cookie.get('name1'); // 'simple value'
$cookie.remove('name1');
$cookie.putObject('name2', {key: 'value'});
$cookie.getObject('name2'); // {key: 'value'}
$cookie.remove('name2');

[gkalpak]

I would make the following changes to the commit message:

1. fix($cookies) --> fix(ngCookies) (because you are not really changing $cookies)
2. $cookie --> $cookies (everywhere)
3. It might be simpler to drop the differentiation between simple and object values and just recommend $cookies.get/putObject() (since this is what $cookieStore was using under the hood.
   At least, we should mention that $cookies.get() will not correctly retrieve simple values stored with $cookieStore.put() (since the stored values will be JSON-stringified and need to be JSON-parsed).

[gkalpak]

$cookie --> $cookies

[mgol]

@gkalpak PR updated.