Improve flash hidden element detection

app/extensions/brave/content/scripts/flashListener.js

@@ -71,12 +71,23 @@ function hasHiddenFlashElement (elem) {
   })
 }
 
-
 // If Flash is enabled but not runnable, show a permission notification for small
 // Flash elements
-if (chrome.contentSettings.flashEnabled == 'allow' && chrome.contentSettings.plugins != 'allow' && hasHiddenFlashElement(document.documentElement)) {
-  chrome.ipcRenderer.send('dispatch-action', JSON.stringify([{
-    actionType: 'app-flash-permission-requested',
-    location: window.location.href
-  }]))
+if (chrome.contentSettings.flashEnabled == 'allow' && chrome.contentSettings.flashAllowed != 'allow') {
+  const maxFlashAttempts = 3
+  let flashAttempts = 0
+  const intervalId = window.setInterval(() => {
+    if (flashAttempts >= maxFlashAttempts) {
+      window.clearInterval(intervalId)
+      return
+    }
+    flashAttempts = flashAttempts + 1
+    if (hasHiddenFlashElement(document.documentElement)) {
+      chrome.ipcRenderer.send('dispatch-action', JSON.stringify([{
+        actionType: 'app-flash-permission-requested',
+        location: window.location.href
+      }]))
+      window.clearInterval(intervalId)
+    }
+  }, 1000)
 }

js/state/contentSettings.js

@@ -97,10 +97,14 @@ const getDefaultUserPrefContentSettings = (braveryDefaults, appSettings, appConf
       setting: 'block',
       primaryPattern: '*'
     }],
-    flashEnabled: [{
+    flashEnabled: [{ // whether flash is installed and enabled
       setting: braveryDefaults.get('flash') ? 'allow' : 'block',
       primaryPattern: '*'
     }],
+    flashAllowed: [{ // whether user has expressed intent to run flash
+      setting: 'block',
+      primaryPattern: '*'
+    }],
     popups: [{
       setting: 'block',
       primaryPattern: '*'
@@ -268,6 +272,7 @@ const siteSettingsToContentSettings = (currentSiteSettings, defaultContentSettin
     }
     if (typeof siteSetting.get('flash') === 'number' && braveryDefaults.get('flash')) {
       contentSettings = addContentSettings(contentSettings, 'plugins', primaryPattern, '*', 'allow', getFlashResourceId())
+      contentSettings = addContentSettings(contentSettings, 'flashAllowed', primaryPattern, '*', 'allow', getFlashResourceId())
     }
     if (typeof siteSetting.get('widevine') === 'number' && braveryDefaults.get('widevine')) {
       contentSettings = addContentSettings(contentSettings, 'plugins', primaryPattern, '*', 'allow', appConfig.widevine.resourceId)