Fix finding onConsoleMessage class

Fix support for the Via browser
JingMatrix · Mar 12, 2024 · 72b8462 · 72b8462
1 parent 7336de1
commit 72b8462
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/app/src/main/java/org/matrix/chromext/hook/WebView.kt b/app/src/main/java/org/matrix/chromext/hook/WebView.kt
@@ -34,9 +34,11 @@ object WebViewHook : BaseHook() {
 
   override fun init() {
 
-    findMethod(ChromeClient!!, true) { name == "onConsoleMessage" }
+    findMethod(ChromeClient!!, true) { name == "onConsoleMessage" && parameterCount == 1 }
         // public boolean onConsoleMessage (ConsoleMessage consoleMessage)
         .hookAfter {
+          // Don't use ConsoleMessage to specify this method since Mi Browser uses its own
+          // implementation
           // This should be the way to communicate with the front-end of ChromeXt
           val chromeClient = it.thisObject
           val consoleMessage = it.args[0]