Detect output state at startup to avoid changes

Detect output state at startup to avoid changes

Resetting the state of the outputs causes a black flash on the screen so avoid
doing this in the cases where the outputs were already configured such that they
were in a state we can recognize.

BUG=chromium:130943
TEST=Manually tested on lumpy with extra instrumentation to ensure that logging
in as guest does not change the CRTCs.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=141209

[Daniel Erat, 2012-06-07 19:17:40]

LGTM

https://chromiumcodereview.appspot.com/10532053/diff/1/chromeos/monitor/outpu...
File chromeos/monitor/output_configurator.cc (right):

https://chromiumcodereview.appspot.com/10532053/diff/1/chromeos/monitor/outpu...
chromeos/monitor/output_configurator.cc:526: if (primary_mode ==
output_cache_[primary_output_index_].ideal_mode) {
nit: remove curly braces

[Daniel Erat, 2012-06-07 19:19:38]

Side comment: make sure you explicitly send mail about new issues on the code
review site ('m' shortcut) when you're ready for a review; otherwise the
reviewer won't hear about it.

[disher, 2012-06-07 19:40:23]

Thanks.  I was going to ping you on IM once I had a moment.

I will put this in CQ once I run some trybots over this, just to be sure there
is nothing trivially wrong in my environment.

https://chromiumcodereview.appspot.com/10532053/diff/1/chromeos/monitor/outpu...
File chromeos/monitor/output_configurator.cc (right):

https://chromiumcodereview.appspot.com/10532053/diff/1/chromeos/monitor/outpu...
chromeos/monitor/output_configurator.cc:526: if (primary_mode ==
output_cache_[primary_output_index_].ideal_mode) {
On 2012/06/07 19:17:40, Daniel Erat wrote:
> nit: remove curly braces

Done.

[commit-bot: I haz the power, 2012-06-08 13:11:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/disher@chromium.org/10532053/2002

[commit-bot: I haz the power, 2012-06-08 15:27:28]

Change committed as 141209