ScreenOrientationController to start observing even without an internal display.

ScreenOrientationController to start observing even without an internal display.

Ozone loads display configurations asynchronously. Due to this the internal
display is not always known when Maximize Mode is triggered.

Update ScreenOrientationController to become an observer even when there is no
internal display ready yet. That way, once it becomes ready, it is able to
process accelerometer events.

TEST=ScreenOrientationControllerTest.InternalDisplayNotAvailableAtStartup
BUG=chrome-os-partner:38796
Committed: https://crrev.com/31fed68a524d49f99cebf02dca6ad49019e9900c
Cr-Commit-Position: refs/heads/master@{#325298}

[jonross, 2015-04-10 14:59:54]

Patchset #1 (id:1) has been deleted

[jonross, 2015-04-10 15:01:21]

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

[jonross, 2015-04-10 15:01:22]

Hi Oshima,

Could you provide an owners review for this change?

I've updated ScreenOrientationController to account for delays in knowing what
the internal display is.

Thanks

[oshima, 2015-04-10 20:16:09]

I updated the bug. Please let me know if you still want to proceed.

[jonross, 2015-04-13 14:13:52]

On 2015/04/10 20:16:09, oshima wrote:
> I updated the bug. Please let me know if you still want to proceed.

I read your update. I agree that is the root cause of issues such as this.

I would still like to proceed with this review. I think that it would be good to
have this code more resilient to these issues.

[oshima, 2015-04-15 18:47:47]

lgtm

https://codereview.chromium.org/1080553002/diff/20001/ash/content/display/scr...
File ash/content/display/screen_orientation_controller_chromeos.cc (right):

https://codereview.chromium.org/1080553002/diff/20001/ash/content/display/scr...
ash/content/display/screen_orientation_controller_chromeos.cc:217: // Mode has
started. Always start observing.
Can you add reference to EDID bug?

[jonross, 2015-04-15 20:11:47]

The CQ bit was checked by jonross@chromium.org

[jonross, 2015-04-15 20:11:48]

The patchset sent to the CQ was uploaded after l-g-t-m from oshima@chromium.org
Link to the patchset: https://codereview.chromium.org/1080553002/#ps40001
(title: " ")

[commit-bot: I haz the power, 2015-04-15 20:12:06]

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

[commit-bot: I haz the power, 2015-04-15 20:51:25]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-04-15 20:53:20]

Patchset 2 (id:??) landed as
https://crrev.com/31fed68a524d49f99cebf02dca6ad49019e9900c
Cr-Commit-Position: refs/heads/master@{#325298}