Define Print Preview default printer selection policy

chrome/common/chrome_switches.cc

 // 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.
 
 #include "chrome/common/chrome_switches.h"
 
 #include "base/base_switches.h"
 #include "base/command_line.h"
 
 namespace switches {
@@ skipping 295 matching lines @@
 
 // Disable pop-up blocking.
 const char kDisablePopupBlocking[]          = "disable-popup-blocking";
 
 // Disable speculative TCP/IP preconnection.
 const char kDisablePreconnect[]             = "disable-preconnect";
 
 // Disables print preview (For testing, and for users who don't like us. :[ )
 const char kDisablePrintPreview[]           = "disable-print-preview";
 
+// Default print destination selection rules.
+// See PrintPreviewDefaultPrinterSelection policy for details.
+const char kPrintPreviewDefaultDestinationSelectionRules[] =
+    "default-printer-selection-rules";

[Thiemo Nagel, 2015/10/09 16:35:47]

Adding command line switches is not allowed unless there's a good reason for it.

[Aleksey Shlyapnikov, 2015/10/09 21:02:51]

My rationale was to offer non-enterprise users unhappy with default printer
policy a way to address it.
Actually, was it a request to remove this command line switch or to explain it?

[Thiemo Nagel, 2015/10/12 19:10:09]

I guess the request was to remove the flag unless there's a very good reason to
have it, for a discussion on removing command line switches, cf.:

https://groups.google.com/a/chromium.org/d/topic/chromium-dev/8EjjXUoFqMI/dis...

I believe that allowing users better control is not a good justification for
adding a command line flag.  If a setting is not important enough to make it
into chrome://settings, it shouldn't exist at all.  At least that's my
understanding of Chrome philosophy.

[Aleksey Shlyapnikov, 2015/10/13 18:14:57]

Done.

+
 // Normally when the user attempts to navigate to a page that was the result of
 // a post we prompt to make sure they want to. This switch may be used to
 // disable that check. This switch is used during automated testing.
 const char kDisablePromptOnRepost[]         = "disable-prompt-on-repost";
 
 // Disables support for the QUIC protocol.
 const char kDisableQuic[]                   = "disable-quic";
 
 // Disable use of Chromium's port selection for the ephemeral port via bind().
 // This only has an effect if QUIC protocol is enabled.
@@ skipping 1062 matching lines @@
 
 // -----------------------------------------------------------------------------
 // DO NOT ADD YOUR CRAP TO THE BOTTOM OF THIS FILE.
 //
 // You were going to just dump your switches here, weren't you? Instead, please
 // put them in alphabetical order above, or in order inside the appropriate
 // ifdef at the bottom. The order should match the header.
 // -----------------------------------------------------------------------------
 
 }  // namespace switches