InfoBarService inherits from InfoBarManager

chrome/browser/translate/translate_browsertest.cc

 // Copyright 2013 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 "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/infobars/infobar.h"
@@ skipping 74 matching lines @@
           browser()->tab_strip_model()->GetActiveWebContents();
       if (web_contents)
         infobar_service_ = InfoBarService::FromWebContents(web_contents);
     }
     if (!infobar_service_) {
       ADD_FAILURE() << "infobar service is not available";
       return NULL;
     }
 
     TranslateInfoBarDelegate* delegate = NULL;
-    InfoBarManager* infobar_manager = infobar_service_->infobar_manager();
-    for (size_t i = 0; i < infobar_manager->infobar_count(); ++i) {
+    for (size_t i = 0; i < infobar_service_->infobar_count(); ++i) {
       // Check if the shown infobar is a confirm infobar coming from the
       // |kTranslateSecurityOrigin| flag specified in SetUpCommandLine().
       // This infobar appears in all tests of TranslateBrowserTest and can be
       // ignored here.
-      ConfirmInfoBarDelegate* confirm = infobar_manager->infobar_at(i)
-          ->delegate()->AsConfirmInfoBarDelegate();
+      ConfirmInfoBarDelegate* confirm = infobar_service_->infobar_at(i)
+                                            ->delegate()
+                                            ->AsConfirmInfoBarDelegate();

[Peter Kasting, 2014/04/10 00:08:59]

Nit: I'd condense/wrap like this:

      if (infobar_service_->infobar_at(i)->delegate()->
          AsConfirmInfoBarDelegate())
        continue;

      TranslateInfoBarDelegate* translate =
          infobar_service_->infobar_at(i)->delegate()->
              AsTranslateInfoBarDelegate();
      ...

       if (confirm)
         continue;
 
-      TranslateInfoBarDelegate* translate = infobar_manager->infobar_at(i)
-          ->delegate()->AsTranslateInfoBarDelegate();
+      TranslateInfoBarDelegate* translate = infobar_service_->infobar_at(i)
+                                                ->delegate()
+                                                ->AsTranslateInfoBarDelegate();
       if (translate) {
         EXPECT_FALSE(delegate) << "multiple infobars are shown unexpectedly";
         delegate = translate;
         continue;
       }
 
       // Other infobar should not be shown.
       EXPECT_TRUE(delegate);
     }
     return delegate;
@@ skipping 240 matching lines @@
       GetNonSecureURL(kUpdateLocationAtOnloadTestPath));
 
   // Wait for the page title is changed after the test finished.
   const base::string16 result = watcher.WaitAndGetTitle();
   EXPECT_EQ("PASS", base::UTF16ToASCII(result));
 
   // Check if there is no Translate infobar.
   translate = GetExistingTranslateInfoBarDelegate();
   EXPECT_FALSE(translate);
 }