diff options
| author | kmadhusu@chromium.org <kmadhusu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-03-30 16:29:28 +0000 |
|---|---|---|
| committer | kmadhusu@chromium.org <kmadhusu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-03-30 16:29:28 +0000 |
| commit | 351676e05007b8eb8d124f9d2cce80108ed314a1 (patch) | |
| tree | d483fb8fe9c10fbb62dd9efac4b381abef216380 /chrome/browser/printing/print_view_manager.cc | |
| parent | 8f608c5bd8ab842a934068149e2da3250cd92686 (diff) | |
| download | chromium_src-351676e05007b8eb8d124f9d2cce80108ed314a1.zip chromium_src-351676e05007b8eb8d124f9d2cce80108ed314a1.tar.gz chromium_src-351676e05007b8eb8d124f9d2cce80108ed314a1.tar.bz2 | |
DCHECK triggered while trying to print a webpage.
DCHECK is triggered from InitFromData() function while validating the |context_| member variable.
BUG=77842
TEST=Printing works after code changes.
Review URL: http://codereview.chromium.org/6770027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79840 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/printing/print_view_manager.cc')
| -rw-r--r-- | chrome/browser/printing/print_view_manager.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/chrome/browser/printing/print_view_manager.cc b/chrome/browser/printing/print_view_manager.cc index 20ac4ea..756e5fe 100644 --- a/chrome/browser/printing/print_view_manager.cc +++ b/chrome/browser/printing/print_view_manager.cc @@ -129,9 +129,11 @@ void PrintViewManager::OnDidPrintPage( } } - scoped_ptr<NativeMetafile> metafile(NativeMetafileFactory::Create()); + scoped_ptr<NativeMetafile> metafile; if (metafile_must_be_valid) { - if (!metafile->InitFromData(shared_buf.memory(), params.data_size)) { + metafile.reset(NativeMetafileFactory::CreateFromData(shared_buf.memory(), + params.data_size)); + if (!metafile.get()) { NOTREACHED() << "Invalid metafile header"; tab_contents()->Stop(); return; |