Remove extra Release in pageCapture extension function implementation.

chrome/browser/extensions/api/page_capture/page_capture_api.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/extensions/api/page_capture/page_capture_api.h"
 
 #include <limits>
 
 #include "base/bind.h"
 #include "base/files/file_util.h"
@@ skipping 139 matching lines @@
   ReturnSuccess(mhtml_file_size);
 }
 
 void PageCaptureSaveAsMHTMLFunction::ReturnFailure(const std::string& error) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
   error_ = error;
 
   SendResponse(false);
 
-  Release();  // Balanced in Run()
+  // Must not Release() here, OnMessageReceived will call it eventually.

[Devlin, 2016/03/04 20:31:15]

Wow, this function is crazy.

[meacer, 2016/03/04 20:37:59]

Well, at least there is only one AddRef()/Release() pair in the text now :)

 }
 
 void PageCaptureSaveAsMHTMLFunction::ReturnSuccess(int64_t file_size) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
   WebContents* web_contents = GetWebContents();
   if (!web_contents || !render_frame_host()) {
     ReturnFailure(kTabClosedError);
     return;
   }
@@ skipping 22 matching lines @@
                                     GetProfile(),
                                     include_incognito(),
                                     &browser,
                                     NULL,
                                     &web_contents,
                                     NULL)) {
     return NULL;
   }
   return web_contents;
 }