Moving updateClipConservativelyUsingBounds into SkCanvas

Moving updateClipConservativelyUsingBounds into SkCanvas

This method is being moved from SkPictureRecord to become a protected member of SkCanvas so that other classes derived from SkCanvas can use it.
This Patch also applies the fix provided by updateClipConservativelyUsingBounds to SkPictureUtils::GatherPixelRefs

BUG=https://code.google.com/p/chromium/issues/detail?id=244893

Committed: https://code.google.com/p/skia/source/detail?r=9326

[Justin Novosad, 2013-05-29 16:58:37]

PTAL

Purpose: allow chromium's AnalysisCanvas and friends to track clip bounds
correctly.

[reed1, 2013-05-29 18:54:01]

lgtm w/ request for comment explaining the use of SkCanvas:: in calls to this.

https://codereview.chromium.org/15894005/diff/1/src/core/SkCanvas.cpp
File src/core/SkCanvas.cpp (right):

https://codereview.chromium.org/15894005/diff/1/src/core/SkCanvas.cpp#newcode...
src/core/SkCanvas.cpp:1282: bool
SkCanvas::updateClipConservativelyUsingBounds(const SkRect& bounds, SkRegion::Op
op,
/**
 * This *never* calls back through a virtual on canvas, so subclasses don't have
to worry about
 * getting caught in a loop. Thus anywhere we call a virtual method, we
explicitly prefix it with
 * SkCanvas:: to be sure to call the base-class.
 */

[Justin Novosad, 2013-05-29 19:09:53]

Committed patchset #1 manually as r9326 (presubmit successful).