Test orientation controller should update current orientation when it is not allowed by lock.

Test orientation controller should update current orientation when it is not allowed by lock.

BUG=162827
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=272020

[Inactive, 2014-05-20 18:20:20]

https://codereview.chromium.org/296813002/diff/1/content/renderer/screen_orie...
File content/renderer/screen_orientation/mock_screen_orientation_controller.cc
(right):

https://codereview.chromium.org/296813002/diff/1/content/renderer/screen_orie...
content/renderer/screen_orientation/mock_screen_orientation_controller.cc:34: if
(!IsOrientationAllowedByCurrentLock(current_orientation_))
Ah, my bad. I broke that in when refactoring my CL. Thank you from fixing it.
non-OWNER lgtm.

[mlamouri (slow - plz ping), 2014-05-20 19:59:22]

On 2014/05/20 18:20:20, Chris Dumez wrote:
>
https://codereview.chromium.org/296813002/diff/1/content/renderer/screen_orie...
> File content/renderer/screen_orientation/mock_screen_orientation_controller.cc
> (right):
> 
>
https://codereview.chromium.org/296813002/diff/1/content/renderer/screen_orie...
> content/renderer/screen_orientation/mock_screen_orientation_controller.cc:34:
if
> (!IsOrientationAllowedByCurrentLock(current_orientation_))
> Ah, my bad. I broke that in when refactoring my CL. Thank you from fixing it.
> non-OWNER lgtm.

LGTM but it seems that contrary to me believe, I do not own that directory :(

[jochen (gone - plz use gerrit), 2014-05-21 08:48:54]

lgtm

[mlamouri (slow - plz ping), 2014-05-21 09:01:10]

The CQ bit was checked by mlamouri@chromium.org

[commit-bot: I haz the power, 2014-05-21 09:02:39]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sl.ostapenko@samsung.com/296813002/1

[commit-bot: I haz the power, 2014-05-21 23:51:10]

Change committed as 272020