Implement FakeAuthPolicyClient::RefreshDevicePolicy()

chromeos/BUILD.gn

 # Copyright 2014 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.
 
 import("//build/config/ui.gni")
 import("//testing/test.gni")
 import("//third_party/protobuf/proto_library.gni")
 
 assert(is_chromeos, "Non-ChromeOS builds must not depend on //chromeos")
 
@@ skipping 12 matching lines @@
   public_deps = [
     "//dbus",
   ]
   deps = [
     ":attestation_proto",
     ":cryptohome_proto",
     ":power_manager_proto",
     "//base",
     "//base:i18n",
     "//base/third_party/dynamic_annotations",
+    "//chrome/browser/chromeos:device_policy_proto",

[brettw, 2017/08/24 23:54:46]

Hi, this is a layering violation as //chromeos is as a lower layer than
//chrome.

I filed https://bugs.chromium.org/p/chromium/issues/detail?id=758767 to track
this. I would like to get this fixed soon before it becomes even harder to fix.

Thanks!

     "//components/device_event_log",
     "//components/onc",
     "//components/policy/proto",
     "//components/pref_registry",
     "//components/prefs",
     "//components/proxy_config",
     "//components/signin/core/account_id",
     "//components/user_manager",
     "//crypto",
     "//crypto:platform",
@@ skipping 172 matching lines @@
   proto_out_dir = "chromeos/attestation"
 }
 
 proto_library("cryptohome_signkey_proto") {
   sources = [
     "//third_party/cros_system_api/dbus/cryptohome/signed_secret.proto",
   ]
 
   proto_out_dir = "chromeos/cryptohome"
 }