Remove unneeded AddRef/Release calls handled by baes::Bind

electron · Feb 24, 2017 · 7428b6a · 7428b6a
1 parent 6837ec8
commit 7428b6a
Showing 1 changed file with 0 additions and 3 deletions.
diff --git a/atom/browser/web_dialog_helper.cc b/atom/browser/web_dialog_helper.cc
@@ -36,8 +36,6 @@ class FileSelectHelper : public base::RefCounted<FileSelectHelper>,
     auto web_contents = content::WebContents::FromRenderFrameHost(
         render_frame_host);
     content::WebContentsObserver::Observe(web_contents);
-    // Add ref that will be released when the dialog is completed
-    AddRef();
   }
 
   void ShowOpenDialog(const file_dialog::DialogSettings& settings) {
@@ -90,7 +88,6 @@ class FileSelectHelper : public base::RefCounted<FileSelectHelper>,
       const std::vector<content::FileChooserFileInfo>& file_info) {
     if (render_frame_host_)
       render_frame_host_->FilesSelectedInChooser(file_info, mode_);
-    Release();
   }
 
   // content::WebContentsObserver: