From cf047bfab9f8effe61200cbb54587bac11a29216 Mon Sep 17 00:00:00 2001
From: jahorton <jahorton@sil.org>
Date: Fri, 27 Mar 2020 09:10:55 +0700
Subject: [PATCH] change(web/engine): addressing PR review

---
 web/source/text/engineDeviceSpec.ts | 8 ++++----
 web/source/text/processor.ts        | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/web/source/text/engineDeviceSpec.ts b/web/source/text/engineDeviceSpec.ts
index 6ce11b77dd..1117546e7e 100644
--- a/web/source/text/engineDeviceSpec.ts
+++ b/web/source/text/engineDeviceSpec.ts
@@ -43,7 +43,7 @@ namespace com.keyman.text {
         case Browser.Native:
         case Browser.Opera:
         case Browser.Safari:
-          this.browser = browser as Browser;
+          this.browser = browser.toLowerCase() as Browser;
           break;
         default:
           this.browser = Browser.Other;
@@ -53,10 +53,10 @@ namespace com.keyman.text {
         case FormFactor.Desktop:
         case FormFactor.Phone:
         case FormFactor.Tablet:
-          this.formFactor = formFactor as FormFactor;
+          this.formFactor = formFactor.toLowerCase() as FormFactor;
           break;
         default:
-          throw ("Invalid form factor specified for device!");
+          throw ("Invalid form factor specified for device: " + formFactor);
       }
 
       switch(OS.toLowerCase()) {
@@ -65,7 +65,7 @@ namespace com.keyman.text {
         case OperatingSystem.Linux.toLowerCase():
         case OperatingSystem.Android.toLowerCase():
         case OperatingSystem.iOS.toLowerCase():
-          this.OS = OS as OperatingSystem;
+          this.OS = OS.toLowerCase() as OperatingSystem;
           break;
         default:
           this.OS = OperatingSystem.Other;
diff --git a/web/source/text/processor.ts b/web/source/text/processor.ts
index cfbdeb8974..84cf7b70db 100644
--- a/web/source/text/processor.ts
+++ b/web/source/text/processor.ts
@@ -10,7 +10,7 @@
 /// <reference path="../keyboards/keyboard.ts" />
 // Defines built-in keymapping.
 /// <reference path="keyMapping.ts" />
-// Defines a headless-compatible 'Device' analogue for use in keyEvent processing
+// Defines a core-compatible 'Device' analogue for use in keyEvent processing
 /// <reference path="engineDeviceSpec.ts" />
 
 namespace com.keyman.text {