Chromium Code Reviews| Index: content/browser/devtools/protocol/page_handler.cc |
| diff --git a/content/browser/devtools/protocol/page_handler.cc b/content/browser/devtools/protocol/page_handler.cc |
| index 5325a66f8fda5917cdb34b9fd98e7e5792c46221..c1e88169d5d9d4dbf8734971433842f5c782986c 100644 |
| --- a/content/browser/devtools/protocol/page_handler.cc |
| +++ b/content/browser/devtools/protocol/page_handler.cc |
| @@ -397,8 +397,9 @@ void PageHandler::InnerSwapCompositorFrame() { |
| gfx::SizeF viewport_size_dip = gfx::ScaleSize( |
| metadata.scrollable_viewport_size, metadata.page_scale_factor); |
| - gfx::SizeF screen_size_dip = gfx::ScaleSize(view->GetPhysicalBackingSize(), |
| - 1 / metadata.device_scale_factor); |
| + gfx::SizeF screen_size_dip = |
| + gfx::ScaleSize(gfx::SizeF(view->GetPhysicalBackingSize()), |
|
no sievers
2015/09/24 00:07:33
maybe this should have a version of ScaleSize() fo
danakj
2015/09/24 00:36:05
I didn't want to do that because ScaleSize returns
|
| + 1 / metadata.device_scale_factor); |
| blink::WebScreenInfo screen_info; |
| view->GetScreenInfo(&screen_info); |
| @@ -472,8 +473,9 @@ void PageHandler::ScreencastFrameEncoded( |
| if (!view) |
| return; |
| - gfx::SizeF screen_size_dip = gfx::ScaleSize( |
| - view->GetPhysicalBackingSize(), 1 / metadata.device_scale_factor); |
| + gfx::SizeF screen_size_dip = |
| + gfx::ScaleSize(gfx::SizeF(view->GetPhysicalBackingSize()), |
| + 1 / metadata.device_scale_factor); |
| scoped_refptr<ScreencastFrameMetadata> param_metadata = |
| ScreencastFrameMetadata::Create() |
| ->set_page_scale_factor(metadata.page_scale_factor) |