Disable info bar animations during automated testing.

components/infobars/core/infobar_manager.cc

Index: components/infobars/core/infobar_manager.cc
diff --git a/components/infobars/core/infobar_manager.cc b/components/infobars/core/infobar_manager.cc
index 06d1b2861c9f8e5a6376aaaec1a60dd168e3ab2e..bed4f50dc708a6ffc6f1a7e32f74e3e4d580ddf8 100644
--- a/components/infobars/core/infobar_manager.cc
+++ b/components/infobars/core/infobar_manager.cc
@@ -101,7 +101,7 @@ void InfoBarManager::RemoveObserver(Observer* obs) {
 }
 
 InfoBarManager::InfoBarManager()
-    : infobars_enabled_(true) {
+    : infobars_enabled_(true), animations_enabled_(true) {

[Peter Kasting, 2017/02/23 00:33:56]

Nit: You're welcome to move these initializations to the member declaration
points in the header if you want.

[samuong, 2017/02/23 00:45:26]

Done. Just to confirm, you meant using brace-initialization?

[Peter Kasting, 2017/02/23 00:51:34]

I would use "= true" in preference to "{true}", since AFAIK both are legal, and
that's what Chromium prefers in such cases:
https://www.chromium.org/developers/coding-style/cpp-dos-and-donts#TOC-Variab...

   if (base::CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kDisableInfoBars))
     infobars_enabled_ = false;