Add an infobar if a session is being controlled by an automated test.

chrome/browser/ui/extensions/extension_message_bubble_factory.cc

 // Copyright 2015 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/browser/ui/extensions/extension_message_bubble_factory.h"
 
 #include "base/base_switches.h"
 #include "base/command_line.h"
 #include "base/lazy_instance.h"
 #include "base/metrics/field_trial.h"
 #include "build/build_config.h"
 #include "chrome/browser/extensions/dev_mode_bubble_delegate.h"
 #include "chrome/browser/extensions/extension_message_bubble_controller.h"
 #include "chrome/browser/extensions/install_verifier.h"
 #include "chrome/browser/extensions/proxy_overridden_bubble_delegate.h"
 #include "chrome/browser/extensions/settings_api_bubble_delegate.h"
 #include "chrome/browser/extensions/settings_api_helpers.h"
 #include "chrome/browser/extensions/suspicious_extension_bubble_delegate.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/startup/startup_browser_creator.h"
 #include "chrome/common/channel_info.h"
 #include "components/version_info/version_info.h"
+#include "content/public/common/content_switches.h"
 #include "extensions/common/feature_switch.h"
 
 namespace {
 
 // A map of all profiles evaluated, so we can tell if it's the initial check.
 // TODO(devlin): It would be nice to coalesce all the "profiles evaluated" maps
 // that are in the different bubble controllers.
 base::LazyInstance<std::set<Profile*> > g_profiles_evaluated =
     LAZY_INSTANCE_INITIALIZER;
 
@@ skipping 43 matching lines @@
   return g_override_for_testing ==
              ExtensionMessageBubbleFactory::OVERRIDE_ENABLED ||
          IsExperimentEnabled(kEnableProxyWarningExperimentName);
 #endif
 }
 
 bool EnableDevModeBubble() {
   if (extensions::FeatureSwitch::force_dev_mode_highlighting()->IsEnabled())
     return true;
 
+  base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();

[Devlin, 2017/02/06 16:20:21]

We should add a comment here like:
// If ChromeDriver is controlling the browser, we don't show the
// dev mode bubble because it interferes with focus. This isn't a
// security concern because we'll instead show an (even scarier)
// infobar. See also <infobar class>.

[samuong, 2017/02/07 19:11:09]

Done.

+  if (command_line->HasSwitch(switches::kEnableChromeDriver))
+    return false;
+
 #if defined(OS_WIN)
   if (chrome::GetChannel() >= version_info::Channel::BETA)
     return true;
 #endif
 
   return g_override_for_testing ==
              ExtensionMessageBubbleFactory::OVERRIDE_ENABLED ||
          IsExperimentEnabled(kEnableDevModeWarningExperimentName);
 }
 
@@ skipping 72 matching lines @@
 
   controller.reset();
   return controller;
 }
 
 // static
 void ExtensionMessageBubbleFactory::set_override_for_tests(
     OverrideForTesting override) {
   g_override_for_testing = override;
 }