diff options
| author | vitalybuka@chromium.org <vitalybuka@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-23 21:41:39 +0000 |
|---|---|---|
| committer | vitalybuka@chromium.org <vitalybuka@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-23 21:41:39 +0000 |
| commit | 4de981d0be7432f2f92110062806a7bf93181969 (patch) | |
| tree | 5867aeba8e9193095475cb50e7efd8d5155d277d /chrome/browser/printing | |
| parent | 2d7a4dc30d657d21dab22e76b89d2098ee7bc564 (diff) | |
| download | chromium_src-4de981d0be7432f2f92110062806a7bf93181969.zip chromium_src-4de981d0be7432f2f92110062806a7bf93181969.tar.gz chromium_src-4de981d0be7432f2f92110062806a7bf93181969.tar.bz2 | |
Revert 211966 "Stores JobEventDetails in scoped_refptr."
Speculative revert as potatial sourse of http://crbug.com/263359
If next Canary will have no crashe we will fix issue in PdfMetafileCg and re-apply this patch.
BUG=260806, 263359
> Stores JobEventDetails in scoped_refptr.
>
> BUG=260806
>
> Review URL: https://chromiumcodereview.appspot.com/19299006
TBR=vitalybuka@chromium.org
Review URL: https://codereview.chromium.org/19809003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213235 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/printing')
| -rw-r--r-- | chrome/browser/printing/print_job_worker.cc | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/chrome/browser/printing/print_job_worker.cc b/chrome/browser/printing/print_job_worker.cc index 0ffcaf2..a3298f5 100644 --- a/chrome/browser/printing/print_job_worker.cc +++ b/chrome/browser/printing/print_job_worker.cc @@ -40,13 +40,12 @@ void NotificationCallback(PrintJobWorkerOwner* print_job, JobEventDetails::Type detail_type, PrintedDocument* document, PrintedPage* page) { - scoped_refptr<JobEventDetails> details(new JobEventDetails(detail_type, - document, page)); + JobEventDetails* details = new JobEventDetails(detail_type, document, page); content::NotificationService::current()->Notify( chrome::NOTIFICATION_PRINT_JOB_EVENT, // We know that is is a PrintJob object in this circumstance. content::Source<PrintJob>(static_cast<PrintJob*>(print_job)), - content::Details<JobEventDetails>(details.get())); + content::Details<JobEventDetails>(details)); } PrintJobWorker::PrintJobWorker(PrintJobWorkerOwner* owner) |