Commit 957eba0c authored by Raphael Kubo da Costa's avatar Raphael Kubo da Costa Committed by Commit Bot

wake lock: Fix a few style issues after #670009

Address a few nitpicks raised by engedy@ in
https://chromium-review.googlesource.com/c/chromium/src/+/1652939.

Bug: 967669
Change-Id: I71abcb6a81b839b5d842dadc6199009378e30295
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1664074
Auto-Submit: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com>
Reviewed-by: default avatarBalazs Engedy <engedy@chromium.org>
Reviewed-by: default avatarDominick Ng <dominickn@chromium.org>
Commit-Queue: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com>
Cr-Commit-Position: refs/heads/master@{#670050}
parent 8bdec06d
...@@ -7,7 +7,6 @@ ...@@ -7,7 +7,6 @@
#include "base/macros.h" #include "base/macros.h"
#include "chrome/browser/permissions/permission_context_base.h" #include "chrome/browser/permissions/permission_context_base.h"
#include "components/content_settings/core/common/content_settings.h"
#include "components/content_settings/core/common/content_settings_types.h" #include "components/content_settings/core/common/content_settings_types.h"
class WakeLockPermissionContext : public PermissionContextBase { class WakeLockPermissionContext : public PermissionContextBase {
......
...@@ -3,6 +3,7 @@ ...@@ -3,6 +3,7 @@
// found in the LICENSE file. // found in the LICENSE file.
#include "chrome/browser/wake_lock/wake_lock_permission_context.h" #include "chrome/browser/wake_lock/wake_lock_permission_context.h"
#include "chrome/test/base/testing_profile.h" #include "chrome/test/base/testing_profile.h"
#include "components/content_settings/core/common/content_settings.h" #include "components/content_settings/core/common/content_settings.h"
#include "content/public/test/test_browser_thread_bundle.h" #include "content/public/test/test_browser_thread_bundle.h"
...@@ -23,11 +24,11 @@ TEST_F(WakeLockPermissionContextTests, InsecureOriginsAreRejected) { ...@@ -23,11 +24,11 @@ TEST_F(WakeLockPermissionContextTests, InsecureOriginsAreRejected) {
GURL insecure_url("http://www.example.com"); GURL insecure_url("http://www.example.com");
GURL secure_url("https://www.example.com"); GURL secure_url("https://www.example.com");
const ContentSettingsType wake_lock_types[] = { const ContentSettingsType kWakeLockTypes[] = {
CONTENT_SETTINGS_TYPE_WAKE_LOCK_SCREEN, CONTENT_SETTINGS_TYPE_WAKE_LOCK_SCREEN,
CONTENT_SETTINGS_TYPE_WAKE_LOCK_SYSTEM}; CONTENT_SETTINGS_TYPE_WAKE_LOCK_SYSTEM};
for (const auto& content_settings_type : wake_lock_types) { for (const auto& content_settings_type : kWakeLockTypes) {
WakeLockPermissionContext permission_context(profile(), WakeLockPermissionContext permission_context(profile(),
content_settings_type); content_settings_type);
EXPECT_EQ(CONTENT_SETTING_BLOCK, EXPECT_EQ(CONTENT_SETTING_BLOCK,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment