Create AutofillDriverImpl unit test.

components/autofill/content/browser/autofill_driver_impl.h

 // 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.
 
 #ifndef COMPONENTS_AUTOFILL_CONTENT_BROWSER_AUTOFILL_DRIVER_IMPL_H_
 #define COMPONENTS_AUTOFILL_CONTENT_BROWSER_AUTOFILL_DRIVER_IMPL_H_
 
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
@@ skipping 36 matching lines @@
 
   AutofillExternalDelegate* autofill_external_delegate() {
     return autofill_external_delegate_.get();
   }
 
   // Sets the external delegate to |delegate| both within this class and in the
   // shared Autofill code. Takes ownership of |delegate|.
   void SetAutofillExternalDelegate(
       scoped_ptr<AutofillExternalDelegate> delegate);
 
-  AutofillManager* autofill_manager() { return &autofill_manager_; }
+  AutofillManager* autofill_manager() { return autofill_manager_.get(); }
 
  private:
   AutofillDriverImpl(
       content::WebContents* web_contents,
       autofill::AutofillManagerDelegate* delegate,
       const std::string& app_locale,
       AutofillManager::AutofillDownloadManagerState enable_download_manager,
       bool enable_native_ui);
   virtual ~AutofillDriverImpl();
 
   // content::WebContentsObserver:
   virtual void DidNavigateMainFrame(
       const content::LoadCommittedDetails& details,
       const content::FrameNavigateParams& params) OVERRIDE;
   virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
 
+  // Sets the manager to |manager|. Takes ownership of |manager|.
+  void set_autofill_manager(scoped_ptr<AutofillManager> manager) {
+    autofill_manager_.reset(manager.release());

[Ilya Sherman, 2013/06/20 00:07:16]

nit: Can you use .Pass() instead of .release() here?  I think that's preferred,
when possible.

[blundell, 2013/06/21 12:55:23]

Done.

+  }
+
   // AutofillExternalDelegate instance that this object instantiates in the
   // case where the autofill native UI is enabled.
   scoped_ptr<AutofillExternalDelegate> autofill_external_delegate_;
 
   // AutofillManager instance via which this object drives the shared Autofill
   // code.
-  AutofillManager autofill_manager_;
+  scoped_ptr<AutofillManager> autofill_manager_;
+
+  friend class AutofillDriverImplTest;

[Ilya Sherman, 2013/06/20 00:07:16]

Please don't add a friend class; instead, move set_autofill_manager to have
protected visibility, and create a subclass for testing that ups the visibility
to public.

[blundell, 2013/06/21 12:55:23]

Done.

 };
 
 }  // namespace autofill
 
 #endif  // COMPONENTS_AUTOFILL_CONTENT_BROWSER_AUTOFILL_DRIVER_IMPL_H_