Summary: | REGRESSION: printing is broken if stylesheet has @page | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> | ||||||||
Component: | Printing | Assignee: | Yuzo Fujishima <yuzo> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | ap, hamaji, mitz | ||||||||
Priority: | P1 | Keywords: | InRadar | ||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | OS X 10.5 | ||||||||||
Attachments: |
|
Description
Simon Fraser (smfr)
2010-06-10 16:22:05 PDT
Sorry for the regression. I'll look into this. Created attachment 58441 [details] Fix Bug 40452: REGRESSION: printing is broken if stylesheet has @page Comment on attachment 58441 [details] Fix Bug 40452: REGRESSION: printing is broken if stylesheet has @page WebCore/css/CSSStyleSelector.cpp:2795 + if (childItem->isPageRule()) { Cannot we create a function to do this so we can share code? WebCore/ChangeLog:8 + No new tests because currently there is no easy way to test print layout. I'm not 100% sure, but I guess we can test this patch by setPrinting though pixel dump isn't supported yet. Created attachment 58444 [details]
Refactored and added a test.
Thank you for the review. Can you take another look? (In reply to comment #4) > (From update of attachment 58441 [details]) > WebCore/css/CSSStyleSelector.cpp:2795 > + if (childItem->isPageRule()) { > Cannot we create a function to do this so we can share code? Agreed. Extracted a method. > > WebCore/ChangeLog:8 > + No new tests because currently there is no easy way to test print layout. > I'm not 100% sure, but I guess we can test this patch by setPrinting though pixel dump isn't supported yet. Added a test. Comment on attachment 58444 [details]
Refactored and added a test.
Looks good to me.
Ah, please add chromium's test_expectations.txt before you land. Committed r60992: <http://trac.webkit.org/changeset/60992> Committed r63070: <http://trac.webkit.org/changeset/63070> |