options: temporarily fix text rendering bug in overlays.

options: temporarily fix text rendering bug in overlays.

R=reveman@chromium.org
BUG=372765
TEST=visual

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271580

[reveman, 2014-05-19 21:50:25]

https://codereview.chromium.org/295713004/diff/20001/ui/webui/resources/css/o...
File ui/webui/resources/css/overlay.css (right):

https://codereview.chromium.org/295713004/diff/20001/ui/webui/resources/css/o...
ui/webui/resources/css/overlay.css:33: -webkit-perspective: 1px;
hm, I think this needs to be on the container. does it work if you move this to
.overlay?

[Dan Beam, 2014-05-19 21:59:22]

https://codereview.chromium.org/295713004/diff/20001/ui/webui/resources/css/o...
File ui/webui/resources/css/overlay.css (right):

https://codereview.chromium.org/295713004/diff/20001/ui/webui/resources/css/o...
ui/webui/resources/css/overlay.css:33: -webkit-perspective: 1px;
On 2014/05/19 21:50:25, reveman wrote:
> hm, I think this needs to be on the container. does it work if you move this
to
> .overlay?

Done. (that works!)

[reveman, 2014-05-19 22:25:31]

lgtm

https://codereview.chromium.org/295713004/diff/40001/ui/webui/resources/css/o...
File ui/webui/resources/css/overlay.css (right):

https://codereview.chromium.org/295713004/diff/40001/ui/webui/resources/css/o...
ui/webui/resources/css/overlay.css:10: -webkit-perspective: 1px;
nit: maybe worth adding a TODO here for removing this when we have a better
solution. I just filed crbug.com/374970 in case you want to refer to that.

[Dan Beam, 2014-05-19 22:37:52]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2014-05-19 22:37:57]

https://codereview.chromium.org/295713004/diff/40001/ui/webui/resources/css/o...
File ui/webui/resources/css/overlay.css (right):

https://codereview.chromium.org/295713004/diff/40001/ui/webui/resources/css/o...
ui/webui/resources/css/overlay.css:10: -webkit-perspective: 1px;
On 2014/05/19 22:25:32, reveman wrote:
> nit: maybe worth adding a TODO here for removing this when we have a better
> solution. I just filed crbug.com/374970 in case you want to refer to that.

Done.

[commit-bot: I haz the power, 2014-05-19 22:38:32]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dbeam@chromium.org/295713004/80001

[commit-bot: I haz the power, 2014-05-20 02:03:22]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-05-20 06:16:03]

Change committed as 271580