[Ozone-DRI] Fix panel fitting lookup

ui/ozone/platform/dri/display_snapshot_dri.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/ozone/platform/dri/display_snapshot_dri.h"
 
 #include <stdint.h>
 #include <stdlib.h>
 #include <xf86drmMode.h>
 
@@ skipping 25 matching lines @@
     case DRM_MODE_CONNECTOR_HDMIA:
     case DRM_MODE_CONNECTOR_HDMIB:
       return DISPLAY_CONNECTION_TYPE_HDMI;
     default:
       return DISPLAY_CONNECTION_TYPE_UNKNOWN;
   }
 }
 
 bool IsAspectPreserving(DriWrapper* drm, drmModeConnector* connector) {
   ScopedDrmPropertyPtr property(drm->GetProperty(connector, "scaling mode"));
-  if (property) {
+  if (!property)
+    return false;
+
+  for (int i = 0; i < connector->count_props; ++i) {
+    if (connector->props[i] != property->prop_id)

[alexst (slow to review), 2014/10/10 15:00:08]

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]

Done.

+      continue;
+
     for (int j = 0; j < property->count_enums; ++j) {
-      if (property->enums[j].value ==
-              connector->prop_values[property->prop_id] &&
+      if (property->enums[j].value == connector->prop_values[i] &&
           strcmp(property->enums[j].name, "Full aspect") == 0)
         return true;
     }
   }
 
   return false;
 }
 
 }  // namespace
 
@@ skipping 58 matching lines @@
   return base::StringPrintf(
       "[type=%d, connector=%" PRIu32 ", crtc=%" PRIu32 ", mode=%s, dim=%s]",
       type_,
       connector_,
       crtc_,
       current_mode_ ? current_mode_->ToString().c_str() : "NULL",
       physical_size_.ToString().c_str());
 }
 
 }  // namespace ui