Preserve the native mode when parsing external display modes

Preserve the native mode when parsing external display modes

If the native mode is interlaced it can be replaced by a non-interlaced
mode (which will not show as native).

BUG=none

Committed: https://crrev.com/28b5396f75570b021617b2f2c626a6116d322c7e
Cr-Commit-Position: refs/heads/master@{#299114}

[dnicoara, 2014-10-08 17:59:12]

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

[dnicoara, 2014-10-08 17:59:13]

PTAL

[oshima, 2014-10-08 18:39:29]

oshima@chromium.org changed reviewers:
+ mukai@chromium.org

[oshima, 2014-10-08 18:39:30]

-> mukai

[dnicoara, 2014-10-08 19:13:38]

Just as a bit more context, the native mode is used to determine the native
display size when adjusting the touch transform. It is also used by
DisplayManager as a fallback when getting the active mode.

[Jun Mukai, 2014-10-08 19:41:53]

https://codereview.chromium.org/635403002/diff/20001/ash/display/display_chan...
File ash/display/display_change_observer_chromeos.cc (right):

https://codereview.chromium.org/635403002/diff/20001/ash/display/display_chan...
ash/display/display_change_observer_chromeos.cc:149:
display_mode_list.push_back(DisplayMode(native_mode.size(),
why not simply pushing |native_mode| here?

[dnicoara, 2014-10-08 19:50:43]

https://codereview.chromium.org/635403002/diff/20001/ash/display/display_chan...
File ash/display/display_change_observer_chromeos.cc (right):

https://codereview.chromium.org/635403002/diff/20001/ash/display/display_chan...
ash/display/display_change_observer_chromeos.cc:149:
display_mode_list.push_back(DisplayMode(native_mode.size(),
On 2014/10/08 19:41:53, Jun Mukai wrote:
> why not simply pushing |native_mode| here?

Arr, right, sorry, completely missed that it was there.

Done.

[Jun Mukai, 2014-10-08 20:03:56]

lgtm

[dnicoara, 2014-10-10 13:56:11]

On 2014/10/08 20:03:56, Jun Mukai wrote:
> lgtm

Thanks, Oshima-san whould you mind taking a look as well. I still need an OWNERS
review for ash/touch/*.

[oshima, 2014-10-10 14:16:18]

lgtm

[dnicoara, 2014-10-10 14:33:03]

The CQ bit was checked by dnicoara@chromium.org

[commit-bot: I haz the power, 2014-10-10 14:34:14]

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

[commit-bot: I haz the power, 2014-10-10 15:01:38]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2014-10-10 15:02:17]

Patchset 3 (id:??) landed as
https://crrev.com/28b5396f75570b021617b2f2c626a6116d322c7e
Cr-Commit-Position: refs/heads/master@{#299114}