Use refs for GraphicsContext

Use refs for GraphicsContext

Committed: https://crrev.com/a734170517b9576bedce9feeddfe53473510c4e9
Cr-Commit-Position: refs/heads/master@{#364486}

[Xianzhu, 2015-12-10 16:55:34]

wangxianzhu@chromium.org changed reviewers:
+ pdr@chromium.org

[Xianzhu, 2015-12-10 16:55:36]

You can just review the non-trivial changes noted below.

https://codereview.chromium.org/1512803004/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/layout/ImageQualityController.h (left):

https://codereview.chromium.org/1512803004/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/layout/ImageQualityController.h:58:
InterpolationQuality chooseInterpolationQuality(GraphicsContext*, const
LayoutObject*, Image*, const void* layer, const LayoutSize&);
The GraphicsContext parameter was not used.

https://codereview.chromium.org/1512803004/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/paint/SVGPaintContext.cpp (left):

https://codereview.chromium.org/1512803004/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/paint/SVGPaintContext.cpp:47: // Reset the paint
info after the filter effect has been completed.
The next patch set will restore the above line of comment.

https://codereview.chromium.org/1512803004/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/paint/SVGPaintContext.h (right):

https://codereview.chromium.org/1512803004/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/paint/SVGPaintContext.h:88: OwnPtr<PaintInfo>
m_filterPaintInfo;
We no longer allow changing the context field of PaintInfo because it's now a
reference. m_filterPaintInfo will override m_paintInfo when it is not null.

[pdr., 2015-12-10 18:36:32]

On 2015/12/10 at 16:55:36, wangxianzhu wrote:
> You can just review the non-trivial changes noted below.
> 
>
https://codereview.chromium.org/1512803004/diff/80001/third_party/WebKit/Sour...
> File third_party/WebKit/Source/core/layout/ImageQualityController.h (left):
> 
>
https://codereview.chromium.org/1512803004/diff/80001/third_party/WebKit/Sour...
> third_party/WebKit/Source/core/layout/ImageQualityController.h:58:
InterpolationQuality chooseInterpolationQuality(GraphicsContext*, const
LayoutObject*, Image*, const void* layer, const LayoutSize&);
> The GraphicsContext parameter was not used.
> 
>
https://codereview.chromium.org/1512803004/diff/80001/third_party/WebKit/Sour...
> File third_party/WebKit/Source/core/paint/SVGPaintContext.cpp (left):
> 
>
https://codereview.chromium.org/1512803004/diff/80001/third_party/WebKit/Sour...
> third_party/WebKit/Source/core/paint/SVGPaintContext.cpp:47: // Reset the
paint info after the filter effect has been completed.
> The next patch set will restore the above line of comment.
> 
>
https://codereview.chromium.org/1512803004/diff/80001/third_party/WebKit/Sour...
> File third_party/WebKit/Source/core/paint/SVGPaintContext.h (right):
> 
>
https://codereview.chromium.org/1512803004/diff/80001/third_party/WebKit/Sour...
> third_party/WebKit/Source/core/paint/SVGPaintContext.h:88: OwnPtr<PaintInfo>
m_filterPaintInfo;
> We no longer allow changing the context field of PaintInfo because it's now a
reference. m_filterPaintInfo will override m_paintInfo when it is not null.

LGTM

[Xianzhu, 2015-12-10 19:34:42]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2015-12-10 19:34:43]

The patchset sent to the CQ was uploaded after l-g-t-m from pdr@chromium.org
Link to the patchset: https://codereview.chromium.org/1512803004/#ps100001
(title: " ")

[commit-bot: I haz the power, 2015-12-10 19:39:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1512803004/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1512803004/100001

[commit-bot: I haz the power, 2015-12-10 21:45:26]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-12-10 21:46:11]

Description was changed from

==========
Use refs for GraphicsContext
==========

to

==========
Use refs for GraphicsContext

Committed: https://crrev.com/a734170517b9576bedce9feeddfe53473510c4e9
Cr-Commit-Position: refs/heads/master@{#364486}
==========

[commit-bot: I haz the power, 2015-12-10 21:46:13]

Patchset 6 (id:??) landed as
https://crrev.com/a734170517b9576bedce9feeddfe53473510c4e9
Cr-Commit-Position: refs/heads/master@{#364486}