SkPDF Metafile: fix recording canvas scaling

SkPDF Metafile: fix recording canvas scaling

Scale the recording canvas's page size, not it's canvas matrix.  This
allows getTotalMatrix() to return the identity matrix.

BUG=469656
Committed: https://crrev.com/5baa8fb45970fbc343e130c8d14ba528b3dd934a
Cr-Commit-Position: refs/heads/master@{#323820}

[hal.canary, 2015-04-03 19:33:27]

halcanary@google.com changed reviewers:
+ fmalita@chromium.org, vitalybuka@chromium.org

[f(malita), 2015-04-03 19:35:21]

lgtm

[f(malita), 2015-04-03 19:35:40]

The CQ bit was checked by fmalita@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-04-03 19:35:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1060603003/1

[commit-bot: I haz the power, 2015-04-03 19:46:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-03 19:46:42]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Vitaly Buka (NO REVIEWS), 2015-04-03 20:51:39]

lgtm

https://codereview.chromium.org/1060603003/diff/1/printing/pdf_metafile_skia.cc
File printing/pdf_metafile_skia.cc (right):

https://codereview.chromium.org/1060603003/diff/1/printing/pdf_metafile_skia....
printing/pdf_metafile_skia.cc:99: DCHECK_GT(scale_factor, 0.0f);
DCHECK is unnecessary 
crash on divided by 0 is obviuos

[hal.canary, 2015-04-03 20:54:40]

The CQ bit was checked by halcanary@google.com

[commit-bot: I haz the power, 2015-04-03 20:55:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1060603003/1

[hal.canary, 2015-04-03 20:58:02]

On 2015/04/03 20:51:39, Vitaly Buka wrote:
> lgtm
> 
>
https://codereview.chromium.org/1060603003/diff/1/printing/pdf_metafile_skia.cc
> File printing/pdf_metafile_skia.cc (right):
> 
>
https://codereview.chromium.org/1060603003/diff/1/printing/pdf_metafile_skia....
> printing/pdf_metafile_skia.cc:99: DCHECK_GT(scale_factor, 0.0f);
> DCHECK is unnecessary 
> crash on divided by 0 is obviuos

Yes, I also wanted to assert that you aren't passing in a negative number.

[Vitaly Buka (NO REVIEWS), 2015-04-03 21:00:14]

On 2015/04/03 20:58:02, Hal Canary wrote:
> On 2015/04/03 20:51:39, Vitaly Buka wrote:
> > lgtm
> > 
> >
>
https://codereview.chromium.org/1060603003/diff/1/printing/pdf_metafile_skia.cc
> > File printing/pdf_metafile_skia.cc (right):
> > 
> >
>
https://codereview.chromium.org/1060603003/diff/1/printing/pdf_metafile_skia....
> > printing/pdf_metafile_skia.cc:99: DCHECK_GT(scale_factor, 0.0f);
> > DCHECK is unnecessary 
> > crash on divided by 0 is obviuos
> 
> Yes, I also wanted to assert that you aren't passing in a negative number.

sgtm

[commit-bot: I haz the power, 2015-04-03 21:39:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-04-03 21:40:42]

Patchset 1 (id:??) landed as
https://crrev.com/5baa8fb45970fbc343e130c8d14ba528b3dd934a
Cr-Commit-Position: refs/heads/master@{#323820}