Bug 1792156. Fix width/height mixup from bug 1773813. r=emilio a=pascalc
authorTimothy Nikkel <tnikkel@gmail.com>
Fri, 23 Sep 2022 08:56:53 +0000
changeset 707184 56618e9a201be5aa7631aac851d94311f7b8668d
parent 707183 06696602d6c9bbc9ff36abc2f93f6aff7c285a1c
child 707185 8835d26553d3ecf751ff24148e8000cfc9f782e7
push id17113
push userpchevrel@mozilla.com
push dateThu, 29 Sep 2022 11:24:54 +0000
treeherdermozilla-beta@56618e9a201b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersemilio, pascalc
bugs1792156, 1773813
milestone106.0
Bug 1792156. Fix width/height mixup from bug 1773813. r=emilio a=pascalc Differential Revision: https://phabricator.services.mozilla.com/D158008
xpfe/appshell/AppWindow.cpp
--- a/xpfe/appshell/AppWindow.cpp
+++ b/xpfe/appshell/AppWindow.cpp
@@ -2153,17 +2153,17 @@ nsresult AppWindow::GetPrimaryContentShe
 
   // We want to return CSS pixels. First, we get device pixels
   // from the content area...
   // And then get the device pixel scaling factor. Dividing device
   // pixels by this scaling factor gives us CSS pixels.
   CSSIntSize size = RoundedToInt(
       shellWindow->GetSize() / shellWindow->UnscaledDevicePixelsPerCSSPixel());
   *aWidth = size.width;
-  *aHeight = size.width;
+  *aHeight = size.height;
   return NS_OK;
 }
 
 NS_IMETHODIMP
 AppWindow::SetPrimaryContentSize(int32_t aWidth, int32_t aHeight) {
   if (mPrimaryBrowserParent) {
     return SetPrimaryRemoteTabSize(aWidth, aHeight);
   } else if (mPrimaryContentShell) {