New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Oct 2011
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug

Restricted
  • Only users with EditIssue permission may comment.



Sign in to add a comment
link

Issue 96063: Chrome: Crash Report - Stack Signature: PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)-1d3e21dd_1dca11dd_cfda3746_d343a4a2_efaaf4b4

Reported by dharani@chromium.org, Sep 9 2011 Project Member

Issue description

9 crash reports so far.

Product: Chrome
Stack Signature: PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)-B51905
New Signature Label: PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
New Signature Hash: 1d3e21dd_1dca11dd_cfda3746_d343a4a2_efaaf4b4

Report link: http://go/crash/reportdetail?reportid=3486acec620e0c2a

Meta information:
Product Name: Chrome
Product Version: 15.0.874.0
Report ID: 3486acec620e0c2a
Report Time: 2011/09/08 21:23:36, Thu
Uptime: 739 sec
Cumulative Uptime: 0 sec
OS Name: Windows NT
OS Version: 5.1.2600 Service Pack 3
CPU Architecture: x86
CPU Info: AuthenticAMD family 15 model 95 stepping 3
gpu-devid	 0x5974
gpu-driver	 8.30.0.0
gpu-psver	 2.0
gpu-venid	 0x1002
gpu-vsver	 2.0
num-extensions	 0
plat	 Win32
ptype	 renderer


Thread 0 *CRASHED* ( EXCEPTION_STACK_OVERFLOW @ 0x01e7373d )

