Add GetActiveScreen to DisplayInfoProvider

This removes an ash dependency from the system.display API in order to move it to extensions/. As a side effect, we got rid of #ifdefs on system_info_api.cc

BUG=392842
Committed: https://crrev.com/567bb9246c5fac1377761ab1b30b0b6db2b78ebf
Cr-Commit-Position: refs/heads/master@{#292373}

[tmpsantos, 2014-08-27 12:12:00]

thiago.santos@intel.com changed reviewers:
+ hongbo.min@intel.com, jamescook@chromium.org

[tmpsantos, 2014-08-27 12:12:00]

Please review. This is needed in order to move this API to extensions/ without
bringing in the ash/ dependency.

[tmpsantos, 2014-08-27 12:12:17]

Patchset #1 (id:1) has been deleted

[tmpsantos, 2014-08-27 14:33:15]

thiago.santos@intel.com changed reviewers:
+ yoz@chromium.org

[James Cook, 2014-08-27 16:03:15]

LGTM but this needs a better CL description before landing. You should indicate
that it is related to extensions and the display API.

[Yoyo Zhou, 2014-08-27 19:12:07]

On 2014/08/27 16:03:15, James Cook wrote:
> LGTM but this needs a better CL description before landing. You should
indicate
> that it is related to extensions and the display API.

LGTM, what James said. It should be along the lines of:

Add GetActiveScreen to DisplayInfoProvider...

This removes an ash dependency from the system.display API in order to move it
to extensions/.

[tmpsantos, 2014-08-28 12:38:05]

On 2014/08/27 19:12:07, Yoyo Zhou wrote:
> On 2014/08/27 16:03:15, James Cook wrote:
> > LGTM but this needs a better CL description before landing. You should
> indicate
> > that it is related to extensions and the display API.
> 
> LGTM, what James said. It should be along the lines of:
> 
> Add GetActiveScreen to DisplayInfoProvider...
> 
> This removes an ash dependency from the system.display API in order to move it
> to extensions/.

Thanks for reviewing and yes, this was indeed a sloppy changelog message.

[tmpsantos, 2014-08-28 12:38:16]

The CQ bit was checked by thiago.santos@intel.com

[commit-bot: I haz the power, 2014-08-28 12:39:00]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/thiago.santos@intel.com/507783003/20001

[commit-bot: I haz the power, 2014-08-28 13:30:11]

Committed patchset #1 (id:20001) as 88d628a042b25f3c0890d6ffdb63481ed64b9f86

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

Patchset 1 (id:??) landed as
https://crrev.com/567bb9246c5fac1377761ab1b30b0b6db2b78ebf
Cr-Commit-Position: refs/heads/master@{#292373}