Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(90)

Issue 11316172: Always disable "content has shadow". (Closed)

Created:
8 years ago by Avi (use Gerrit)
Modified:
8 years ago
Reviewers:
Nico
CC:
chromium-reviews, sail+watch_chromium.org
Visibility:
Public.

Description

Always disable "content has shadow". BUG=161603 TEST=as in bug Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=169755

Patch Set 1 #

Patch Set 2 : updated comment #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -3 lines) Patch
M ui/base/cocoa/underlay_opengl_hosting_window.mm View 1 1 chunk +10 lines, -3 lines 2 comments Download

Messages

Total messages: 7 (0 generated)
Avi (use Gerrit)
I need to verify this on 10.6 and 10.7 but I believe this is the ...
8 years ago (2012-11-26 19:24:20 UTC) #1
Nico
https://codereview.chromium.org/11316172/diff/1002/ui/base/cocoa/underlay_opengl_hosting_window.mm File ui/base/cocoa/underlay_opengl_hosting_window.mm (right): https://codereview.chromium.org/11316172/diff/1002/ui/base/cocoa/underlay_opengl_hosting_window.mm#newcode117 ui/base/cocoa/underlay_opengl_hosting_window.mm:117: [self _setContentHasShadow:NO]; Hum, I would've expected this to force ...
8 years ago (2012-11-27 00:28:12 UTC) #2
Avi (use Gerrit)
https://codereview.chromium.org/11316172/diff/1002/ui/base/cocoa/underlay_opengl_hosting_window.mm File ui/base/cocoa/underlay_opengl_hosting_window.mm (right): https://codereview.chromium.org/11316172/diff/1002/ui/base/cocoa/underlay_opengl_hosting_window.mm#newcode117 ui/base/cocoa/underlay_opengl_hosting_window.mm:117: [self _setContentHasShadow:NO]; That's the magic of "content has shadow ...
8 years ago (2012-11-27 14:51:52 UTC) #3
Avi (use Gerrit)
Verified on 10.8, 10.7 HiDPI. Attempting to run a binary built on my 10.8 machine ...
8 years ago (2012-11-27 15:36:59 UTC) #4
Avi (use Gerrit)
Verified on 10.6. Landing.
8 years ago (2012-11-27 18:39:47 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/avi@chromium.org/11316172/1002
8 years ago (2012-11-27 18:40:31 UTC) #6
commit-bot: I haz the power
8 years ago (2012-11-27 21:18:09 UTC) #7
Message was sent while issue was closed.
Change committed as 169755

Powered by Google App Engine
This is Rietveld 408576698