commit 2834f2ffcf4ece0a49bdd666f4cfcf4c33487600
author Mike Dougherty <michaeldo@chromium.org> Sat Oct 18 06:05:56 2025
committer Copybara-Service <copybara-worker@google.com> Sat Oct 18 06:13:54 2025
tree d1a986b140b013c825db319b783dc1bdc2a21e21
parent 9218df621f4d6e3ecb11b9cb409e4f8ebb3d5f1f

Send JS reports when kLogCrWebJavaScriptErrors is enabled

Bug: 40269960
Change-Id: I0747978b4474f3a2b8fd17e2e7096b36525c169b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7056009
Reviewed-by: Sergio Collazos <sczs@chromium.org>
Commit-Queue: Mike Dougherty <michaeldo@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1531770}
NOKEYCHECK=True
GitOrigin-RevId: fda833dc52c236990841eb3139b4f8f2450f930a

web/js_features/window_error/script_error_message_handler_java_script_feature.mm

diff --git a/web/js_features/window_error/script_error_message_handler_java_script_feature.mm b/web/js_features/window_error/script_error_message_handler_java_script_feature.mm
index 9f63572..54273e7 100644
--- a/web/js_features/window_error/script_error_message_handler_java_script_feature.mm
+++ b/web/js_features/window_error/script_error_message_handler_java_script_feature.mm
@@ -99,8 +99,11 @@
     details.url = script_message.request_url().value();
   }
 
-  if (base::FeatureList::IsEnabled(features::kLogJavaScriptErrors) &&
-      log_message && stack && !is_crweb) {
+  if (log_message && stack &&
+      ((base::FeatureList::IsEnabled(features::kLogJavaScriptErrors) &&
+        !is_crweb) ||
+       (base::FeatureList::IsEnabled(features::kLogCrWebJavaScriptErrors) &&
+        is_crweb))) {
     script_error_stack_util::FrameComponents top_stack_frame =
         script_error_stack_util::TopFrameComponentsFromStack(*stack);
 

web/js_messaging/web_frame_impl.mm

diff --git a/web/js_messaging/web_frame_impl.mm b/web/js_messaging/web_frame_impl.mm
index aa0bc7b..9c70f04 100644
--- a/web/js_messaging/web_frame_impl.mm
+++ b/web/js_messaging/web_frame_impl.mm
@@ -291,7 +291,8 @@
 
   UMA_HISTOGRAM_BOOLEAN("IOS.JavaScript.ScriptExecutionFailed", true);
 
-  if (!base::FeatureList::IsEnabled(features::kLogJavaScriptErrors)) {
+  if (!base::FeatureList::IsEnabled(features::kLogJavaScriptErrors) &&
+      !base::FeatureList::IsEnabled(features::kLogCrWebJavaScriptErrors)) {
     return;
   }