chromeos: output_configurator: various style fixes

chromeos: output_configurator: various style fixes

- Define member functions of class OutputConfigurator in the order
  they were declared.

- Made struct CachedOutputDescription to be private member of class
  OutputConfigurator.  No need for it to be exposed publicly.

- Renamed "enum State" to "enum OutputState" to be more specific.

- In places where a large if block extend to the end of the function,
  use an early return to reduce indentation.

BUG=chromium-os:32857
TEST=build successfully

Change-Id: Id39dff3b38d16b2496df0df288a332fe29b2e37b
Signed-off-by: Simon Que <sque@chromium.org>;


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

[Daniel Erat, 2012-07-30 21:31:34]

lgtm

[commit-bot: I haz the power, 2012-07-30 21:33:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sque@chromium.org/10824071/1

[commit-bot: I haz the power, 2012-07-30 21:34:01]

Presubmit check for 10824071-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome/browser/ui/webui/options2

[Simon Que, 2012-07-30 21:38:02]

Requesting review from OWNERS of  chrome/browser/ui/webui/options2.

[csilv, 2012-07-30 21:48:28]

chrome/browser/ui/webui/options2  LGTM

[commit-bot: I haz the power, 2012-07-30 21:50:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sque@chromium.org/10824071/1

[commit-bot: I haz the power, 2012-07-30 23:18:52]

Change committed as 149072