[6/8][Ozone-Drm] Fix lookup for DRM property values

[Ozone-Drm] Fix lookup for DRM property values

Property IDs are not indexes, so we need to iterate over the properties
to find the correct one.

BUG=442317
Committed: https://crrev.com/db26cbf00e794d7de3c1b98a505b220dd99d1a4b
Cr-Commit-Position: refs/heads/master@{#325238}

[dnicoara, 2015-03-26 17:47:46]

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

[spang, 2015-03-26 17:49:02]

please CC me on the whole series next time

[spang, 2015-03-26 17:56:29]

https://codereview.chromium.org/1035523004/diff/1/ui/ozone/platform/drm/gpu/d...
File ui/ozone/platform/drm/gpu/drm_gpu_display_manager.cc (right):

https://codereview.chromium.org/1035523004/diff/1/ui/ozone/platform/drm/gpu/d...
ui/ozone/platform/drm/gpu/drm_gpu_display_manager.cc:351: for (int prop_idx = 0;
prop_idx < connector->count_props; ++prop_idx) {
Please factor out the property search.

https://codereview.chromium.org/1035523004/diff/1/ui/ozone/platform/drm/gpu/d...
ui/ozone/platform/drm/gpu/drm_gpu_display_manager.cc:355: for (int enum_idx = 0;
enum_idx < hdcp_property->count_enums; ++enum_idx) {
It'll be a lot clearer if you do.

drm_mode_property_enum* property_enum = hdcp_property->enums[enum_idx];

[dnicoara, 2015-04-14 21:57:17]

https://codereview.chromium.org/1035523004/diff/1/ui/ozone/platform/drm/gpu/d...
File ui/ozone/platform/drm/gpu/drm_gpu_display_manager.cc (right):

https://codereview.chromium.org/1035523004/diff/1/ui/ozone/platform/drm/gpu/d...
ui/ozone/platform/drm/gpu/drm_gpu_display_manager.cc:351: for (int prop_idx = 0;
prop_idx < connector->count_props; ++prop_idx) {
On 2015/03/26 17:56:29, spang wrote:
> Please factor out the property search.

Done.

https://codereview.chromium.org/1035523004/diff/1/ui/ozone/platform/drm/gpu/d...
ui/ozone/platform/drm/gpu/drm_gpu_display_manager.cc:355: for (int enum_idx = 0;
enum_idx < hdcp_property->count_enums; ++enum_idx) {
On 2015/03/26 17:56:29, spang wrote:
> It'll be a lot clearer if you do.
> 
> drm_mode_property_enum* property_enum = hdcp_property->enums[enum_idx];

Done.

[dnicoara, 2015-04-14 21:57:29]

On 2015/03/26 17:49:02, spang wrote:
> please CC me on the whole series next time

Sorry, will do.

[spang, 2015-04-14 21:58:48]

lgtm

[dnicoara, 2015-04-15 13:46:31]

The CQ bit was checked by dnicoara@chromium.org

[commit-bot: I haz the power, 2015-04-15 13:47:01]

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

[commit-bot: I haz the power, 2015-04-15 13:49:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-15 13:51:07]

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