Fix inclusion of SkDrawPictureCallback.h

Fix inclusion of SkDrawPictureCallback.h

SkDrawPictureCallback is going to be renamed
(https://codereview.chromium.org/837693002/). This CL
updates the includes to stage the transition.

Committed: https://crrev.com/90334831a0069cee46abf26a0ed2b04a25a498d8
Cr-Commit-Position: refs/heads/master@{#310119}

[robertphillips, 2015-01-06 15:29:40]

robertphillips@google.com changed reviewers:
+ enne@chromium.org, jamesr@chromium.org, nduca@chromium.org, reed@google.com

[danakj, 2015-01-06 15:42:46]

danakj@chromium.org changed reviewers:
+ danakj@chromium.org

[danakj, 2015-01-06 15:42:46]

https://codereview.chromium.org/806553004/diff/20001/cc/resources/drawing_dis...
File cc/resources/drawing_display_item.cc (right):

https://codereview.chromium.org/806553004/diff/20001/cc/resources/drawing_dis...
cc/resources/drawing_display_item.cc:8: #include
"third_party/skia/include/core/SkDrawPictureCallback.h"
A forward declaration seems like enough here?

https://codereview.chromium.org/806553004/diff/20001/cc/resources/picture.cc
File cc/resources/picture.cc (right):

https://codereview.chromium.org/806553004/diff/20001/cc/resources/picture.cc#...
cc/resources/picture.cc:24: #include
"third_party/skia/include/core/SkDrawPictureCallback.h"
the forward declaration from the .h should be enough here? It's not dereffed?

[robertphillips, 2015-01-06 16:08:09]

https://codereview.chromium.org/806553004/diff/20001/cc/resources/drawing_dis...
File cc/resources/drawing_display_item.cc (right):

https://codereview.chromium.org/806553004/diff/20001/cc/resources/drawing_dis...
cc/resources/drawing_display_item.cc:8: #include
"third_party/skia/include/core/SkDrawPictureCallback.h"
This include will go away after the Skia-side renaming has landed and I perform
the Chromium-side renaming. It is needed for the Skia-side renaming to land
however so that it is known that an SkDrawPictureCallback is derived from
SkAbortCallback.

https://codereview.chromium.org/806553004/diff/20001/cc/resources/picture.cc
File cc/resources/picture.cc (right):

https://codereview.chromium.org/806553004/diff/20001/cc/resources/picture.cc#...
cc/resources/picture.cc:24: #include
"third_party/skia/include/core/SkDrawPictureCallback.h"
This will also go away after the Chromium-side renaming. It is again needed so
the compiler knows SkDrawPictureCallback derives from SkAbortCallback.

https://codereview.chromium.org/806553004/diff/20001/cc/resources/picture.h
File cc/resources/picture.h (right):

https://codereview.chromium.org/806553004/diff/20001/cc/resources/picture.h#n...
cc/resources/picture.h:25: 
This also will go away after the renaming as the pre-declaration of
SkAbortCallback in SkPicture.h should suffice.

[danakj, 2015-01-06 16:11:30]

I see, thanks. LGTM

[robertphillips, 2015-01-06 17:42:40]

robertphillips@google.com changed reviewers:
+ fmalita@google.com

[f(malita), 2015-01-06 17:46:18]

fmalita@chromium.org changed reviewers:
+ fmalita@chromium.org

[f(malita), 2015-01-06 17:46:18]

skia/ lgtm

[robertphillips, 2015-01-06 17:46:38]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2015-01-06 17:47:04]

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

[commit-bot: I haz the power, 2015-01-06 20:03:15]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-06 20:04:07]

Patchset 2 (id:??) landed as
https://crrev.com/90334831a0069cee46abf26a0ed2b04a25a498d8
Cr-Commit-Position: refs/heads/master@{#310119}