Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(69)

Issue 1207813002: Remove a Mac control clip that does nothing. (Closed)

Created:
4 years, 10 months ago by Stephen Chennney
Modified:
4 years, 10 months ago
Reviewers:
pdr., chrishtr
CC:
blink-reviews, blink-reviews-paint_chromium.org, dshwang, slimming-paint-reviews_chromium.org
Target Ref:
refs/heads/master
Project:
blink
Visibility:
Public.

Description

Remove a Mac control clip that does nothing. The clip does not apply to the SkBitLocker and hence has no effect. The graphics context state saver is unnecessary given the local graphcis context. R=pdr,chrishtr BUG=487281 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=197779

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -5 lines) Patch
M Source/core/paint/ThemePainterMac.mm View 1 chunk +0 lines, -5 lines 0 comments Download

Messages

Total messages: 9 (3 generated)
Stephen Chennney
After discussing with pdr, this is the plan.
4 years, 10 months ago (2015-06-24 19:10:36 UTC) #2
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1207813002/1
4 years, 10 months ago (2015-06-24 19:10:40 UTC) #3
pdr.
lgtm
4 years, 10 months ago (2015-06-24 19:49:48 UTC) #4
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 10 months ago (2015-06-24 20:20:05 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1207813002/1
4 years, 10 months ago (2015-06-24 21:25:24 UTC) #8
commit-bot: I haz the power
4 years, 10 months ago (2015-06-24 21:28:15 UTC) #9
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=197779

Powered by Google App Engine
This is Rietveld 408576698