Remove permission exception for brave and pdf viewer

quick fix for
#14679 (comment)
by just removing the special exception for these origins. this probably
breaks something but i'm not sure what.

app/filtering.js

@@ -549,14 +549,16 @@ function registerPermissionHandler (session, partition) {
         requestingOrigin = isPDFOrigin ? 'PDF Viewer' : 'Brave Browser'
         // display on all tabs
         mainFrameOrigin = null
-      } else if (isOpenExternal) {
+      } else {
+        requestingOrigin = getOrigin(requestingUrl) || requestingUrl
+      }
+
+      if (isOpenExternal) {
         // Open external is a special case since we want to apply the permission
         // for the entire scheme to avoid cluttering the saved permissions. See
         // https://github.com/brave/browser-laptop/issues/13642
         const protocol = urlParse(requestingUrl).protocol
         requestingOrigin = protocol ? `${protocol} URLs` : requestingUrl
-      } else {
-        requestingOrigin = getOrigin(requestingUrl) || requestingUrl
       }
 
       // Look up by host pattern since requestingOrigin is not necessarily
@@ -577,10 +579,6 @@ function registerPermissionHandler (session, partition) {
       } else if (isFullscreen && alwaysAllowFullscreen) {
         // Always allow fullscreen if setting is ON
         response.push(true)
-      } else if (isOpenExternal && (
-        // The Brave extension and PDFJS are always allowed to open files in an external app
-        isPDFOrigin || isBraveOrigin)) {
-        response.push(true)
       } else {
         const permissionName = permission + 'Permission'
         let isAllowed