Merge pull request #4366 from arekouzounian/master

Fix for #2488. Use element offsetHeight rather than getBoundingRect

src/browser/services/CharSizeService.ts

@@ -77,7 +77,10 @@ class DomMeasureStrategy implements IMeasureStrategy {
     this._measureElement.style.fontSize = `${this._optionsService.rawOptions.fontSize}px`;
 
     // Note that this triggers a synchronous layout
-    const geometry = this._measureElement.getBoundingClientRect();
+    const geometry = {
+      height: Number(this._measureElement.offsetHeight),
+      width: Number(this._measureElement.offsetWidth)
+    };
 
     // If values are 0 then the element is likely currently display:none, in which case we should
     // retain the previous value.