CHANGELOG.md

@@ -4,8 +4,7 @@
 
 ### v3.4.0
 * In-player UI now appears in full-screen even for websites that use top layer
-* Embedded sites now inherit settings of the parent frame by default
+* Embedded sites now inherit settings of the parent frame
-* Added validation to custom aspect ratio entry menu. Corrected parsing of aspect ratios given in the X:Y format, even though aspect ratios should be ideally given as a single number.
 
 ### v6.3.0
 * Added zoom segment to in-player UI and popup. 

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "ultrawidify",
-  "version": "6.3.93",
+  "version": "6.3.9",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ultrawidify",
-  "version": "6.3.93",
+  "version": "6.3.9",
   "description": "Aspect ratio fixer for youtube and other sites, with automatic aspect ratio detection. Supports ultrawide and other ratios.",
   "author": "Tamius Han <tamius.han@gmail.com>",
   "scripts": {

scripts/build-all.sh

@@ -5,27 +5,11 @@
 # any other way probably isn't going to work.
 
 # pre-build steps:
-
-echo "—————————————————————————————————————————————————————";
-echo "      BUILDING ALL ULTRAWIDIFY VERSIONS";
-echo "—————————————————————————————————————————————————————";
-
 mkdir -p ./build/old
 npm run pre-build
-
-echo "::"
-echo "::"
-echo "::"
-echo ":: Pre-build complete. Moving old builds from dist-zip ..."
-
 rm ./dist-zip/uw-amo-source.zip
 mv -f ./dist-zip/*.zip ./build/old
 
-echo "::"
-echo "::"
-echo "::"
-echo ":: dist-zip is clean. Starting builds ..."
-
 # lets force raise ram limit, but the improper way
 # export NODE_OPTIONS=--max_old_space_size=4096
 

src/common/enums/EmbeddedContentSettingsOverridePolicy.enum.ts

@@ -1,7 +0,0 @@
-enum EmbeddedContentSettingsOverridePolicy {
-  Always = 2,
-  UseAsDefault = 1,
-  Never = 0,
-}
-
-export default EmbeddedContentSettingsOverridePolicy;

src/common/interfaces/SettingsInterface.ts

@@ -1,9 +1,7 @@
 import { Action } from '../../../node_modules/vuex/types/index'
-import { AardPollingOptions } from '../../ext/lib/aard/enums/aard-polling-options.enum'
 import AntiGradientMode from '../enums/AntiGradientMode.enum'
 import AspectRatioType from '../enums/AspectRatioType.enum'
 import CropModePersistence from '../enums/CropModePersistence.enum'
-import EmbeddedContentSettingsOverridePolicy from '../enums/EmbeddedContentSettingsOverridePolicy.enum'
 import ExtensionMode from '../enums/ExtensionMode.enum'
 import StretchType from '../enums/StretchType.enum'
 import VideoAlignmentType from '../enums/VideoAlignmentType.enum'
@@ -75,10 +73,6 @@ export interface AardSettings {
     stopTimeout: number;
   },
 
-  polling: {
-    runInBackgroundTabs: AardPollingOptions;
-    runOnSmallVideos: AardPollingOptions;
-  }
 
   disabledReason: string,     // if automatic aspect ratio has been disabled, show reason
   allowedMisaligned: number,  // top and bottom letterbox thickness can differ by this much.
@@ -92,17 +86,13 @@ export interface AardSettings {
     minimumTimeout: number,
     tickrate: number,          // 1 tick every this many milliseconds
   },
-
-  subtitles: {
-    resetIfDetected?: boolean,  // reset if subtitles are detected
-    resumeAfter?: number,       // resume after X ms of no subtitles
-  },
-
   autoDisable: {            // settings for automatically disabling the extension
-    onFirstChange: boolean,
+    maxExecutionTime: number, // if execution time of main autodetect loop exceeds this many milliseconds,
-    ifNotChanged: boolean,
+                            // we disable it.
-    ifNotChangedTimeout: number,
+    consecutiveTimeoutCount: number,  // we only do it if it happens this many consecutive times
-    ifSubtitles: boolean;
+
+    // FOR FUTURE USE
+    consecutiveArResets: number       // if aspect ratio reverts immediately after AR change is applied, we disable everything
   },
   canvasDimensions: {
     blackframeCanvas: {   // smaller than sample canvas, blackframe canvas is used to recon for black frames
@@ -318,8 +308,7 @@ export interface SiteSettingsInterface {
   enableKeyboard: ExtensionEnvironmentSettingsInterface;
   enableUI: ExtensionEnvironmentSettingsInterface;  // Lies! enableUI doesn't use 'theater' property (but uses the other two)
 
-  applyToEmbeddedContent: EmbeddedContentSettingsOverridePolicy; // presumed to be 'Always' if not defined
+  applyToEmbeddedContent?: boolean; // presumed to be 'true' if not defined
-  overrideWhenEmbedded?: boolean;
 
 
   autocreated?: boolean;

src/csui/PlayerOverlay.vue

@@ -463,7 +463,7 @@ export default {
           if (!this.site) {
             this.origin = event.origin;
             this.site = event.origin.split('//')[1];
-            this.siteSettings = this.settings.getSiteSettings({site: this.site, isIframe: event.data.isIframe});
+            this.siteSettings = this.settings.getSiteSettings({site: this.site});
           }
           return this.handleProbe(event.data, event.origin); // handleProbe is defined in UIProbeMixin
         case 'uw-bus-tunnel':

src/csui/src/PlayerUiPanels/AutodetectionSettingsPanel.vue

@@ -9,7 +9,217 @@
         </div>
         <div class="aard-settings-group">
 
+          <!-- the last time i tried to comment out this block, it didn't work properly — v-if="false" it is -->
+          <div v-if="false">
+            <p>
+              <b>Autodetection performance</b>
+            </p>
+            <p>
+              Automatic aspect ratio detection is a resource-hungry feature.
+              This page allows you to trade autodetection accuracy and/or frequency for
+              better performance.
+            </p>
+            <p>
+              Note that some browsers <a href="https://developer.mozilla.org/en-US/docs/Web/API/Performance/now" target="_blank">limit the accuracy of time measurements</a>, though once the bars go past the blue line those limitations are largely inconsequential.
+            </p>
+            <div class="performance-graph-container">
+              <div class="performance-graph">
+                <div class="time-budget hz144"></div>
+                <div class="time-budget hz120"></div>
+                <div class="time-budget hz60"></div>
+                <div class="time-budget hz30"></div>
+                <div class="time-budget hz24"></div>
+                <div class="time-budget rest"></div>
+
+                <div class="bar-container">
+                  <div class="average-case">
+                    <div class="stats">
+                      <b>Average: </b>
+                      <span class="draw">draw (main) {{(performanceData?.imageDraw?.averageTime ?? 0).toFixed(1)}} ms</span> |
+                      <span class="draw-blackframe">blackframe {{(performanceData?.blackFrame?.averageTime ?? 0).toFixed(1)}} ms</span> |
+                      <span class="processing">
+                        processing {{
+                          Math.max(
+                            (performanceData?.total?.averageTime ?? 0)
+                              - (performanceData?.imageDraw?.averageTime ?? 0)
+                              - (performanceData?.blackFrame?.averageTime ?? 0),
+                            0
+                          ).toFixed(1)
+                        }} ms
+                      </span>
+                    </div>
+                    <div class="bar">
+                      <div
+                        class="draw"
+                        :style="{'width': (performanceData?.imageDraw?.averageTime ?? 0) + '%'}"
+                      >
+                      </div>
+                      <div
+                        class="draw-blackframe"
+                        :style="{'width': (performanceData?.blackFrame?.averageTime ?? 0) + '%'}"
+                      >
+                      </div>
+                      <div
+                        class="processing"
+                        :style="{
+                          'width': Math.max(
+                            (performanceData?.total?.averageTime ?? 0)
+                              - (performanceData?.imageDraw?.averageTime ?? 0)
+                              - (performanceData?.blackFrame?.averageTime ?? 0),
+                            0
+                          ) + '%'
+                        }"
+                      >
+                      </div>
+                    </div>
+                  </div>
+                  <div class="worst-case">
+                    <div class="stats">
+                      <b>Worst: </b>
+                      <span class="draw">draw (main) {{(performanceData?.imageDraw?.worstTime ?? 0).toFixed(1)}} ms</span> |
+                      <span class="draw-blackframe">blackframe {{(performanceData?.blackFrame?.worstTime ?? 0).toFixed(1)}} ms</span> |
+                      <span class="processing">
+                        processing {{
+                          Math.max(
+                            (performanceData?.total?.worstTime ?? 0)
+                              - (performanceData?.imageDraw?.worstTime ?? 0)
+                              - (performanceData?.blackFrame?.worstTime ?? 0),
+                            0
+                          ).toFixed(1)
+                        }} ms
+                      </span>
+                    </div>
+                    <div class="bar">
+                      <div
+                        class="draw"
+                        :style="{'width': (performanceData?.imageDraw?.worstTime ?? 0) + '%'}"
+                      >
+                      </div>
+                      <div
+                        class="draw-blackframe"
+                        :style="{'width': (performanceData?.blackFrame?.worstTime ?? 0) + '%'}"
+                      >
+                      </div>
+                      <div
+                        class="processing"
+                        :style="{
+                          'width': Math.max(
+                            (performanceData?.total?.worstTime ?? 0)
+                              - (performanceData?.imageDraw?.worstTime ?? 0)
+                              - (performanceData?.blackFrame?.worstTime ?? 0),
+                            0
+                          ) + '%'
+                        }"
+                      >
+                      </div>
+                    </div>
+                  </div>
+
+                  <div class="average-case">
+                    <div class="stats">
+                      <b>AR change (average): </b>
+                      <span class="draw">draw (main) {{(performanceData?.imageDraw?.averageTime ?? 0).toFixed(1)}} ms</span> |
+                      <span class="draw-blackframe">blackframe {{(performanceData?.blackFrame?.averageTime ?? 0).toFixed(1)}} ms</span> |
+                      <span class="processing">processing {{
+                        (
+                          (performanceData?.fastLetterbox?.averageTime ?? 0)
+                          + (performanceData?.edgeDetect?.averageTime ?? 0)
+                        ).toFixed(1)
+                        }} ms</span>
+                    </div>
+                    <div class="bar">
+                      <div
+                        class="draw"
+                        :style="{'width': (performanceData?.imageDraw?.averageTime ?? 0) + '%'}"
+                      >
+                      </div>
+                      <div
+                        class="draw-blackframe"
+                        :style="{'width': (performanceData?.blackFrame?.averageTime ?? 0) + '%'}"
+                      >
+                      </div>
+                      <div
+                        class="processing"
+                        :style="{
+                          'width': (
+                            (performanceData?.fastLetterbox?.averageTime ?? 0)
+                            + (performanceData?.edgeDetect?.averageTime ?? 0)
+                          ) + '%'
+                        }"
+                      >
+                      </div>
+                    </div>
+                  </div>
+                  <div class="worst-case">
+                    <div class="stats">
+                      <b>AR change (worst): </b>
+                      <span class="draw">draw (main) {{(performanceData?.imageDraw?.worstTime ?? 0).toFixed(1)}} ms</span> |
+                      <span class="draw-blackframe">blackframe {{(performanceData?.blackFrame?.worstTime ?? 0).toFixed(1)}} ms</span> |
+                      <span class="processing">processing {{
+                        (
+                          (performanceData?.fastLetterbox?.worstTime ?? 0)
+                          + (performanceData?.edgeDetect?.worstTime ?? 0)
+                        ).toFixed(1)
+                        }} ms</span>
+                    </div>
+                    <div class="bar">
+                      <div
+                        class="draw"
+                        :style="{'width': (performanceData?.imageDraw?.worstTime ?? 0) + '%'}"
+                      >
+                      </div>
+                      <div
+                        class="draw-blackframe"
+                        :style="{'width': (performanceData?.blackFrame?.worstTime ?? 0) + '%'}"
+                      >
+                      </div>
+                      <div
+                        class="processing"
+                        :style="{
+                          'width': (
+                            (performanceData?.fastLetterbox?.worstTime ?? 0)
+                            + (performanceData?.edgeDetect?.worstTime ?? 0)
+                          ) + '%'
+                        }"
+                      >
+                      </div>
+                    </div>
+                  </div>
+                </div>
+
+              </div>
+            </div>
+          </div>
+
+
           <div class="settings-segment">
+            <!-- <h2>Basic settings</h2> -->
+
+            <!-- <div class="field">
+              <div class="label">
+                Stop autodetection after first detection:
+              </div>
+              <div class="">
+                <input type="checkbox" v-model="settings.active.arDetect.earlyStopOptions.stopAfterFirstDetection" />
+              </div>
+            </div>
+            <div class="field">
+              <div class="label">
+                Stop detection after a period of time:
+              </div>
+              <div class="">
+                <input type="checkbox" v-model="settings.active.arDetect.earlyStopOptions.stopAfterTimeout" />
+              </div>
+            </div>
+            <div class="field">
+              <div class="label">
+                Stop detection after:
+              </div>
+              <div class="input">
+                <input type="input" v-model="settings.active.arDetect.earlyStopOptions.stopTimeout" />
+                <div class="unit">seconds</div>
+              </div>
+            </div> -->
 
             <div class="field">
               <div class="label">Autodetection frequency (time between samples)</div>
@@ -33,97 +243,7 @@
             </div>
 
             <div class="field">
-              <div class="label">Reduced autodetection frequency</div>
+              <div class="label">Frame extraction canvas type:</div>
-              <div class="range-input">
-                <input
-                  type="range"
-                  :value="Math.log(settings.active.arDetect.timers.playingReduced)"
-                  @change="setArCheckFrequency($event.target.value, 'playingReduced')"
-                  min="2.3"
-                  max="9.3"
-                  step="0.01"
-                />
-                <input
-                  v-model="settings.active.arDetect.timers.playingReduced"
-                  @change="setArCheckFrequency($event.target.value, 'playingReduced')"
-                  class="input"
-                  type="text"
-                >
-                <div class="unit">ms</div>
-              </div>
-            </div>
-
-            <div class="field">
-              <div class="label">Poll for aspect ratio changes in background tabs:</div>
-              <div class="select">
-                <select v-model="settings.active.arDetect.polling.runInBackgroundTabs" @change="settings.saveWithoutReload">
-                  <option :value="AardPollingOptions.No">Never</option>
-                  <option :value="AardPollingOptions.Reduced">Use reduced polling rate</option>
-                  <option :value="AardPollingOptions.Full">Use normal polling rate</option>
-                </select>
-              </div>
-            </div>
-            <div v-if="settings.active.arDetect.polling.runInBackgroundTabs === AardPollingOptions.Full" class="hint warn">
-              Using normal polling rate in background tabs is NOT recommended.
-            </div>
-
-            <div class="field">
-              <div class="label">Poll for aspect ratio changes in small players:</div>
-              <div class="select">
-                <select v-model="settings.active.arDetect.polling.runOnSmallVideos" @change="settings.saveWithoutReload">
-                  <option :value="AardPollingOptions.No">Never</option>
-                  <option :value="AardPollingOptions.Reduced">Use reduced polling rate</option>
-                  <option :value="AardPollingOptions.Full">Use normal polling rate</option>
-                </select>
-              </div>
-            </div>
-            <div v-if="settings.active.arDetect.polling.runOnSmallVideos === AardPollingOptions.Full" class="hint warn">
-              Using normal polling rate on small videos is NOT recommended.
-            </div>
-
-            <div class="field">
-              <div class="label">Stop autodetection after first detection:</div>
-              <div class="select">
-                <select v-model="settings.active.arDetect.autoDisable.onFirstChange" @change="settings.saveWithoutReload">
-                  <option :value="true">Yes</option>
-                  <option :value="false">No</option>
-                </select>
-              </div>
-            </div>
-
-            <div class="field" :class="{disabled: settings.active.arDetect.autoDisable.onFirstChange}">
-              <div class="label">Stop autodetection if aspect ratio doesn't change for some time:</div>
-              <div class="select">
-                <select v-model="settings.active.arDetect.autoDisable.ifNotChanged" @change="settings.saveWithoutReload">
-                  <option :value="true">Yes</option>
-                  <option :value="false">No</option>
-                </select>
-              </div>
-            </div>
-
-            <div v-if="settings.active.arDetect.autoDisable.ifNotChanged" class="field">
-              <div class="label">Stop autodetection if aspect ratio doesn't change for:</div>
-              <div class="range-input">
-                <input
-                  type="range"
-                  :value="Math.log(settings.active.arDetect.autoDisable.ifNotChangedTimeout / 10)"
-                  @change="setAutoDisableTimeout($event.target.value, 10)"
-                  min="2.3"
-                  max="9.3"
-                  step="0.01"
-                />
-                <input
-                  :value="settings.active.arDetect.autoDisable.ifNotChangedTimeout / 1000"
-                  @change="setAutoDisableTimeout($event.target.value, 1000)"
-                  class="input"
-                  type="text"
-                >
-                <div class="unit">s</div>
-              </div>
-            </div>
-
-            <div class="field">
-              <div class="label">Autodetection canvas type:</div>
               <div class="select">
                 <select v-model="settings.active.arDetect.aardType" @change="settings.saveWithoutReload">
                   <option value="auto">Automatic</option>
@@ -138,11 +258,11 @@
               <div class="input">
                 <input v-model="settings.active.arDetect.allowedMisaligned" />
               </div>
-            </div>
+              <div class="hint">
-            <div class="hint">
+                Ultrawidify detects letterbox only if video is vertically centered. Some people are bad at vertically
-              Ultrawidify detects letterbox only if video is vertically centered. Some people are bad at vertically
+                centering the content, though. This is how off-center the video can be before autodetection will
-              centering the content, though. This is how off-center the video can be before autodetection will
+                refuse to crop it (% of total height).
-              refuse to crop it (% of total height).
+              </div>
             </div>
           </div>
 
@@ -190,7 +310,6 @@ import StretchType from '../../../common/enums/StretchType.enum';
 import CropModePersistence from '../../../common/enums/CropModePersistence.enum';
 import AlignmentOptionsControlComponent from './AlignmentOptionsControlComponent.vue';
 import JsonEditor from '@csui/src/components/JsonEditor';
-import {AardPollingOptions} from '@src/ext/lib/aard/enums/aard-polling-options.enum';
 
 export default {
   components: {
@@ -214,9 +333,6 @@ export default {
       performanceData: {},
       graphRefreshInterval: undefined,
       settingsJson: {},
-
-      // enums n stuff
-      AardPollingOptions,
     }
   },
   computed: {
@@ -232,7 +348,7 @@ export default {
   },
   mounted() {
     this.eventBus.sendToTunnel('get-aard-timing');
-    // this.graphRefreshInterval = setInterval(() => this.eventBus.sendToTunnel('get-aard-timing'), 500);
+    this.graphRefreshInterval = setInterval(() => this.eventBus.sendToTunnel('get-aard-timing'), 500);
     this.resetSettingsEditor();
   },
   destroyed() {
@@ -243,13 +359,8 @@ export default {
     async openOptionsPage() {
       BrowserDetect.runtime.openOptionsPage();
     },
-    setArCheckFrequency(event, timer) {
+    setArCheckFrequency(event) {
-      this.settings.active.arDetect.timers[timer ?? 'playing'] = Math.floor(Math.pow(Math.E, event));
+      this.settings.active.arDetect.timers.playing = Math.floor(Math.pow(Math.E, event));
-      this.settings.saveWithoutReload();
-    },
-    setAutoDisableTimeout(event, multiplier) {
-      this.settings.active.arDetect.autoDisable.ifNotChangedTimeout = Math.floor(Math.pow(Math.E, event)) * multiplier;
-      this.settings.saveWithoutReload();
     },
     refreshGraph() {
        this.eventBus.sendToTunnel('get-aard-timing');

src/csui/src/PlayerUiPanels/ChangelogPanel.vue

@@ -9,7 +9,6 @@
         <ul>
           <li>In-player UI can now appear in full-screen for websites that put players into top layer</li>
           <li>Embedded sites now inherit settings of the parent frame</li>
-          <li>Added validation to custom aspect ratio entry menu. Corrected parsing of aspect ratios given in the X:Y format, even though aspect ratios should be ideally given as a single number.</li>
         </ul>
 
       </div>

src/csui/src/PlayerUiPanels/PanelComponents/ExtensionSettings/SiteExtensionSettings.vue

@@ -165,7 +165,7 @@
         </div>
       </div>
 
-      <div class="field">
+       <div class="field">
         <div class="label">
           Use these settings for <span class="color-emphasis">embedded content</span>?
           <!-- <span class="sub-label"><br/>under the following conditions:</span> -->
@@ -174,35 +174,12 @@
           <select
             :value="siteDefaultForEmbedded"
             @click="setSiteOption('applyToEmbeddedContent', $event)"
-          >
-            <option :value="EmbeddedContentSettingsOverridePolicy.Always">
-              Always(-ish)
-            </option>
-            <option :value="EmbeddedContentSettingsOverridePolicy.UseAsDefault">
-              Use as default
-            </option>
-            <option :value="EmbeddedContentSettingsOverridePolicy.Never">
-              Never
-            </option>
-          </select>
-        </div>
-      </div>
-
-      <div class="field">
-        <div class="label">
-          Force these settings <span class="color-emphasis">when embedded</span>?
-          <!-- <span class="sub-label"><br/>under the following conditions:</span> -->
-        </div>
-        <div class="select">
-          <select
-            :value="siteDefaultOverrideEmbedded"
-            @click="setSiteOption('overrideWhenEmbedded', $event)"
           >
             <option :value="true">
-              Yes
+              Unless overridden
             </option>
             <option :value="false">
-              No
+              Never
             </option>
           </select>
         </div>
@@ -313,14 +290,12 @@
 import ExtensionMode from '@src/common/enums/ExtensionMode.enum';
 import VideoAlignmentType from '@src/common/enums/VideoAlignmentType.enum';
 import CropModePersistence from '@src/common/enums/CropModePersistence.enum';
-import EmbeddedContentSettingsOverridePolicy from '@src/common/enums/EmbeddedContentSettingsOverridePolicy.enum';
 
 export default {
   data() {
     return {
       CropModePersistence: CropModePersistence,
       ExtensionMode,
-      EmbeddedContentSettingsOverridePolicy,
       alignmentOptions: [
         {label: 'Top left', arguments: {x: VideoAlignmentType.Left, y: VideoAlignmentType.Top}},
         {label: 'Top center', arguments: {x: VideoAlignmentType.Center, y: VideoAlignmentType.Top}},
@@ -371,10 +346,7 @@ export default {
       };
     },
     siteDefaultForEmbedded() {
-      return this.siteSettings.raw?.applyToEmbeddedContent;
+      return this.siteSettings.raw?.applyToEmbeddedContent ?? true;
-    },
-    siteDefaultOverrideEmbedded() {
-      return this.siteSettings.raw?.overrideWhenEmbedded ?? false;
     },
     siteDefaultCrop() {
       return this.siteSettings.raw?.defaults?.crop ? JSON.stringify(this.siteSettings.raw?.defaults?.crop) : JSON.stringify({useDefault: true});
@@ -425,6 +397,7 @@ export default {
           break;
       }
 
+      // console.log('getting data from:', settingsData);
 
       try {
         if (
@@ -585,14 +558,11 @@ export default {
       this.$nextTick( () => this.$forceUpdate());
     },
 
-    setSiteOption(optionPath, event) {
-      const value = event.target.value;
-      this.siteSettings.set(optionPath, value);
-    },
-
     setExtensionMode(component, event) {
       const option = event.target.value;
 
+      console.log('SET EXTENSION MODE — OPTIONS:', option);
+
       if (option === 'complex') {
         return;
       }

src/csui/src/PlayerUiPanels/PanelComponents/VideoSettings/CropOptionsPanel.vue

@@ -55,13 +55,10 @@
             <!-- We do an ugly in order to avoid spamming functions down at the bottom -->
             <input
               v-model="editModeOptions.crop.selected.arguments.ratio"
-              @blur="updateLabel('crop')"
+              @blur="editModeOptions.crop.selected.label === 'New aspect ratio' ? editModeOptions.crop.selected.label = editModeOptions.crop.selected.arguments.ratio : null"
             >
           </div>
         </div>
-        <div v-if="editModeOptions.crop?.error" class="hint error">
-          {{editModeOptions.crop.error}}
-        </div>
         <div class="hint">
           You can enter a ratio in width:height format (e.g. "21:9" or "1:2.39"), or just the factor
           (in this case, "1:2.39" would become "2.39" and "21:9" would become "2.33"). You should enter

src/csui/src/PlayerUiPanels/PanelComponents/VideoSettings/StretchOptionsPanel.vue

@@ -70,13 +70,10 @@
             <!-- We do an ugly in order to avoid spamming functions down at the bottom -->
             <input
               v-model="editModeOptions.stretch.selected.arguments.ratio"
-              @blur="updateLabel('stretch')"
+              @blur="editModeOptions.stretch.selected.label === 'Stretch to ...' ? editModeOptions.stretch.selected.label = `Stretch to ${editModeOptions.stretch.selected.arguments.ratio}` : null"
             >
           </div>
         </div>
-        <div v-if="editModeOptions.stretch?.error" class="hint error">
-          {{editModeOptions.stretch.error}}
-        </div>
         <div class="hint">
             You can enter a ratio in width:height format (e.g. "21:9" or "1:2.39"), or just the factor
             (in this case, "1:2.39" would become "2.39" and "21:9" would become "2.33"). You should enter

src/csui/src/PlayerUiPanels/PanelComponents/VideoSettings/ZoomOptionsPanel.vue

@@ -48,20 +48,16 @@
                 <!-- We do an ugly in order to avoid spamming functions down at the bottom -->
                 <input
                   v-model="editModeOptions.zoom.selected.arguments.ratio"
-                  @blur="updateLabel('zoom')"
+                  @blur="editModeOptions.zoom.selected.label === 'New aspect ratio' ? editModeOptions.zoom.selected.label = editModeOptions.zoom.selected.arguments.ratio : null"
                 >
               </div>
             </div>
-            <div v-if="editModeOptions.zoom?.error" class="hint error">
-              {{editModeOptions.zoom.error}}
-            </div>
             <div class="hint">
               You can enter a ratio in width:height format (e.g. "21:9" or "1:2.39"), or just the factor
               (in this case, "1:2.39" would become "2.39" and "21:9" would become "2.33"). You should enter
               your numbers without quote marks. Number will be converted to factor form on save.
             </div>
 
-
             <div class="field">
               <div class="label">
                 Label:

src/csui/src/popup/panels/PopupVideoSettings.vue

@@ -9,21 +9,18 @@
         <div class="info">
           Extension is not enabled for this site.
         </div>
-        <div class="text-center">
+        <div>
-          <p>Please enable extension for this site.</p>
+          Please enable extension for this site.
         </div>
         <div>
           <button
             class="flex flex-row items-center"
-            style="background-color: transparent; padding: 0.5rem 1rem; margin-top: 1rem; border: 1px solid #fa68;"
+            style="background-color: transparent; padding: 0.25rem 0.5rem; margin-top: 1rem;"
             @click="openSettings()"
           >
             Open settings <mdicon style="margin-left: 0.5rem;" name="open-in-new" size="16"></mdicon>
           </button>
         </div>
-        <div class="text-center">
-          <p><small>Extension may work on content embedded from other sites.</small></p>
-        </div>
 
       </div>
     </template>

src/csui/src/res-common/common.scss

@@ -210,10 +210,6 @@ button, .button {
   margin-left: 5rem;
   // width: 100%;
   box-sizing:border-box;
-
-  &.error {
-    color: #f41;
-  }
 }
 
 .options-bar {

src/csui/src/utils/EditModeMixin.js

@@ -52,47 +52,7 @@ export default {
       }
     },
 
-    validateAspectRatio(actionType) {
-      // deal with potentially invalid aspect ratios
-      if (this.editModeOptions[actionType].selected.arguments.ratio.includes(':')) {
-        const ratioParts = this.editModeOptions[actionType].selected.arguments.ratio.split(':');
-        if (ratioParts.length !== 2) {
-          this.editModeOptions[actionType].error = 'Entered aspect ratio is not valid.';
-          return false;
-        } else {
-          const ratio = ratioParts[0] / ratioParts[1];
-          if (isNaN(ratio) || ratio <= 0) {
-            this.editModeOptions[actionType].error = 'Entered aspect ratio is not valid.';
-            return false;
-          }
-          this.editModeOptions[actionType].selected.arguments.ratio = ratio;
-        }
-      } else {
-        if (isNaN(this.editModeOptions[actionType].selected.arguments.ratio)) {
-          this.editModeOptions[actionType].error = "Entered aspect ratio is not valid.";
-          return false;
-        }
-      }
-
-      delete this.editModeOptions[actionType].error;
-      return true;
-    },
-
-    /**
-     *  Updates button label, ideally after aspect ratio is entered,
-     *  after input field blurred, and only if aspect ratio is valid.
-     */
-    updateLabel(actionType) {
-      if (this.validateAspectRatio(actionType) && this.editModeOptions[actionType].selected.label === 'New aspect ratio') {
-        this.editModeOptions[actionType].selected.label = this.editModeOptions[actionType].selected.arguments.ratio;
-      }
-    },
-
     saveShortcut(actionType) {
-      if (! this.validateAspectRatio(actionType)) {
-        return;
-      }
-
       if (!this.editModeOptions[actionType]?.selectedIndex) {
         this.settings.active.commands[actionType].push(this.editModeOptions[actionType].selected);
       }

src/ext/conf/ExtConfPatches.ts

@@ -9,7 +9,6 @@ import { _cp } from '../../common/js/utils';
 import CropModePersistence from '../../common/enums/CropModePersistence.enum';
 import AspectRatioType from '../../common/enums/AspectRatioType.enum';
 import { update } from 'lodash';
-import EmbeddedContentSettingsOverridePolicy from '../../common/enums/EmbeddedContentSettingsOverridePolicy.enum';
 
 const ExtensionConfPatch = Object.freeze([
   {
@@ -36,7 +35,8 @@ const ExtensionConfPatch = Object.freeze([
   }, {
     forVersion: '6.2.6',
     updateFn: (userOptions: SettingsInterface, defaultOptions) => {
-      console.log('[ultrawidify] Migrating settings — applying patches for version 6.2.6');
+
+      console.warn('[ultrawidify] STARTING SETTINGS MIGRATION TO 6.2.6');
 
       if (!userOptions.commands) {
         userOptions.commands = {
@@ -239,28 +239,7 @@ const ExtensionConfPatch = Object.freeze([
         }
       }
     }
-  }, {
-    forVersion: '6.3.92',
-    updateFn: (userOptions: SettingsInterface) => {
-      // applyToEmbeddedContent is now an enum, and also no longer optional
-      for (const site in userOptions.sites) {
-        if (userOptions.sites[site].applyToEmbeddedContent === undefined) {
-          userOptions.sites[site].applyToEmbeddedContent = EmbeddedContentSettingsOverridePolicy.Always;
-        } else {
-          userOptions.sites[site].applyToEmbeddedContent = userOptions.sites[site].applyToEmbeddedContent ? EmbeddedContentSettingsOverridePolicy.Always : EmbeddedContentSettingsOverridePolicy.Never;
-        }
-      }
-    }
-  },
-  {
-    forVersion: '6.3.93',
-    updateFn: (userOptions: SettingsInterface, defaultOptions: SettingsInterface) => {
-      userOptions.arDetect.polling = defaultOptions.arDetect.polling;
-      userOptions.arDetect.subtitles = defaultOptions.arDetect.subtitles;
-      userOptions.arDetect.autoDisable = defaultOptions.arDetect.autoDisable;
-    }
   }
-
 ]);
 
 

src/ext/conf/ExtensionConf.ts

@@ -9,8 +9,6 @@ import CropModePersistence from '../../common/enums/CropModePersistence.enum';
 import SettingsInterface from '../../common/interfaces/SettingsInterface';
 import BrowserDetect from './BrowserDetect';
 import { Extension } from 'typescript';
-import EmbeddedContentSettingsOverridePolicy from '../../common/enums/EmbeddedContentSettingsOverridePolicy.enum';
-import { AardPollingOptions } from '../lib/aard/enums/aard-polling-options.enum';
 
 if(Debug.debug)
   console.log("Loading: ExtensionConf.js");
@@ -23,16 +21,6 @@ const ExtensionConf: SettingsInterface = {
   arDetect: {
     aardType: 'auto',
 
-    polling: {
-      runInBackgroundTabs: AardPollingOptions.Reduced,
-      runOnSmallVideos: AardPollingOptions.Reduced
-    },
-
-    subtitles: {
-      resetIfDetected: true,
-      resumeAfter: 5000,
-    },
-
     earlyStopOptions: {
       stopAfterFirstDetection: false,
       stopAfterTimeout: false,
@@ -52,10 +40,12 @@ const ExtensionConf: SettingsInterface = {
       tickrate: 10,          // 1 tick every this many milliseconds
     },
     autoDisable: {            // settings for automatically disabling the extension
-      onFirstChange: false,       // disable once we have a stable aspect ratio
+      maxExecutionTime: 6000, // if execution time of main autodetect loop exceeds this many milliseconds,
-      ifNotChanged: false,        // disable if Ar hasn't changed for this long
+                              // we disable it.
-      ifNotChangedTimeout: 20000, // if user enables ifNotChangedTimeout, we default to 20s
+      consecutiveTimeoutCount: 5,  // we only do it if it happens this many consecutive times
-      ifSubtitles: false,         // disable if subtitles are detected
+
+      // FOR FUTURE USE
+      consecutiveArResets: 5       // if aspect ratio reverts immediately after AR change is applied, we disable everything
     },
     canvasDimensions: {
       blackframeCanvas: {   // smaller than sample canvas, blackframe canvas is used to recon for black frames
@@ -749,7 +739,6 @@ const ExtensionConf: SettingsInterface = {
         theater: ExtensionMode.Disabled,     // Lies! means 'theater-ish'
         normal: ExtensionMode.Disabled       // Not actually used.
       },
-      applyToEmbeddedContent: EmbeddedContentSettingsOverridePolicy.Always,
 
       defaultType: 'unknown',
       persistCSA: CropModePersistence.Disabled,
@@ -781,7 +770,6 @@ const ExtensionConf: SettingsInterface = {
         theater: ExtensionMode.Default,
         normal: ExtensionMode.Disabled
       },
-      applyToEmbeddedContent: EmbeddedContentSettingsOverridePolicy.Always,
 
       type: 'user-defined',
       defaultType: 'user-defined',
@@ -813,8 +801,7 @@ const ExtensionConf: SettingsInterface = {
         theater: ExtensionMode.Enabled,
         normal: ExtensionMode.Disabled
       },
-      applyToEmbeddedContent: EmbeddedContentSettingsOverridePolicy.Always,
+
-      overrideWhenEmbedded: true,
       override: false,                  // ignore value localStorage in favour of this
       type: 'official',                 // is officially supported? (Alternatives are 'community' and 'user-defined')
       defaultType: 'official',          // if user mucks around with settings, type changes to 'user-defined'.
@@ -854,8 +841,6 @@ const ExtensionConf: SettingsInterface = {
         theater: ExtensionMode.Enabled,
         normal: ExtensionMode.Disabled
       },
-      applyToEmbeddedContent: EmbeddedContentSettingsOverridePolicy.Always,
-      overrideWhenEmbedded: true,
 
       override: false,                  // ignore value localStorage in favour of this
       type: 'official',                 // is officially supported? (Alternatives are 'community' and 'user-defined')
@@ -896,7 +881,7 @@ const ExtensionConf: SettingsInterface = {
         theater: ExtensionMode.Enabled,
         normal: ExtensionMode.Disabled
       },
-      applyToEmbeddedContent: EmbeddedContentSettingsOverridePolicy.Always,
+
       override: false,
       type: 'community',
       defaultType: 'community',
@@ -922,7 +907,6 @@ const ExtensionConf: SettingsInterface = {
         theater: ExtensionMode.Enabled,
         normal: ExtensionMode.Default
       },
-      applyToEmbeddedContent: EmbeddedContentSettingsOverridePolicy.Always,
       type: 'community',
       defaultType: 'community',
       activeDOMConfig: 'community-mstefan99',
@@ -964,7 +948,6 @@ const ExtensionConf: SettingsInterface = {
         theater: ExtensionMode.Enabled,
         normal: ExtensionMode.Enabled
       },
-      applyToEmbeddedContent: EmbeddedContentSettingsOverridePolicy.Always,
       type: 'official',
       defaultType: 'official',
     },
@@ -989,7 +972,6 @@ const ExtensionConf: SettingsInterface = {
         theater: ExtensionMode.Disabled,
         normal: ExtensionMode.Disabled
       },
-      applyToEmbeddedContent: EmbeddedContentSettingsOverridePolicy.Never,
       type: 'officially-disabled',
       defaultType: 'officially-disabled',
       activeDOMConfig: 'official',
@@ -1027,7 +1009,6 @@ const ExtensionConf: SettingsInterface = {
         theater: ExtensionMode.Disabled,
         normal: ExtensionMode.Disabled
       },
-      applyToEmbeddedContent: EmbeddedContentSettingsOverridePolicy.Never,
       type: 'officially-disabled',
       defaultType: 'officially-disabled',
       activeDOMConfig: 'official',
@@ -1065,7 +1046,54 @@ const ExtensionConf: SettingsInterface = {
         theater: ExtensionMode.Disabled,
         normal: ExtensionMode.Disabled
       },
-      applyToEmbeddedContent: EmbeddedContentSettingsOverridePolicy.Never,
+      type: 'officially-disabled',
+      defaultType: 'officially-disabled',
+    },
+    "gfycat.com": {
+      enable: {
+        fullscreen: ExtensionMode.Disabled,
+        theater: ExtensionMode.Disabled,
+        normal: ExtensionMode.Disabled,
+      },
+      enableAard: {
+        fullscreen: ExtensionMode.Disabled,
+        theater: ExtensionMode.Disabled,
+        normal: ExtensionMode.Disabled,
+      },
+      enableKeyboard: {
+        fullscreen: ExtensionMode.Disabled,
+        theater: ExtensionMode.Disabled,
+        normal: ExtensionMode.Disabled,
+      },
+      enableUI: {
+        fullscreen: ExtensionMode.Default,
+        theater: ExtensionMode.Default,
+        normal: ExtensionMode.Default
+      },
+      type: 'officially-disabled',
+      defaultType: 'officially-disabled',
+    },
+    "giant.gfycat.com": {
+      enable: {
+        fullscreen: ExtensionMode.Disabled,
+        theater: ExtensionMode.Disabled,
+        normal: ExtensionMode.Disabled,
+      },
+      enableAard: {
+        fullscreen: ExtensionMode.Disabled,
+        theater: ExtensionMode.Disabled,
+        normal: ExtensionMode.Disabled,
+      },
+      enableKeyboard: {
+        fullscreen: ExtensionMode.Disabled,
+        theater: ExtensionMode.Disabled,
+        normal: ExtensionMode.Disabled,
+      },
+      enableUI: {
+        fullscreen: ExtensionMode.Default,
+        theater: ExtensionMode.Default,
+        normal: ExtensionMode.Default
+      },
       type: 'officially-disabled',
       defaultType: 'officially-disabled',
     },
@@ -1090,7 +1118,6 @@ const ExtensionConf: SettingsInterface = {
         theater: ExtensionMode.Enabled,
         normal: ExtensionMode.Enabled
       },
-      applyToEmbeddedContent: EmbeddedContentSettingsOverridePolicy.Always,
       type: 'community',
       defaultType: 'community',
       activeDOMConfig: 'community',
@@ -1127,7 +1154,6 @@ const ExtensionConf: SettingsInterface = {
         theater: ExtensionMode.Enabled,
         normal: ExtensionMode.Disabled
       },
-      applyToEmbeddedContent: EmbeddedContentSettingsOverridePolicy.Always,
       type: 'community',
       defaultType: 'community',
       activeDOMConfig: 'community',
@@ -1159,7 +1185,6 @@ const ExtensionConf: SettingsInterface = {
         theater: ExtensionMode.Enabled,
         normal: ExtensionMode.Disabled
       },
-      applyToEmbeddedContent: EmbeddedContentSettingsOverridePolicy.Always,
       type: "community",
       defaultType: "community",
       activeDOMConfig: 'community',
@@ -1195,7 +1220,6 @@ const ExtensionConf: SettingsInterface = {
         theater: ExtensionMode.Enabled,
         normal: ExtensionMode.Disabled
       },
-      applyToEmbeddedContent: EmbeddedContentSettingsOverridePolicy.Always,
       type: "community",
       defaultType: "community",
       activeDOMConfig: 'community',

src/ext/lib/EventBus.ts

@@ -115,19 +115,8 @@ export default class EventBus {
     // CommsServer's job. EventBus does not have enough data for this decision.
     // We do, however, have enough data to prevent backflow of messages that
     // crossed CommsServer once already.
-    if (
+    if (this.comms && context?.origin !== CommsOrigin.Server && !context?.borderCrossings?.commsServer) {
-      this.comms
+      this.comms.sendMessage({command, config: commandData, context}, context);
-      && context?.origin !== CommsOrigin.Server
-      && !context?.borderCrossings?.commsServer
-    ) {
-      try {
-        this.comms.sendMessage({command, config: commandData, context}, context);
-      } catch (e) {
-        if (command !== 'reload-required') {
-          // We shouldn't let reload-required command to trigger new reload-required commands.
-          this.send('reload-required', {});
-        }
-      }
     };
 
     // call forwarding functions if they exist
@@ -172,13 +161,7 @@ export default class EventBus {
       // also need to set up a detour because the tunnel is closed
       // in the popup
       if (this.comms) {
-        try {
+        this.comms.sendMessage({command, config, context: this.popupContext}, this.popupContext);
-          this.comms.sendMessage({command, config, context: this.popupContext}, this.popupContext);
-        } catch (e) {
-          if (command !== 'reload-required') {
-            this.send('reload-required', {});
-          }
-        }
       }
     }
   }

src/ext/lib/aard/Aard.ts

@@ -19,7 +19,6 @@ import { AardStatus, initAardStatus } from './interfaces/aard-status.interface';
 import { AardTestResults, initAardTestResults, resetAardTestResults, resetGuardLine } from './interfaces/aard-test-results.interface';
 import { AardTimers, initAardTimers } from './interfaces/aard-timers.interface';
 import { ComponentLogger } from '../logging/ComponentLogger';
-import { AardPollingOptions } from './enums/aard-polling-options.enum';
 
 
 /**
@@ -233,7 +232,7 @@ export class Aard {
   private eventBusCommands = {
     'uw-environment-change': {
       function: (newEnvironment: ExtensionEnvironment) => {
-        // console.log('received extension environment:', newEnvironment, 'player env:', this.videoData?.player?.environment);
+        console.log('received extension environment:', newEnvironment, 'player env:', this.videoData?.player?.environment);
         this.startCheck();
       }
     },
@@ -245,11 +244,6 @@ export class Aard {
           this.hideDebugCanvas();
         }
       }
-    },
-    'url-changed': {
-      function: () => {
-        this.clearAutoDisabled();
-      }
     }
     //   'get-aard-timing': {
     //     function: () => this.handlePerformanceDataRequest()
@@ -436,19 +430,10 @@ export class Aard {
     }
   }
 
-  /**
-   * Clears autoDisable flag
-   */
-  clearAutoDisabled() {
-    this.status.autoDisabled = false;
-    this.timers.autoDisableAt = undefined;
-  }
-
   /**
    * Starts autodetection loop.
    */
   start() {
-    this.clearAutoDisabled();
     this.forceFullRecheck = true;
     if (this.videoData.resizer.lastAr.type === AspectRatioType.AutomaticUpdate) {
       // ensure first autodetection will run in any case
@@ -465,11 +450,6 @@ export class Aard {
 
     this.status.aardActive = true;
     this.animationFrame = window.requestAnimationFrame( (ts: DOMHighResTimeStamp) => this.onAnimationFrame(ts));
-
-    // set auto-disable timer if detection timeout is set
-    if (this.settings.active.arDetect.autoDisable.ifNotChanged) {
-      this.timers.autoDisableAt = Date.now() + this.settings.active.arDetect.autoDisable.ifNotChangedTimeout;
-    }
   }
 
   /**
@@ -502,42 +482,24 @@ export class Aard {
    * @returns
    */
   private canTriggerFrameCheck() {
-
+    // if (this._paused || this._halted || this._exited) {
-    // console.log('ard check status:', this.status);
+    //   return false;
+    // }
 
     // if video was paused & we know that we already checked that frame,
     // we will not check it again.
     const videoState = this.getVideoPlaybackState();
-    const polling = this.settings.active.arDetect.polling;
-    const now = Date.now();
 
     if (videoState !== VideoPlaybackState.Playing) {
       if (this.status.lastVideoStatus === videoState) {
         return false;
       }
     }
-    if (this.status.autoDisabled) {
-      return false;
-    }
-    if (this.timers.autoDisableAt < now) {
-      this.status.autoDisabled = true;
-      return false;
-    }
     this.status.lastVideoStatus = videoState;
 
-    const tabVisible = document.visibilityState === 'visible';
+    const now = Date.now();
-    if (!tabVisible && polling.runInBackgroundTabs === AardPollingOptions.No) {
-      return false;
-    }
-    if (this.videoData.player.isTooSmall && polling.runOnSmallVideos === AardPollingOptions.No) {
-      return false;
-    }
 
-    const isActive = (tabVisible || polling.runInBackgroundTabs !== AardPollingOptions.Reduced)
+    if (now < (this.videoData.player.isTooSmall ? this.timers.reducedPollingNextCheckTime : this.timers.nextFrameCheckTime)) {
-       && (!this.videoData.player.isTooSmall || polling.runOnSmallVideos !== AardPollingOptions.Reduced);
-    const nextCheck = isActive ? this.timers.nextFrameCheckTime : this.timers.reducedPollingNextCheckTime;
-
-    if (now < nextCheck) {
       return false;
     }
 
@@ -546,12 +508,6 @@ export class Aard {
     return true;
   }
 
-  /**
-   * Bootstraps the main loop.
-   *
-   * Honestly this doesn't need description, but I want to put some green
-   * between two adjacent functions.
-   */
   private onAnimationFrame(ts: DOMHighResTimeStamp) {
     if (this.canTriggerFrameCheck()) {
       resetAardTestResults(this.testResults);
@@ -715,7 +671,7 @@ export class Aard {
 
           // console.warn('ASPECT RATIO UNCERTAIN, GUARD LINE INVALIDATED (resetting)')
           this.timer.arChanged();
-          this.updateAspectRatio(this.defaultAr, {uncertainDetection: true});
+          this.updateAspectRatio(this.defaultAr);
 
           break;
         }
@@ -2071,12 +2027,14 @@ export class Aard {
   /**
    * Updates aspect ratio if new aspect ratio is different enough from the old one
    */
-  private updateAspectRatio(ar: number, options?: {uncertainDetection?: boolean}) {
+  private updateAspectRatio(overrideAr?: number) {
+    const ar = overrideAr ?? this.getAr();
+
     // Calculate difference between two ratios
     const maxRatio = Math.max(ar, this.testResults.activeAspectRatio);
     const diff = Math.abs(ar - this.testResults.activeAspectRatio);
 
-    if ((diff / maxRatio) > this.settings.active.arDetect.allowedArVariance) {
+    if (overrideAr || (diff / maxRatio) > this.settings.active.arDetect.allowedArVariance) {
       this.videoData.resizer.updateAr({
         type: AspectRatioType.AutomaticUpdate,
         ratio: this.getAr(),
@@ -2084,16 +2042,6 @@ export class Aard {
         variant: this.arVariant
       });
       this.testResults.activeAspectRatio = ar;
-
-      if (!options?.uncertainDetection) {
-        if (this.settings.active.arDetect.autoDisable.onFirstChange) {
-          this.status.autoDisabled = true;
-        }
-        if (this.settings.active.arDetect.autoDisable.ifNotChanged) {
-          this.timers.autoDisableAt = Date.now() + this.settings.active.arDetect.autoDisable.ifNotChangedTimeout;
-        }
-      }
-
     }
   }
 

src/ext/lib/aard/enums/aard-polling-options.enum.ts

@@ -1,5 +0,0 @@
-export enum AardPollingOptions {
-  No = 0,
-  Reduced = 1,
-  Full = 2
-}

src/ext/lib/aard/interfaces/aard-status.interface.ts

@@ -5,7 +5,7 @@ export interface AardStatus {
   aardReducedPolling: boolean,
   checkInProgress: boolean,
   lastVideoStatus: VideoPlaybackState,
-  autoDisabled: boolean,
+
 }
 
 export function initAardStatus(): AardStatus {
@@ -14,6 +14,5 @@ export function initAardStatus(): AardStatus {
     aardReducedPolling: true,
     checkInProgress: false,
     lastVideoStatus: VideoPlaybackState.NotInitialized,
-    autoDisabled: false,
   }
 }

src/ext/lib/aard/interfaces/aard-timers.interface.ts

@@ -1,5 +1,4 @@
 export interface AardTimers {
-  autoDisableAt: number | undefined;
   nextFrameCheckTime: number;
   reducedPollingNextCheckTime: number;
 }
@@ -8,6 +7,5 @@ export function initAardTimers(): AardTimers {
   return {
     nextFrameCheckTime: 0,
     reducedPollingNextCheckTime: 0,
-    autoDisableAt: undefined,
   };
 }

src/ext/lib/angle-detect/detect-angle-backend.ts

@@ -93,46 +93,38 @@ function detectBackend(str) {
  * @returns
  */
 function detectANGLEBackend(): AngleVersion {
-  try {
+  const canvas = document.createElement("canvas");
-    const canvas = document.createElement("canvas");
+  const gl = canvas.getContext("webgl2") ||
-    const gl = canvas.getContext("webgl2") ||
+    canvas.getContext("webgl") ||
-      canvas.getContext("webgl") ||
+    canvas.getContext("experimental-webgl");
-      canvas.getContext("experimental-webgl");
 
-    if (!gl) {
+  const ext = (gl as any).getExtension("WEBGL_debug_shaders");
-      return AngleVersion.NotAvailable;
-    }
 
-    const ext = (gl as any).getExtension("WEBGL_debug_shaders");
+  if (!ext) {
-
-    if (!ext) {
-      return AngleVersion.NotAvailable;
-    }
-
-    const isWebGL1 = gl instanceof WebGLRenderingContext;
-    const shader = (gl as any).createShader((gl as any).VERTEX_SHADER);
-
-    (gl as any).shaderSource(
-      shader,
-      `#version ${isWebGL1 ? "100" : "300 es"}
-      void main() {
-        gl_Position = vec4(__VERSION__, 1.0, 1.0, 1.0);
-      }
-    `
-    );
-
-    (gl as any).compileShader(shader);
-
-    if (!(gl as any).getShaderParameter(shader, (gl as any).COMPILE_STATUS)) {
-      console.error("invalid shader", (gl as any).getShaderInfoLog(shader));
-      return AngleVersion.NotAvailable;
-    }
-
-    const source = ext.getTranslatedShaderSource(shader);
-
-    return detectBackend(source);
-  } catch (e) {
     return AngleVersion.NotAvailable;
   }
+
+  const isWebGL1 = gl instanceof WebGLRenderingContext;
+  const shader = (gl as any).createShader((gl as any).VERTEX_SHADER);
+
+  (gl as any).shaderSource(
+    shader,
+    `#version ${isWebGL1 ? "100" : "300 es"}
+    void main() {
+      gl_Position = vec4(__VERSION__, 1.0, 1.0, 1.0);
+    }
+  `
+  );
+
+  (gl as any).compileShader(shader);
+
+  if (!(gl as any).getShaderParameter(shader, (gl as any).COMPILE_STATUS)) {
+    console.error("invalid shader", (gl as any).getShaderInfoLog(shader));
+    return AngleVersion.NotAvailable;
+  }
+
+  const source = ext.getTranslatedShaderSource(shader);
+
+  return detectBackend(source);
 }
 

src/ext/lib/settings/SiteSettings.ts

@@ -5,7 +5,6 @@ import { _cp } from '../../../common/js/utils';
 import Settings from './Settings';
 import StretchType from '../../../common/enums/StretchType.enum';
 import VideoAlignmentType from '../../../common/enums/VideoAlignmentType.enum';
-import EmbeddedContentSettingsOverridePolicy from '../../../common/enums/EmbeddedContentSettingsOverridePolicy.enum';
 
 
 export interface GetSiteSettingsOptions {
@@ -80,39 +79,30 @@ export class SiteSettings {
    *  | different subdomain? |
    *     |             |
    *     no           yes ————————> return settings for matching subdomain
-   *     |
    *     V
-   *   return default settings
+   *   [ are we inside of an iframe? ]
-   *
+   *     |                  |
-   *
+   *     no                yes
-   *   Inside iframes, we also get priorities. We fetch settings for both iframe and the parent page.
+   *     V                  V
-   *
+   *  return default     |  exact parent   |
-   *   START HERE
+   *    settings         | hostname match? |
-   *     |
+   *     A                  |           |
-   *   | are there settings for |    +——> |   does iframe host have   |
+   *     |                 no          yes ————> [ applyToEmbeddedContent set? ]
-   *   |      iframe host?      |    |    | overrideWhenEmbedded set? |
+   *     |                  V                           |               |
-   *     |                   |       |        |                    |
+   *     |        | parent matches hostname |           no             yes
-   *     no                 yes —————+       no                   yes —-> return iframe settings
+   *     |        | on different subdomain? |           V               V
-   *     V                                    V
+   *     |            |                 |          use default    use parent settings
-   *   | does parent host have  |          | does parent use settings for @global ? |
+   *     +———<——————— no               yes
-   *   | applyToEmbeddedContent |             |                    |
+   *     |                              |
-   *   |      set to Never?     |             no                  yes ——> return iframe settings
+   *     +———<—— no ——[ applyToEmbeddedContent set? ]
-   *     |                  |                 V
+   *                                    |
-   *     no                 no             | what's applyToEmbeddedContent |
+   *                                   yes
-   *     |                  |              |     set to on parent host?    |
+   *                                    V
-   *     V                  V                 |             |          |
+   *    use settings for matching subdomain of parent
-   *  return parent    return default       Always    UseAsDefault   Never
-   *    settings          settings            |             |          |
-   *                                          V             V          V
-   *                                    return parent       return iframe
-   *                                      settings            settings
    *
    * @returns
    */
-  private getSettingsForSite(options: GetSiteSettingsOptions): {siteSettings: SiteSettingsInterface, usesSettingsFor: string | undefined} {
+  private getSettingsForSite(options: GetSiteSettingsOptions) {
-
-    // console.warn('getSettingsForSite: getting settings for site', options.site, ' — full options:', options);
-
     if (!options.site) {
       return {
         siteSettings: this.settings.active.sites['@global'],
@@ -120,60 +110,52 @@ export class SiteSettings {
       };
     }
 
-    const defaultSettings = {
-      siteSettings: this.settings.active.sites['@global'],
-      usesSettingsFor: '@global'
-    };
-    let siteSettings: {siteSettings: SiteSettingsInterface, usesSettingsFor: string | undefined};
-
     if (this.settings.active.sites[options.site]) {
-      siteSettings = {
+      return {
         siteSettings: this.settings.active.sites[options.site],
         usesSettingsFor: undefined
       };
-    } else {
+    }
-      const urlSegments = this.site.split('.').reverse();
 
-      siteLoop:
+    const urlSegments = this.site.split('.').reverse();
-      for (const cs in this.settings.active.sites) {
-        const configUrlSegments = cs.split('.').reverse();
 
-        // Match site with wildcard site definitions
+    siteLoop:
-        // Also, if definition starts with 'www', match also other subdomains — e.g. if we have a configuration for
+    for (const cs in this.settings.active.sites) {
-        // `www.example.com`, this will also match `example.com`, `subdomain.example.com`, `nested.subdomain.example.com` ...
+      const configUrlSegments = cs.split('.').reverse();
-        if (configUrlSegments[configUrlSegments.length - 1] === '*' || (configUrlSegments[configUrlSegments.length - 1] === 'www')) {
 
-          for (let i = 0; i < configUrlSegments.length - 1 && i < urlSegments.length; i++) {
+      // Match site with wildcard site definitions
-            if (configUrlSegments[i] !== urlSegments[i]) {
+      // Also, if definition starts with 'www', match also other subdomains — e.g. if we have a configuration for
-              continue siteLoop;
+      // `www.example.com`, this will also match `example.com`, `subdomain.example.com`, `nested.subdomain.example.com` ...
-            }
+      if (configUrlSegments[configUrlSegments.length - 1] === '*' || (configUrlSegments[configUrlSegments.length - 1] === 'www')) {
-          }
+
-          siteSettings = {
+        for (let i = 0; i < configUrlSegments.length - 1 && i < urlSegments.length; i++) {
-            siteSettings: this.settings.active.sites[cs],
+          if (configUrlSegments[i] !== urlSegments[i]) {
-            usesSettingsFor: cs
+            continue siteLoop;
           }
         }
+        return {
+          siteSettings: this.settings.active.sites[cs],
+          usesSettingsFor: cs
+        }
       }
     }
 
-    if (!options.isIframe) {
+    // If we're inside of an iframe, let's see whether we can use parent settings
-      return siteSettings ?? defaultSettings;
+    if (options.isIframe) {
-    } else {
+      const potentialSettings = this.getSettingsForSite({site: options.parentHostname});
-      const parentSettings = this.getSettingsForSite({site: options.parentHostname});
 
-      if (siteSettings) {
+      if (potentialSettings.siteSettings.applyToEmbeddedContent !== false) {
-        return (
+        if (!potentialSettings.usesSettingsFor) {
-          siteSettings.siteSettings.overrideWhenEmbedded || parentSettings.usesSettingsFor === '@global' || (
+          potentialSettings.usesSettingsFor = options.parentHostname;
-            parentSettings.siteSettings.applyToEmbeddedContent && parentSettings.siteSettings.applyToEmbeddedContent !== EmbeddedContentSettingsOverridePolicy.Always
-          )
-        ) ? siteSettings : parentSettings;
-      } else {
-        if (parentSettings.usesSettingsFor !== '@global' && parentSettings.siteSettings.applyToEmbeddedContent === EmbeddedContentSettingsOverridePolicy.Never) {
-          return defaultSettings;
         }
-        return parentSettings;
+        return potentialSettings;
       }
     }
+
+    return {
+      siteSettings: this.settings.active.sites['@global'],
+      usesSettingsFor: '@global'
+    };
   }
 
   /**
@@ -432,19 +414,6 @@ export class SiteSettings {
   //#endregion
 
   //#region set shit
-  /**
-   * Returns target site for settings application
-   */
-  private getTargetSite() {
-    if (!this.usesSettingsFor || this.usesSettingsFor === '@global') {
-      return this.site;
-    }
-    if (this.settings.active.sites[this.usesSettingsFor].applyToEmbeddedContent === EmbeddedContentSettingsOverridePolicy.Always) {
-      return this.usesSettingsFor;
-    }
-    return this.site;
-  }
-
   /**
    * Sets option value.
    * @param optionPath path to value in object notation (dot separated)
@@ -452,29 +421,26 @@ export class SiteSettings {
    * @param reload whether we should trigger a reload in components that require it
    */
   async set(optionPath: string, optionValue: any, options: {reload?: boolean, noSave?: boolean, scripted?: boolean} = {reload: false}) {
-    // const targetSite = this.getTargetSite();
-    const targetSite = this.site;
-
     // if no settings exist for this site, create an empty object.
     // If this function is not being called in response to user actin,
     // create fake settings object.
-    if (options.scripted && !this.settings.active.sites[targetSite]) {
+    if (options.scripted && !this.settings.active.sites[this.site]) {
-      this.settings.active.sites[targetSite] = _cp(this.settings.active.sites['@global']);
+      this.settings.active.sites[this.site] = _cp(this.settings.active.sites['@global']);
-      this.settings.active.sites[targetSite].autocreated = true;
+      this.settings.active.sites[this.site].autocreated = true;
-      this.settings.active.sites[targetSite].type = 'unknown';
+      this.settings.active.sites[this.site].type = 'unknown';
     } else {
-      if (!this.settings.active.sites[targetSite] || this.settings.active.sites[targetSite].autocreated) {
+      if (!this.settings.active.sites[this.site] || this.settings.active.sites[this.site].autocreated) {
-        this.settings.active.sites[targetSite] = _cp(this.data);
+        this.settings.active.sites[this.site] = _cp(this.data);
-        this.settings.active.sites[targetSite].type = 'user-defined';
+        this.settings.active.sites[this.site].type = 'user-defined';
       }
     }
 
     const pathParts = optionPath.split('.');
 
     if (pathParts.length === 1) {
-      this.settings.active.sites[targetSite][optionPath] = optionValue;
+      this.settings.active.sites[this.site][optionPath] = optionValue;
     } else {
-      let iterator = this.settings.active.sites[targetSite];
+      let iterator = this.settings.active.sites[this.site];
       let i;
       let iterated = '';
 
@@ -508,6 +474,7 @@ export class SiteSettings {
     };
   }
 
+
   /**
    * Sets temporary option value (for Persistence.UntilReload)
    * @param optionPath path to value in object notation (dot separated)

src/ext/lib/uwui/UI.js

@@ -61,10 +61,10 @@ class UI {
 
   async init() {
     if (!this.canRun()) {
-      // console.log('ui config: canRun returned false', this.siteSettings?.data.enableUI.fullscreen === ExtensionMode.Enabled, this.siteSettings?.data.enableUI.theater === ExtensionMode.Enabled, this.siteSettings?.data.enableUI.normal === ExtensionMode.Enabled)
+      console.log('ui config: canRun returned false', this.siteSettings?.data.enableUI.fullscreen === ExtensionMode.Enabled, this.siteSettings?.data.enableUI.theater === ExtensionMode.Enabled, this.siteSettings?.data.enableUI.normal === ExtensionMode.Enabled)
       return;
     }
-    // console.log('ui config: canRun returned truie', this.siteSettings?.data.enableUI.fullscreen === ExtensionMode.Enabled, this.siteSettings?.data.enableUI.theater === ExtensionMode.Enabled, this.siteSettings?.data.enableUI.normal === ExtensionMode.Enabled)
+    console.log('ui config: canRun returned truie', this.siteSettings?.data.enableUI.fullscreen === ExtensionMode.Enabled, this.siteSettings?.data.enableUI.theater === ExtensionMode.Enabled, this.siteSettings?.data.enableUI.normal === ExtensionMode.Enabled)
 
 
     this.initUIContainer();
@@ -190,7 +190,6 @@ class UI {
           coords,
           playerDimensions: playerData.playerDimensions,
           canShowUI: playerData.canShowUI,
-          isIframe: this.isIframe,
           ts: +new Date()   // this should be accurate enough for our purposes,
         },
         uiURI
@@ -254,6 +253,7 @@ class UI {
         },
         'uw-get-page-stats': {
           function: (config, routingData) => {
+            console.log('got get page stats!');
             this.eventBus.send(
               'uw-page-stats',
               {

src/ext/lib/video-data/PageInfo.ts

@@ -111,6 +111,15 @@ class PageInfo {
     this.scheduleUrlCheck();
 
     document.addEventListener('fullscreenchange', this.fsEventListener);
+
+    this.eventBus.subscribeMulti({
+      'probe-video': {
+        function: () => {
+          console.log(`[${window.location}] probe-video received.`)
+          this.rescan();
+        }
+      }
+    });
   }
 
   destroy() {
@@ -398,20 +407,17 @@ class PageInfo {
 
   scheduleUrlCheck() {
     try{
-      if(this.urlCheckTimer){
+    if(this.urlCheckTimer){
-        clearTimeout(this.urlCheckTimer);
+      clearTimeout(this.urlCheckTimer);
-      }
+    }
 
-      let ths = this;
+    let ths = this;
 
-      this.urlCheckTimer = setTimeout(
+    this.urlCheckTimer = setTimeout(function(){
-        function(){
+      ths.urlCheckTimer = null;
-          ths.urlCheckTimer = null;
+      ths.ghettoUrlCheck();
-          ths.ghettoUrlCheck();
+      ths = null;
-          ths = null;
+    }, this.settings.active.pageInfo.timeouts.urlCheck)
-        },
-        this.settings.active.pageInfo.timeouts.urlCheck
-      );
     } catch(e){
       this.logger.log('scheduleUrlCheck', "scheduling URL check failed. Here's why:",e)
     }
@@ -422,7 +428,6 @@ class PageInfo {
       this.logger.warn('ghettoUrlCheck', "URL has changed. Triggering a rescan!");
 
       this.rescan(RescanReason.URL_CHANGE);
-      this.eventBus.send("url-changed", {newUrl: window.location.href});
       this.lastUrl = window.location.href;
     }
 

src/ext/lib/video-data/PlayerData.ts

@@ -206,7 +206,7 @@ class PlayerData {
       // do the rest
       this.invalid = false;
       this.updatePlayer();
-      this.isTooSmall = this.checkIfTooSmall();;
+      this.isTooSmall = (this.element.clientWidth < 1208 || this.element.clientHeight < 720);
 
       this.initEventBus();
       this.dimensions = undefined;
@@ -236,20 +236,6 @@ class PlayerData {
     }
   }
 
-  /**
-   * Checks if player dimensions are too small for autodetection to run.
-   * Full screen is never too small, even if it's
-   * @param playerDimensions
-   * @returns
-   */
-  private checkIfTooSmall(playerDimensions?: PlayerDimensions) {
-    if (playerDimensions) {
-      return !playerDimensions.fullscreen && (playerDimensions.width < 1208 || playerDimensions.height < 720);
-    } else {
-      return !document.fullscreenElement && (this.element.clientWidth < 1208 || this.element.clientHeight < 720);
-    }
-  }
-
   private reloadPlayerDataConfig(siteConfUpdate) {
     // this.siteSettings = siteConfUpdate;
     this.updatePlayer();
@@ -503,7 +489,7 @@ class PlayerData {
       });
 
 
-      this.isTooSmall = this.checkIfTooSmall(newDimensions);
+      this.isTooSmall = !newDimensions.fullscreen && (newDimensions.width < 1208 || newDimensions.height < 720);
     }
   }
 

src/manifest.json

@@ -2,7 +2,7 @@
   "manifest_version": 3,
   "name": "Ultrawidify",
   "description": "Removes black bars on ultrawide videos and offers advanced options to fix aspect ratio.",
-  "version": "6.3.93",
+  "version": "6.3.9",
   "icons": {
     "32":"res/icons/uw-32.png",
     "64":"res/icons/uw-64.png"