0x01e7373d	 [chrome.dll	 - print_web_view_helper.cc:955	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
...... (975 stack frames dropped.)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
0x01e73872	 [chrome.dll	 - print_web_view_helper.cc:1000	PrintWebViewHelper::GetPageSizeAndMarginsInPoints(WebKit::WebFrame *,int,PrintMsg_Print_Params const &,printing::PageSizeMargins *)
 

Comment 1 by vandebo@chromium.org, Sep 9 2011

Cc: kmadhusu@chromium.org
Labels: Feature-PrintPreview
Looks like this started happening in M13.

Comment 2 by bugdroid1@chromium.org, Sep 10 2011

Project Member
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=100567

------------------------------------------------------------------------
r100567 | vandebo@chromium.org | Fri Sep 09 22:24:10 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/print_web_view_helper.cc?r1=100567&r2=100566&pathrev=100567

Add check to debug  issue 96063 .

BUG= 96063 
TEST=NONE


Review URL: http://codereview.chromium.org/7868001
------------------------------------------------------------------------

Comment 3 by kareng@google.com, Sep 12 2011

Labels: Stability-CodeYellow

Comment 4 by vandebo@chromium.org, Sep 12 2011

Labels: Merge-Requested
I'd like to merge this Check so I can get some hits on canary and see where we're entering the current infinite loop.

Comment 5 by kareng@google.com, Sep 12 2011

Labels: -Merge-Requested Merge-Approved

Comment 6 by bugdroid1@chromium.org, Sep 12 2011

Project Member
Labels: -merge-approved merge-merged-874
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=100750

------------------------------------------------------------------------
r100750 | vandebo@chromium.org | Mon Sep 12 13:17:23 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/branches/874/src/chrome/renderer/print_web_view_helper.cc?r1=100750&r2=100749&pathrev=100750

Merge 100567 - Add check to debug  issue 96063 .

BUG= 96063 
TEST=NONE


Review URL: http://codereview.chromium.org/7868001

TBR=vandebo@chromium.org
Review URL: http://codereview.chromium.org/7871004
------------------------------------------------------------------------

Comment 7 by kareng@google.com, Sep 19 2011

Status: Fixed

Comment 8 by vandebo@chromium.org, Sep 19 2011

Status: Assigned

Comment 9 by vandebo@chromium.org, Sep 19 2011

Labels: -ReleaseBlock-Beta -merge-merged-874 ReleaseBlock-Stable
The code merged was to aid in debugging - removing the merged label.  This doesn't appear to happen very frequently, so down grading to releaseblock-stable.

Comment 10 by kareng@google.com, Sep 26 2011

still working on this?

Comment 11 by vandebo@chromium.org, Sep 26 2011

Yes.  There are now a total of four reports since I added the Check, so pretty low right of occurance. I'll look a bit closer later.

Comment 12 by thestig@chromium.org, Sep 26 2011

Labels: OS-Windows
BTW, I only see this on Windows. There's other crashes in GetPageSizeAndMarginsInPoints() on Mac/Linux, but not the recursive one we see here.

Comment 13 by vandebo@chromium.org, Sep 27 2011

Indeed, lei pointed out more crashes (11) on beta, I was only looking on canary/dev.  I have CL in queue to add another check: printable area is zero, but it's not clear why, the new check should elucidate if Windows is saying it's zero, or it's getting a value of zero from our post processing.

Comment 14 by bugdroid1@chromium.org, Sep 28 2011

Project Member
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=103083

------------------------------------------------------------------------
r103083 | vandebo@chromium.org | Tue Sep 27 20:10:54 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/printing/page_setup.cc?r1=103083&r2=103082&pathrev=103083

Add check to debug  issue 96063 

BUG= 96063 
TEST=NONE


Review URL: http://codereview.chromium.org/8054031
------------------------------------------------------------------------

Comment 15 by vandebo@chromium.org, Sep 28 2011

Labels: Merge-Requested
When's the next beta push?  If it's really soon, then it makes sense to merge 103083 in order to get sufficient coverage.  If it's not for several days, we might get some hits on canary.

Comment 16 by kareng@google.com, Sep 29 2011

let's let it stew on canary we have a few days to next beta. we'll check in on Monday ok?

Comment 17 by kareng@google.com, Oct 3 2011

Labels: -Merge-Requested Merge-Approved

Comment 18 by bugdroid1@chromium.org, Oct 3 2011

Project Member
Labels: -merge-approved merge-merged-874
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=103745

------------------------------------------------------------------------
r103745 | vandebo@chromium.org | Mon Oct 03 10:57:03 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/branches/874/src/printing/page_setup.cc?r1=103745&r2=103744&pathrev=103745

Merge 103083 - Add check to debug  issue 96063 

BUG= 96063 
TEST=NONE


Review URL: http://codereview.chromium.org/8054031

TBR=vandebo@chromium.org
Review URL: http://codereview.chromium.org/8118009
------------------------------------------------------------------------

Comment 19 by vandebo@chromium.org, Oct 3 2011

Labels: -merge-merged-874
Removing the merge label because this is just for debugging and doesn't fix the issue.

Comment 20 by vandebo@chromium.org, Oct 8 2011

Labels: Merge-Requested
r104615 should fix this issue, but just landed on trunk.  Merge on Monday?

Comment 21 by kareng@google.com, Oct 10 2011

Labels: -Merge-Requested Merge-Approved
sg. pls confirm on canary that the issue is indeed fixed.

Comment 22 by vandebo@chromium.org, Oct 10 2011

Labels: -Merge-Approved Merge-Merged merge-merged-874
Status: Fixed
Fixed merged in r104744

Comment 23 by bugdroid1@chromium.org, Oct 10 2011

Project Member
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=104615

------------------------------------------------------------------------
r104615 | vandebo@chromium.org | Fri Oct 07 17:37:53 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/printing/print_settings_initializer_win.cc?r1=104615&r2=104614&pathrev=104615
 M http://src.chromium.org/viewvc/chrome/trunk/src/printing/page_setup.cc?r1=104615&r2=104614&pathrev=104615
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/print_web_view_helper.cc?r1=104615&r2=104614&pathrev=104615

Sanity check the printable area returned by windows.

BUG= 96063 
TEST=NONE


Review URL: http://codereview.chromium.org/8200010
------------------------------------------------------------------------

Comment 24 by bugdroid1@chromium.org, Oct 10 2011

Project Member

Comment 25 by thestig@chromium.org, Nov 3 2011

BTW, I'm seeing sanity check failures, e.g. http://crash/reportdetail?reportid=c71c12444ea66d41

Comment 26 by bugdroid1@chromium.org, Oct 13 2012

Project Member
Labels: Restrict-AddIssueComment-Commit
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.

Comment 27 by bugdroid1@chromium.org, Mar 10 2013

Project Member
Labels: -Area-WebKit -Mstone-15 -Feature-PrintPreview Cr-Content M-15 Cr-UI-Browser-PrintPreview

Comment 28 by bugdroid1@chromium.org, Mar 13 2013

Project Member
Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue

Comment 29 by bugdroid1@chromium.org, Apr 6 2013

Project Member
Labels: -Cr-Content Cr-Blink

Sign in to add a comment