Chromium Code Reviews| Index: content/renderer/render_view_impl.cc |
| diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc |
| index f46da97c6b175f12de695861fec9990ec0943ce7..925ad94993c98215aab16404a7753fa98fd73691 100644 |
| --- a/content/renderer/render_view_impl.cc |
| +++ b/content/renderer/render_view_impl.cc |
| @@ -4007,6 +4007,33 @@ void RenderViewImpl::SetDeviceScaleFactorForTesting(float factor) { |
| OnResize(params); |
| } |
| +void RenderViewImpl::SetScreenOrientationForTesting( |
| + const blink::WebScreenOrientationType& orientation) { |
| + ViewMsg_Resize_Params params; |
| + params.screen_info = screen_info_; |
| + params.screen_info.orientationType = orientation; |
| + switch (orientation) { |
| + case blink::WebScreenOrientationLandscapePrimary: |
| + params.screen_info.orientationAngle = 90; |
|
mlamouri (slow - plz ping)
2014/05/15 07:06:59
Please, do not do that kind of assumptions. It see
Inactive
2014/05/15 13:01:47
I agree we generally shouldn't make this kind of a
|
| + break; |
| + case blink::WebScreenOrientationLandscapeSecondary: |
| + params.screen_info.orientationAngle = -90; |
| + break; |
| + case blink::WebScreenOrientationPortraitSecondary: |
| + params.screen_info.orientationAngle = 180; |
| + break; |
| + default: |
| + params.screen_info.orientationAngle = 0; |
| + } |
| + params.new_size = size(); |
| + params.physical_backing_size = gfx::ToCeiledSize( |
| + gfx::ScaleSize(size(), params.screen_info.deviceScaleFactor)); |
| + params.overdraw_bottom_height = 0.f; |
| + params.resizer_rect = WebRect(); |
| + params.is_fullscreen = is_fullscreen(); |
| + OnResize(params); |
| +} |
| + |
| void RenderViewImpl::SetDeviceColorProfileForTesting( |
| const std::vector<char>& color_profile) { |
| // TODO(noel): Add RenderViewImpl::SetDeviceColorProfile(color_profile). |