5

我是第一次开发附加组件。它在状态栏中放置了一个小部件，显示未读谷歌阅读器项目的数量。为了适应这种情况，插件进程每分钟查询一次 Google Reader API 并将响应传递给小部件。当我运行时，cfx test我收到此错误：

错误：页面已被破坏，无法再使用。

我确保捕获小部件的detach事件并停止刷新计时器作为响应，但我仍然看到错误。我究竟做错了什么？以下是相关代码：

// main.js - Main entry point
const tabs = require('tabs');
const widgets = require('widget');
const data = require('self').data;
const timers = require("timers");
const Request = require("request").Request;

function refreshUnreadCount() {
    // Put in Google Reader API request
    Request({
        url: "https://www.google.com/reader/api/0/unread-count?output=json",
        onComplete: function(response) {
            // Ignore response if we encountered a 404 (e.g. user isn't logged in)
            // or a different HTTP error.
            // TODO: Can I make this work when third-party cookies are disabled?
            if (response.status == 200) {
                monitorWidget.postMessage(response.json);
            } else {
                monitorWidget.postMessage(null);
            }
        }
    }).get();
}

var monitorWidget = widgets.Widget({
    // Mandatory widget ID string
    id: "greader-monitor",

    // A required string description of the widget used for
    // accessibility, title bars, and error reporting.
    label: "GReader Monitor",
    contentURL: data.url("widget.html"),
    contentScriptFile: [data.url("jquery-1.7.2.min.js"), data.url("widget.js")],

    onClick: function() {
        // Open Google Reader when the widget is clicked.
        tabs.open("https://www.google.com/reader/view/");
    },

    onAttach: function(worker) {
        // If the widget's inner width changes, reflect that in the GUI
        worker.port.on("widthReported", function(newWidth) {
            worker.width = newWidth;
        });

        var refreshTimer = timers.setInterval(refreshUnreadCount, 60000);

        // If the monitor widget is destroyed, make sure the timer gets cancelled.
        worker.on("detach", function() {
            timers.clearInterval(refreshTimer);
        });

        refreshUnreadCount();
    }
});

// widget.js - Status bar widget script

// Every so often, we'll receive the updated item feed. It's our job
// to parse it.
self.on("message", function(json) {
    if (json == null) {
        $("span#counter").attr("class", "");
        $("span#counter").text("N/A");
    } else {
        var newTotal = 0;
        for (var item in json.unreadcounts) {
            newTotal += json.unreadcounts[item].count;
        }

        // Since the cumulative reading list count is a separate part of the
        // unread count info, we have to divide the total by 2.
        newTotal /= 2;
        $("span#counter").text(newTotal);

        // Update style
        if (newTotal > 0)
            $("span#counter").attr("class", "newitems");
        else
            $("span#counter").attr("class", "");
    }

    // Reports the current width of the widget
    self.port.emit("widthReported", $("div#widget").width());
});

编辑：我已经将整个项目上传到这个 GitHub 存储库。

4

3 回答 3

0

我认为如果您使用该方法monitorWidget.port.emit("widthReported", response.json);，您可以触发该事件。这是与内容脚本和附加脚本进行通信的第二种方式。

于 2012-09-13T19:55:22.867 回答
0

我刚刚在 irc 上看到了您的消息，感谢您报告您的问题。您在 SDK 中遇到了一些内部错误。我在这里打开了一个关于此的错误。

您绝对应该保留您向小部件发送消息的第一个版本的代码，即widget.postMessage（而不是worker.postMessage）。然后我们将不得不修复我链接到的错误，以使您的代码正常工作！！

然后我建议您将 setInterval 移到顶层，否则您将触发多个间隔和请求，每个窗口一个。attach每个新的 firefox 窗口都会触发此事件。

于 2012-09-10T13:22:29.837 回答
0

我想当你打电话时会出现这条monitorWidget.postMessage()消息refreshUnreadCount()。其明显的原因是：虽然您确保refreshUnreadCount()仅在工作人员仍处于活动状态时调用，但此函数将执行可能需要一段时间的异步请求。所以当这个请求完成时，worker 可能已经被销毁了。

一种解决方案是将工作者作为参数传递给refreshUnreadCount(). 然后，它可以添加自己的detach侦听器（在请求完成时将其删除），如果在执行请求时工作人员被分离，则忽略响应。

function refreshUnreadCount(worker) {
    var detached = false;
    function onDetach()
    {
        detached = true;
    }
    worker.on("detach", onDetach);

    Request({
        ...
        onComplete: function(response) {
            worker.removeListener("detach", onDetach);
            if (detached)
                return;  // Nothing to update with out data

            ...
        }
    }).get();
}

再说一次，使用try..catch来检测这种情况并抑制错误可能会更简单 - 但不是一个干净的解决方案。

于 2012-08-13T11:32:22.577 回答