[Ozone-DRI] Fix panel fitting lookup

[Ozone-DRI] Fix panel fitting lookup

BUG=none
NOTRY=true

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

[dnicoara, 2014-10-10 14:38:53]

dnicoara@chromium.org changed reviewers:
+ alexst@chromium.org

[dnicoara, 2014-10-10 14:38:54]

PTAL

[alexst (slow to review), 2014-10-10 15:00:09]

lgtm

https://codereview.chromium.org/642333002/diff/1/ui/ozone/platform/dri/displa...
File ui/ozone/platform/dri/display_snapshot_dri.cc (right):

https://codereview.chromium.org/642333002/diff/1/ui/ozone/platform/dri/displa...
ui/ozone/platform/dri/display_snapshot_dri.cc:50: if (connector->props[i] !=
property->prop_id)
I found my mind wondering looking at i and j here trying to figure out which is
which. Do you think it would add clarity to call them enums_i and props_i or
some such?

[dnicoara, 2014-10-10 15:07:14]

https://codereview.chromium.org/642333002/diff/1/ui/ozone/platform/dri/displa...
File ui/ozone/platform/dri/display_snapshot_dri.cc (right):

https://codereview.chromium.org/642333002/diff/1/ui/ozone/platform/dri/displa...
ui/ozone/platform/dri/display_snapshot_dri.cc:50: if (connector->props[i] !=
property->prop_id)
On 2014/10/10 15:00:08, alexst wrote:
> I found my mind wondering looking at i and j here trying to figure out which
is
> which. Do you think it would add clarity to call them enums_i and props_i or
> some such?

Done.

[alexst (slow to review), 2014-10-10 15:28:16]

still lgtm, thank you.

[dnicoara, 2014-10-10 16:47:35]

The CQ bit was checked by dnicoara@chromium.org

[commit-bot: I haz the power, 2014-10-10 16:48:41]

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

[commit-bot: I haz the power, 2014-10-10 16:49:20]

Committed patchset #2 (id:80001)

[commit-bot: I haz the power, 2014-10-10 16:50:08]

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