diff options
author | Mustafa Emre Acer - Google <meacer@google.com> | 2016-03-08 10:33:32 -0800 |
---|---|---|
committer | Mustafa Emre Acer - Google <meacer@google.com> | 2016-03-08 18:35:33 +0000 |
commit | 086206ef10bc3cef44adb9be6ed9c84d2783f3e8 (patch) | |
tree | 23ce79da076ec175d8c0e809c3d4ab55e3b68749 | |
parent | b10fce996f8c1632db1fa864ef8e2aa26b47704e (diff) | |
download | chromium_src-086206ef10bc3cef44adb9be6ed9c84d2783f3e8.zip chromium_src-086206ef10bc3cef44adb9be6ed9c84d2783f3e8.tar.gz chromium_src-086206ef10bc3cef44adb9be6ed9c84d2783f3e8.tar.bz2 |
[Merge to 2623] Remove extra Release in pageCapture extension function implementation.
BUG=401364
Review URL: https://codereview.chromium.org/1761303003
Cr-Commit-Position: refs/heads/master@{#379361}
(cherry picked from commit 587ba472dfefe862855487c4e6faee859b686160)
Review URL: https://codereview.chromium.org/1776003003 .
Cr-Commit-Position: refs/branch-heads/2623@{#601}
Cr-Branched-From: 92d77538a86529ca35f9220bd3cd512cbea1f086-refs/heads/master@{#369907}
-rw-r--r-- | chrome/browser/extensions/api/page_capture/page_capture_api.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/extensions/api/page_capture/page_capture_api.cc b/chrome/browser/extensions/api/page_capture/page_capture_api.cc index ba0fd50..4707e64 100644 --- a/chrome/browser/extensions/api/page_capture/page_capture_api.cc +++ b/chrome/browser/extensions/api/page_capture/page_capture_api.cc @@ -157,7 +157,7 @@ void PageCaptureSaveAsMHTMLFunction::ReturnFailure(const std::string& error) { SendResponse(false); - Release(); // Balanced in Run() + // Must not Release() here, OnMessageReceived will call it eventually. } void PageCaptureSaveAsMHTMLFunction::ReturnSuccess(int64_t file_size) { |