Fix memory leak in unit_test SelectFileDialogExtensionTest

chrome/browser/ui/views/select_file_dialog_extension_unittest.cc

 // Copyright (c) 2011 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/ui/views/select_file_dialog_extension.h"
 
 #include "base/file_path.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 class SelectFileDialogExtensionTest : public testing::Test {
@@ skipping 16 matching lines @@
  public:
   explicit SelfDeletingClient(int32 tab_id) {
     dialog_ = SelectFileDialogExtensionTest::CreateDialog(this, tab_id);
   }
 
   virtual ~SelfDeletingClient() {
     if (dialog_.get())
       dialog_->ListenerDestroyed();
   }
 
+  SelectFileDialogExtension* dialog() const { return dialog_.get(); }
+
   // SelectFileDialog::Listener implementation
   virtual void FileSelected(const FilePath& path,
                             int index, void* params) OVERRIDE {
     delete this;
   }
 
  private:
   scoped_refptr<SelectFileDialogExtension> dialog_;
 };
 
 TEST_F(SelectFileDialogExtensionTest, SelfDeleting) {
   const int32 kTabId = 123;
-  // Registers itself with an internal map, so we don't need the pointer,
-  // and it would be unused anyway.
-  new SelfDeletingClient(kTabId);
+  SelfDeletingClient* client = new SelfDeletingClient(kTabId);
   // Ensure we don't crash or trip an Address Sanitizer warning about
   // use-after-free.
   SelectFileDialogExtension::OnFileSelected(kTabId, FilePath(), 0);
+  // Simulate closing the dialog so the listener gets invoked.
+  client->dialog()->ExtensionDialogClosing(NULL);
 }