Chrome扩展中if语句内sendResponse失效，疑因indexOf错误导致

我帮你梳理下代码里的核心问题，以及对应的修复方案：

1. 监听器嵌套导致的重复绑定bug #

你把chrome.runtime.onMessage监听器放在了chrome.tabs.onUpdated的回调里，这会导致每次标签页更新时，都会新增一个消息监听器。当消息发送过来时，多个监听器会同时触发，sendResponse的调用就会混乱（甚至可能因为已经被调用过而直接失效）。

正确的做法是把消息监听器放在全局作用域，只绑定一次就够了。

2. indexOf的逻辑误用 #

你提到怀疑indexOf的问题，这里要明确：string.indexOf(substring)找不到匹配项时返回的是-1，而不是false。如果你的判断逻辑是if (url.indexOf("xxx"))，那当url开头就是目标字符串时，indexOf返回0，会被当成false处理，直接跳过判断分支，自然不会执行对应的sendResponse。

修复后的完整代码示例 #

// 扩展初始化逻辑
chrome.runtime.onInstalled.addListener(function () {
    console.log('扩展已加载完成');
});

// 全局消息监听器，只绑定一次
chrome.runtime.onMessage.addListener(function (request, sender, sendResponse) {
    // 获取当前活跃标签页的URL
    chrome.tabs.query({ 'active': true, 'lastFocusedWindow': true }, function (tabs) {
        // 先做边界判断，避免tabs为空或无url的情况
        if (!tabs[0] || !tabs[0].url) {
            sendResponse({ status: 'error', msg: '无法获取当前标签页URL' });
            return;
        }
        
        const currentUrl = tabs[0].url;
        console.log('当前标签页URL:', currentUrl);

        // 正确的indexOf判断方式：用 !== -1 确认包含目标字符串
        if (currentUrl.indexOf('safe-domain.com') !== -1) {
            sendResponse({ status: 'safe', msg: '该网站已验证为安全' });
        } else if (currentUrl.indexOf('risk-domain.com') !== -1) {
            sendResponse({ status: 'risk', msg: '该网站存在安全风险' });
        } else {
            sendResponse({ status: 'unknown', msg: '暂无法识别该网站安全性' });
        }
    });

    // 因为在异步回调中调用sendResponse，必须返回true告知Chrome保留消息通道
    return true;
});

// 标签页更新的独立逻辑（如果需要的话）
chrome.tabs.onUpdated.addListener(function (tabId, changeInfo, tab) {
    // 可以在这里做标签页更新后的其他操作，比如记录URL
    if (changeInfo.status === 'complete' && tab.active) {
        console.log('标签页更新完成，URL:', tab.url);
    }
});

额外注意点 #

• 异步回调里的sendResponse：如果在chrome.tabs.query这类异步操作的回调中调用sendResponse，必须在onMessage的回调末尾返回true，让Chrome保持消息通道开放，直到sendResponse执行完毕。
• 边界判断：处理tabs数据时先做非空校验，避免因为标签页状态异常导致的报错。

内容的提问来源于stack exchange，提问作者Dan H