Implementing app command to query EULA acceptance state for Chrome.

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 1082 matching lines @@
 // affects HTTP and HTTPS requests.
 const char kProxyServer[]                   = "proxy-server";
 
 // Adds a "Purge memory" button to the Task Manager, which tries to dump as
 // much memory as possible. This is mostly useful for testing how well the
 // MemoryPurger functionality works.
 //
 // NOTE: This is only implemented for Views.
 const char kPurgeMemoryButton[]             = "purge-memory-button";
 
+// Determines whether or not EULA has been accepted at some point, and returns
+// the result (0 or 1) as exit code.

[grt (UTC plus 2), 2013/01/23 19:02:29]

please be explicit as to what 0 and 1 mean. in general, a process exit code of 0
means success and non-zero means failure, but it looks like the implementation
returns 1 for has-been-accepted and 0 for not-been-accepted (in contrast to this
comment's implied 0=accepted, 1=not accepted).

[huangs, 2013/01/24 00:08:08]

Done; made comment explicit, and returning E_FAIL on error.

+const char kQueryEULAAcceptance[]           = "query-eula-acceptance";
+
 // Capture resource consumption information through page cycling and output the
 // data to the specified file.
 const char kRecordStats[]                   = "record-stats";
 
 // Chrome supports a playback and record mode.  Record mode saves *everything*
 // to the cache.  Playback mode reads data exclusively from the cache.  This
 // allows us to record a session into the cache and then replay it at will.
 // See also kPlaybackMode.
 const char kRecordMode[]                    = "record-mode";
 
@@ skipping 508 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