Commit b8863f18 authored by Owen Min's avatar Owen Min Committed by Commit Bot

Put policy_browsertests.cc into maintenance mode

policy_browsertest.cc contains too many policy tests today. Write a file
comment to ask developers to stop adding more into it. Also give a simple
hint about how to test policy outside policy directory.

Bug: 1002483
Change-Id: I06c660c9b8df00a719e39280f112d029622d5a9d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1872806Reviewed-by: default avatarJulian Pastarmov <pastarmovj@chromium.org>
Commit-Queue: Owen Min <zmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#710797}
parent a2969a55
// Copyright (c) 2012 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.
//
// This file is in maintenance mode, please do NOT add new tests into this file.
//
// policy_browsertests.cc contains lots of tests for multiple policies. However,
// it became huge with hundreds of policies. Instead of adding even more tests
// here, please put new ones with the policy implementation. For example, a
// network policy test can be moved to chrome/browser/net.
//
// Policy component dependency is not necessary for policy test. Most of
// policy values are copied into local state or Profile prefs. They can be used
// to enable policy during test.
//
// Simple policy to prefs mapping can be tested with policy_test_cases.json. If
// the conversion is complicated and requires custom policy handler, we
// recommend to test the handler separately.
#include <stddef.h>
#include <stdint.h>
......
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