Implement PermissionService::GetNextPermissionChange.

chrome/browser/content_settings/permission_context.h

Index: chrome/browser/content_settings/permission_context.h
diff --git a/chrome/browser/content_settings/permission_context.h b/chrome/browser/content_settings/permission_context.h
new file mode 100644
index 0000000000000000000000000000000000000000..faa08db003ecc38da9b31afad3503f71c6e28bd8
--- /dev/null
+++ b/chrome/browser/content_settings/permission_context.h
@@ -0,0 +1,22 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_CONTENT_SETTINGS_PERMISSION_CONTEXT_H_
+#define CHROME_BROWSER_CONTENT_SETTINGS_PERMISSION_CONTEXT_H_
+
+#include "components/content_settings/core/common/content_settings_types.h"
+
+class PermissionContextBase;
+class Profile;
+
+namespace PermissionContext {

[Bernhard Bauer, 2015/03/11 14:06:18]

Namespaces should be lowercase_with_underscores. Or you could make this a class
with only static methods...

[mlamouri (slow - plz ping), 2015/03/18 16:24:40]

Done.

+
+// Helper method returning the PermissionContextBase object associated with the
+// given ContentSettingsType.
+PermissionContextBase* Get(Profile* profile,

[mlamouri (slow - plz ping), 2015/03/11 11:38:17]

This could be renamed PermissionContextFactory::Create() to match what it is
actually abstracting.
Also, I would gladly move that one directory up if it's a problem to have
chrome/browser/content_settings/ to be aware of the different
PermissionContextBase factories.

[Bernhard Bauer, 2015/03/11 14:06:18]

I prefer Get() over Create(), as the method doesn't unconditionally create a
permission context. And neither PermissionContext nor PermissionContextFactory
are actually classses, so both would be in the wrong naming style.
Meh, as long as there is no conceptual layering violation, I'm fine to keep both
in the same directory until content settings and permissions get separated out
more explicitly.

[mlamouri (slow - plz ping), 2015/03/18 16:24:40]

Sounds good.

+                           ContentSettingsType content_settings_type);
+
+}  // namespace PermissionContext
+
+#endif // CHROME_BROWSER_CONTENT_SETTINGS_PERMISSION_CONTEXT_H_