Remove direct DBus calls from OutputConfigurator

Remove direct DBus calls from OutputConfigurator

Remove DBus signal to power_manager::kUseNewMonitorConfigSignal since it is no
longer needed by the other side (this used to be required but it was only meant
to be temporary and the other side no longer listens for the signal).
Move DBus call to power_manager::kSetIsProjectingMethod into the DBus
PowerManagerClient class which is where the other methods in the IPC interface
live.

BUG=chromium:142039
TEST=Manually tested on Lumpy and verified that adding/removing displays results
in powerd receiving a SetIsProjecting call.


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

[satorux1, 2012-08-14 18:44:53]

http://codereview.chromium.org/10831312/diff/6001/chromeos/display/output_con...
File chromeos/display/output_configurator.cc (left):

http://codereview.chromium.org/10831312/diff/6001/chromeos/display/output_con...
chromeos/display/output_configurator.cc:239: // control.
oh this code was unnecessary?

[disher, 2012-08-14 19:27:31]

https://chromiumcodereview.appspot.com/10831312/diff/6001/chromeos/display/ou...
File chromeos/display/output_configurator.cc (left):

https://chromiumcodereview.appspot.com/10831312/diff/6001/chromeos/display/ou...
chromeos/display/output_configurator.cc:239: // control.
On 2012/08/14 18:44:53, satorux1 wrote:
> oh this code was unnecessary?

Yes, this code was only required while this logic was in the process of
migrating between powerd and Chrome.  To allow it to move between the projects
without introducing a window of broken behaviour, this mechanism was used. 
powerd would use its logic, by default, but Chrome would send this message when
it started to tell powerd that it should disable its logic.  The old logic for
this was removed from powerd on June 6 and the corresponding change was never
made on this end (easily slips through the cracks since it is wrong but
harmless).

[satorux1, 2012-08-14 20:46:52]

LGTM. Thank you for fixing this.

[commit-bot: I haz the power, 2012-08-16 13:43:49]

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

[commit-bot: I haz the power, 2012-08-16 15:07:09]

Change committed as 151894