Fix issue introduced by moving queryWindowSize to update.

Author: Yuncong Zhang

Runtime/editor/editor_window.cs

@@ -181,6 +181,12 @@ public void onViewMetricsChanged() {
         public void OnEnable() {
             this._devicePixelRatio = this.queryDevicePixelRatio();
             this._antiAliasing = this.queryAntiAliasing();
+            var size = this.queryWindowSize();
+            this._lastWindowWidth = size.x;
+            this._lastWindowHeight = size.y;
+            this._physicalSize = new Size(
+                this._lastWindowWidth * this._devicePixelRatio,
+                this._lastWindowHeight * this._devicePixelRatio);
 
             this.updateSafeArea();
             D.assert(this._surface == null);
@@ -437,7 +443,7 @@ void _updateScrollInput(float deltaTime) {
         }
 
         public void Update() {
-            if (this._physicalSize == null) {
+            if (this._physicalSize == null || this._physicalSize.isEmpty) {
                 var size = this.queryWindowSize();
                 this._lastWindowWidth = size.x;
                 this._lastWindowHeight = size.y;

Runtime/engine/UIWidgetsPanel.cs

@@ -65,18 +65,22 @@ public override GUIContent titleContent {
         protected override float queryDevicePixelRatio() {
             return this._uiWidgetsPanel.devicePixelRatio;
         }
-        
+
         protected override int queryAntiAliasing() {
             return this._uiWidgetsPanel.antiAliasing;
         }
 
         protected override Vector2 queryWindowSize() {
             var rect = this._uiWidgetsPanel.rectTransform.rect;
-            var size = new Vector2(rect.width, rect.height) *
-                       this._uiWidgetsPanel.canvas.scaleFactor / this._uiWidgetsPanel.devicePixelRatio;
-            size.x = Mathf.Round(size.x);
-            size.y = Mathf.Round(size.y);
-            return size;
+            if (!ReferenceEquals(this._uiWidgetsPanel.canvas, null)) {
+                var size = new Vector2(rect.width, rect.height) *
+                           this._uiWidgetsPanel.canvas.scaleFactor / this._uiWidgetsPanel.devicePixelRatio;
+                size.x = Mathf.Round(size.x);
+                size.y = Mathf.Round(size.y);
+                return size;
+            }
+
+            return new Vector2(0, 0);
         }
 
         public Offset windowPosToScreenPos(Offset windowPos) {
@@ -134,7 +138,7 @@ protected override void OnEnable() {
 
             this._displayMetrics = DisplayMetricsProvider.provider();
             this._displayMetrics.OnEnable();
-            
+
             this._enteredPointers.Clear();
 
             if (_repaintEvent == null) {
@@ -162,11 +166,9 @@ public float devicePixelRatio {
                     : this._displayMetrics.devicePixelRatio;
             }
         }
-        
+
         public int antiAliasing {
-            get {
-                return this.antiAliasingOverride >= 0 ? this.antiAliasingOverride : QualitySettings.antiAliasing;
-            }
+            get { return this.antiAliasingOverride >= 0 ? this.antiAliasingOverride : QualitySettings.antiAliasing; }
         }
 
         public WindowPadding viewPadding {
@@ -177,17 +179,17 @@ public WindowPadding viewInsets {
             get { return this._displayMetrics.viewInsets; }
         }
 
-        protected override void OnDisable() {	
-            D.assert(this._windowAdapter != null);	
-            this._windowAdapter.OnDisable();	
-            this._windowAdapter = null;	
-            base.OnDisable();	
+        protected override void OnDisable() {
+            D.assert(this._windowAdapter != null);
+            this._windowAdapter.OnDisable();
+            this._windowAdapter = null;
+            base.OnDisable();
         }
-        
+
         protected virtual Widget createWidget() {
             return null;
         }
-        
+
         public void recreateWidget() {
             Widget root;
             using (this._windowAdapter.getScope()) {
@@ -205,12 +207,10 @@ internal void applyRenderTexture(Rect screenRect, Texture texture, Material mat)
         protected virtual void Update() {
             this._displayMetrics.Update();
             UIWidgetsMessageManager.ensureUIWidgetsMessageManagerIfNeeded();
-            
+
 #if UNITY_ANDROID
             if (Input.GetKeyDown(KeyCode.Escape)) {
-                this._windowAdapter.withBinding(() => {
-                    WidgetsBinding.instance.handlePopRoute();
-                });
+                this._windowAdapter.withBinding(() => { WidgetsBinding.instance.handlePopRoute(); });
             }
 #endif
 
@@ -277,6 +277,7 @@ int getMouseButtonDown() {
                     break;
                 }
             }
+
             return InputUtils.getMouseButtonKey(defaultKey);
         }