Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(275)

Side by Side Diff: android_webview/native/aw_pdf_exporter.cc

Issue 1013843002: favor DCHECK_CURRENTLY_ON for better logs in android_webview/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: change a few more Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "android_webview/native/aw_pdf_exporter.h" 5 #include "android_webview/native/aw_pdf_exporter.h"
6 6
7 #include "base/android/jni_android.h" 7 #include "base/android/jni_android.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "content/public/browser/browser_thread.h" 9 #include "content/public/browser/browser_thread.h"
10 #include "content/public/browser/web_contents.h" 10 #include "content/public/browser/web_contents.h"
(...skipping 27 matching lines...) Expand all
38 if (obj.is_null()) 38 if (obj.is_null())
39 return; 39 return;
40 // Clear the Java peer's weak pointer to |this| object. 40 // Clear the Java peer's weak pointer to |this| object.
41 Java_AwPdfExporter_setNativeAwPdfExporter(env, obj.obj(), 0); 41 Java_AwPdfExporter_setNativeAwPdfExporter(env, obj.obj(), 0);
42 } 42 }
43 43
44 void AwPdfExporter::ExportToPdf(JNIEnv* env, 44 void AwPdfExporter::ExportToPdf(JNIEnv* env,
45 jobject obj, 45 jobject obj,
46 int fd, 46 int fd,
47 jobject cancel_signal) { 47 jobject cancel_signal) {
48 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 48 DCHECK_CURRENTLY_ON(BrowserThread::UI);
49 CreatePdfSettings(env, obj); 49 CreatePdfSettings(env, obj);
50 PrintManager* print_manager = 50 PrintManager* print_manager =
51 PrintManager::CreateForWebContents( 51 PrintManager::CreateForWebContents(
52 web_contents_, print_settings_.get(), fd, this); 52 web_contents_, print_settings_.get(), fd, this);
53 if (!print_manager->PrintNow()) 53 if (!print_manager->PrintNow())
54 DidExportPdf(false); 54 DidExportPdf(false);
55 } 55 }
56 56
57 namespace { 57 namespace {
58 // Converts from 1/1000 of inches to device units using DPI. 58 // Converts from 1/1000 of inches to device units using DPI.
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
108 // TODO(sgurun) implement. Needs connecting with the |cancel_signal| passed 108 // TODO(sgurun) implement. Needs connecting with the |cancel_signal| passed
109 // in the constructor. 109 // in the constructor.
110 return false; 110 return false;
111 } 111 }
112 112
113 bool RegisterAwPdfExporter(JNIEnv* env) { 113 bool RegisterAwPdfExporter(JNIEnv* env) {
114 return RegisterNativesImpl(env); 114 return RegisterNativesImpl(env);
115 } 115 }
116 116
117 } // namespace android_webview 117 } // namespace android_webview
OLDNEW
« no previous file with comments | « android_webview/native/aw_http_auth_handler.cc ('k') | android_webview/native/aw_quota_manager_bridge_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698