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

chrome/browser/ui/startup/chromedriver_infobar_delegate.h

+// Copyright 2017 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.
+
+#ifndef CHROME_BROWSER_UI_STARTUP_CHROMEDRIVER_INFOBAR_DELEGATE_H_
+#define CHROME_BROWSER_UI_STARTUP_CHROMEDRIVER_INFOBAR_DELEGATE_H_
+
+#include "base/compiler_specific.h"
+#include "base/macros.h"
+#include "base/strings/string16.h"
+#include "components/infobars/core/confirm_infobar_delegate.h"
+#include "ui/gfx/vector_icons_public.h"
+#include "url/gurl.h"
+
+class Browser;
+
+// An infobar to inform users if their browser is controlled by ChromeDriver.
+class ChromeDriverInfoBarDelegate : public ConfirmInfoBarDelegate {
+ public:
+  ~ChromeDriverInfoBarDelegate() override;

[Peter Kasting, 2017/02/13 23:16:52]

Nit: This should be private

[samuong, 2017/02/14 00:19:17]

I get a compile error if I try to make it private:

.../unique_ptr.h:63:9: error: calling a private destructor of class
'AutomationInfoBarDelegate'
...
.../automation_infobar_delegate.cc:16:46: note: in instantiation of member
function 'std::unique_ptr<AutomationInfoBarDelegate,
std::default_delete<AutomationInfoBarDelegate> >::~unique_ptr' requested here
  std::unique_ptr<AutomationInfoBarDelegate> delegate(
                                             ^

[Peter Kasting, 2017/02/14 00:23:17]

Declare that unique_ptr as a unique_ptr<ConfirmInfoBarDelegate> and this should
clear up.

[samuong, 2017/02/14 00:26:54]

Ah right, done.

+  static void CreateAndShow(Browser* browser);

[Peter Kasting, 2017/02/13 23:16:52]

Nit: Name this Create() in keeping with other infobars.

Eliminate the |browser| arg, since it's unused.

[samuong, 2017/02/14 00:19:17]

Done.

+
+ private:
+  ChromeDriverInfoBarDelegate();
+
+  infobars::InfoBarDelegate::InfoBarIdentifier GetIdentifier() const override;
+  base::string16 GetMessageText() const override;
+  int GetButtons() const override;
+  bool ShouldExpire(const NavigationDetails& details) const override;

[Peter Kasting, 2017/02/13 23:16:52]

Nit: ShouldExpire() goes just above GetMessageText() (comes from an earlier
ancestor)

[samuong, 2017/02/14 00:19:17]

Done. Did this in the .cc file too.

+
+  DISALLOW_COPY_AND_ASSIGN(ChromeDriverInfoBarDelegate);
+};
+
+#endif  // CHROME_BROWSER_UI_STARTUP_CHROMEDRIVER_INFOBAR_DELEGATE_H_