Fixes bug in HTTP Service rendering

I think with the recent change to the method of hiding text by using a
dedicated blank font, the initial page load and parsing (upon which the
HTTP Service very much depends), browser window resizes were causing a
delay in the time taken for text to be properly parsed, resulting in
mostly empty raw text pages.

The fix was to ensure that the BlankMono.ttf font was loaded with the
initial page load, by using the webextensions injected style.css file.

webext/assets/styles.css

@@ -20,7 +20,8 @@ html * {
 }
 
 .browsh-hide-text,
-.browsh-hide-text * {
+.browsh-hide-text *,
+html * {
   font-family: 'BlankMono' !important;
 }
 

webext/src/background/dimensions.js

@@ -24,6 +24,10 @@ export default class extends utils.mixins(CommonMixin) {
     if (this.char.width != incoming.width ||
         this.char.height != incoming.height) {
       this.char = _.clone(incoming);
+      this.log(
+        `Requesting browser resize for new char dimensions: ` +
+        `${this.char.width}x${this.char.height}`
+      );
       this.resizeBrowserWindow();
     }
   }
@@ -57,7 +61,6 @@ export default class extends utils.mixins(CommonMixin) {
 
   _sendWindowResizeRequest(active_window, width, height) {
     const tag = 'Resizing browser window';
-    this.log(tag, active_window, width, height);
     const updating = browser.windows.update(
       active_window.id,
       {
@@ -67,8 +70,8 @@ export default class extends utils.mixins(CommonMixin) {
       }
     );
     updating.then(
-      info => this.log(tag, info),
-      error => this.log(tag, error)
+      info => this.log(`${tag} successful (${info.width}x${info.height})`),
+      error => this.log(tag + " error: ", error)
     );
   }
 }

webext/src/background/tty_commands_mixin.js

@@ -49,6 +49,10 @@ export default (MixinBase) => class extends MixinBase {
         `/tty_size,${this.dimensions.tty.width},${this.dimensions.tty.height}`
       )
     }
+    this.log(
+      `Requesting browser resize for new TTY dimensions: ` +
+      `${width}x${height}`
+    );
     this.dimensions.resizeBrowserWindow();
   }
 

webext/src/dom/dimensions.js

@@ -30,9 +30,6 @@ export default class extends utils.mixins(CommonMixin) {
       x_last_big_frame: 0,
       y_last_big_frame: 0
     }
-    if (document.body) {
-      this.update();
-    }
   }
 
   update() {

webext/src/dom/graphics_builder.js

@@ -126,7 +126,6 @@ export default class extends utils.mixins(CommonMixin) {
   }
 
   _getScreenshot() {
-    this.dimensions.update()
     return this._getPixelData();
   }
 

webext/src/dom/manager.js

@@ -22,7 +22,6 @@ export default class extends utils.mixins(CommonMixin, CommandsMixin) {
 
   _postCommsConstructor() {
     this.dimensions.channel = this.channel;
-    this.dimensions.update();
     this.graphics_builder = new GraphicsBuilder(this.channel, this.dimensions);
     this.text_builder = new TextBuilder(
       this.channel,

webext/src/dom/text_builder.js

@@ -27,8 +27,15 @@ export default class extends utils.mixins(CommonMixin) {
   }
 
   sendRawText() {
-    this.buildFormattedText();
-    this._sendRawText();
+    // TODO:
+    //   The presence of the `getScreenshotWithText()` and `setTimeout()` calls are a hack
+    //   that I am unable to understand the reasoning for - unfortunately they came about
+    //   by trial and error :( Without them the the returned raw text is largely empty.
+    this.graphics_builder.getScreenshotWithText();
+    setTimeout(() => {
+      this.buildFormattedText();
+      this._sendRawText();
+    }, 200);
   }
 
   buildFormattedText() {