Change infobar creation to use a public static Create() method on the infobar delegate classes.

chrome/browser/three_d_api_observer.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/browser/three_d_api_observer.h"
 
 #include "base/metrics/histogram.h"
 #include "chrome/browser/api/infobars/confirm_infobar_delegate.h"
 #include "chrome/browser/api/infobars/infobar_service.h"
 #include "content/public/browser/gpu_data_manager.h"
@@ skipping 11 matching lines @@
   THREE_D_INFOBAR_CLOSED_WITHOUT_ACTION,
   THREE_D_INFOBAR_DISMISSAL_MAX
 };
 
 }  // namespace
 
 // ThreeDAPIInfoBarDelegate ---------------------------------------------
 
 class ThreeDAPIInfoBarDelegate : public ConfirmInfoBarDelegate {
  public:
+  // Creates a 3D API delegate and adds it to |infobar_service|.
+  static void Create(InfoBarService* infobar_service,
+                     const GURL& url,
+                     content::ThreeDAPIType requester);
+
+ private:
   ThreeDAPIInfoBarDelegate(
       InfoBarService* owner,
       const GURL& url,
       content::ThreeDAPIType requester);
-
- private:
   virtual ~ThreeDAPIInfoBarDelegate();
 
   // ConfirmInfoBarDelegate:
   virtual bool EqualsDelegate(InfoBarDelegate* delegate) const OVERRIDE;
   virtual ThreeDAPIInfoBarDelegate* AsThreeDAPIInfoBarDelegate() OVERRIDE;
   virtual string16 GetMessageText() const OVERRIDE;
   virtual string16 GetButtonLabel(InfoBarButton button) const OVERRIDE;
   virtual bool Accept() OVERRIDE;
   virtual bool Cancel() OVERRIDE;
   virtual string16 GetLinkText() const OVERRIDE;
   virtual bool LinkClicked(WindowOpenDisposition disposition) OVERRIDE;
 
   GURL url_;
   content::ThreeDAPIType requester_;
   // Basically indicates whether the infobar was displayed at all, or
   // was a temporary instance thrown away by the InfobarService.
   mutable bool message_text_queried_;
   bool action_taken_;
 
   DISALLOW_COPY_AND_ASSIGN(ThreeDAPIInfoBarDelegate);
 };
 
+// static
+void ThreeDAPIInfoBarDelegate::Create(InfoBarService* infobar_service,
+                                      const GURL& url,
+                                      content::ThreeDAPIType requester) {
+  infobar_service->AddInfoBar(scoped_ptr<InfoBarDelegate>(
+      new ThreeDAPIInfoBarDelegate(infobar_service, url, requester)));
+}
+
 ThreeDAPIInfoBarDelegate::ThreeDAPIInfoBarDelegate(
     InfoBarService* owner,
     const GURL& url,
     content::ThreeDAPIType requester)
     : ConfirmInfoBarDelegate(owner),
       url_(url),
       requester_(requester),
       message_text_queried_(false),
       action_taken_(false) {
 }
@@ skipping 80 matching lines @@
 
 // ThreeDAPIObserver ----------------------------------------------------
 
 ThreeDAPIObserver::ThreeDAPIObserver(content::WebContents* web_contents)
     : WebContentsObserver(web_contents) {}
 
 ThreeDAPIObserver::~ThreeDAPIObserver() {}
 
 void ThreeDAPIObserver::DidBlock3DAPIs(const GURL& url,
                                        content::ThreeDAPIType requester) {
-  InfoBarService* service = InfoBarService::FromWebContents(web_contents());
-  service->AddInfoBar(new ThreeDAPIInfoBarDelegate(service, url, requester));
+  ThreeDAPIInfoBarDelegate::Create(
+      InfoBarService::FromWebContents(web_contents()), url, requester);
 }