Revert of Switching ExtensionSpecialStoragePolicy::IsStorageDurable to use DurableStorage...

Revert of Switching ExtensionSpecialStoragePolicy::IsStorageDurable to use DurableStoragePermissionContext. (patchset #11 id:200001 of https://codereview.chromium.org/2351803002/ )

Reason for revert:
Caused issue 656495.

Original issue's description:
> Switching IsStorageDurable() to use DurableStoragePermissionContext.
>
> ExtensionSpecialStoragePolicy was using cookies_settings to figure out if the
> given origin was allowed to use durable storage. In order to make this work, the
> cookies_settings had to use a lower level api (HostContentSettingsMap) to avoid
> layer violations.
>
> We found that we could remove the cookies_settings::IsStorageDurable() and,
> instead, just have the ExtensionSpecialStoragePolicy access the
> PermissionManager.
>
> BUG=539538
> R=raymes, jochen, pfeldman
>
> Committed: https://crrev.com/4a19a66d254415f7afb67d6f3c6e6354f280109d
> Cr-Commit-Position: refs/heads/master@{#424165}

TBR=jochen@chromium.org,dgrogan@chromium.org,msramek@chromium.org,pfeldman@chromium.org,raymes@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=539538
Committed: https://crrev.com/c93bb7ff88c19875efd7fdb7d8fe7e6538310e67
Cr-Commit-Position: refs/heads/master@{#427244}

[dougt, 2016-10-18 20:45:25]

The CQ bit was checked by dougt@chromium.org

[dougt, 2016-10-18 20:45:25]

Created Revert of Switching ExtensionSpecialStoragePolicy::IsStorageDurable to
use DurableStoragePermissionContext.

[commit-bot: I haz the power, 2016-10-18 20:46:08]

The CQ bit was unchecked by commit-bot@chromium.org

[dougt, 2016-10-18 20:46:38]

The CQ bit was checked by dougt@chromium.org

[commit-bot: I haz the power, 2016-10-18 20:47:08]

The CQ bit was unchecked by commit-bot@chromium.org

[dougt, 2016-10-25 00:08:02]

The CQ bit was checked by dougt@chromium.org to run a CQ dry run

[dougt, 2016-10-25 00:22:56]

The CQ bit was unchecked by dougt@chromium.org

[dougt, 2016-10-25 00:23:00]

The CQ bit was checked by dougt@chromium.org

[commit-bot: I haz the power, 2016-10-25 00:23:32]

The CQ bit was unchecked by commit-bot@chromium.org

[raymes, 2016-10-25 01:39:11]

lgtm

[raymes, 2016-10-25 01:39:18]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2016-10-25 01:39:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-25 02:29:02]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-25 02:31:02]

Description was changed from

==========
Revert of Switching ExtensionSpecialStoragePolicy::IsStorageDurable to use
DurableStoragePermissionContext. (patchset #11 id:200001 of
https://codereview.chromium.org/2351803002/ )

Reason for revert:
Caused issue 656495.

Original issue's description:
> Switching IsStorageDurable() to use DurableStoragePermissionContext.
> 
> ExtensionSpecialStoragePolicy was using cookies_settings to figure out if the
> given origin was allowed to use durable storage. In order to make this work,
the
> cookies_settings had to use a lower level api (HostContentSettingsMap) to
avoid
> layer violations.
> 
> We found that we could remove the cookies_settings::IsStorageDurable() and,
> instead, just have the ExtensionSpecialStoragePolicy access the
> PermissionManager.
> 
> BUG=539538
> R=raymes, jochen, pfeldman
> 
> Committed: https://crrev.com/4a19a66d254415f7afb67d6f3c6e6354f280109d
> Cr-Commit-Position: refs/heads/master@{#424165}

TBR=jochen@chromium.org,dgrogan@chromium.org,msramek@chromium.org,pfeldman@ch...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=539538
==========

to

==========
Revert of Switching ExtensionSpecialStoragePolicy::IsStorageDurable to use
DurableStoragePermissionContext. (patchset #11 id:200001 of
https://codereview.chromium.org/2351803002/ )

Reason for revert:
Caused issue 656495.

Original issue's description:
> Switching IsStorageDurable() to use DurableStoragePermissionContext.
>
> ExtensionSpecialStoragePolicy was using cookies_settings to figure out if the
> given origin was allowed to use durable storage. In order to make this work,
the
> cookies_settings had to use a lower level api (HostContentSettingsMap) to
avoid
> layer violations.
>
> We found that we could remove the cookies_settings::IsStorageDurable() and,
> instead, just have the ExtensionSpecialStoragePolicy access the
> PermissionManager.
>
> BUG=539538
> R=raymes, jochen, pfeldman
>
> Committed: https://crrev.com/4a19a66d254415f7afb67d6f3c6e6354f280109d
> Cr-Commit-Position: refs/heads/master@{#424165}

TBR=jochen@chromium.org,dgrogan@chromium.org,msramek@chromium.org,pfeldman@ch...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=539538

Committed: https://crrev.com/c93bb7ff88c19875efd7fdb7d8fe7e6538310e67
Cr-Commit-Position: refs/heads/master@{#427244}
==========

[commit-bot: I haz the power, 2016-10-25 02:31:04]

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