Enabled detection of different refresh rates per monitor on Windows

Enabled detection of different refresh rates per monitor on Windows

BUG=418684
Committed: https://crrev.com/47bcf86d02eea44e340acd1bdb1ca2a9b1c3942f
Cr-Commit-Position: refs/heads/master@{#297277}

[bajones, 2014-09-29 18:46:29]

bajones@chromium.org changed reviewers:
+ brianderson@chromium.org, jbauman@chromium.org, kbr@chromium.org

[jbauman, 2014-09-29 19:37:07]

lgtm with a couple of nits.

https://codereview.chromium.org/615613002/diff/1/ui/gl/gl_surface_win.cc
File ui/gl/gl_surface_win.cc (right):

https://codereview.chromium.org/615613002/diff/1/ui/gl/gl_surface_win.cc#newc...
ui/gl/gl_surface_win.cc:119: if (result != 0) {
if (result) {

https://codereview.chromium.org/615613002/diff/1/ui/gl/gl_surface_win.cc#newc...
ui/gl/gl_surface_win.cc:123: if (result != 0 && display_info.dmDisplayFrequency
> 0) {
if (result && display_info.dmDisplayFrequency > 1) {

(apparently dmDisplayFrequency may be 0 or 1 in weird circumstances, and we
should ignore them)

[bajones, 2014-09-29 20:17:42]

Thanks for the feedback, addressed both of those.

kbr@: Can I get an owner's LGTM?

[Ken Russell (switch to Gerrit), 2014-09-29 20:33:49]

lgtm

[bajones, 2014-09-29 20:38:15]

The CQ bit was checked by bajones@chromium.org

[commit-bot: I haz the power, 2014-09-29 20:39:22]

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

[commit-bot: I haz the power, 2014-09-29 21:48:14]

Committed patchset #2 (id:20001) as 2e59fb4a4017ff558d45e663c86d955e9530c186

[commit-bot: I haz the power, 2014-09-29 21:51:37]

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