chrome/test/data/extensions/subscribe_page_action/background.js - Issue 10407017: Convert RSS extension to use manifest version 2 (with CSP protection).

Unified Diff: chrome/test/data/extensions/subscribe_page_action/background.js

Issue 10407017: Convert RSS extension to use manifest version 2 (with CSP protection). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/

Patch Set: Created 8 years, 7 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

« no previous file with comments | « chrome/test/data/extensions/subscribe_page_action/background.html ('k') | chrome/test/data/extensions/subscribe_page_action/manifest.json » ('j') | chrome/test/data/extensions/subscribe_page_action/subscribe.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: chrome/test/data/extensions/subscribe_page_action/background.js

===================================================================

--- chrome/test/data/extensions/subscribe_page_action/background.js (revision 137495)

+++ chrome/test/data/extensions/subscribe_page_action/background.js (working copy)

@@ -1,62 +1,58 @@

-<!--

+/**

* source code is governed by a BSD-style license that can be found in the

* LICENSE file.

--->

-<html>

-<head>

-<script>

- // A dictionary keyed off of tabId that keeps track of data per tab (for

- // example what feedUrl was detected in the tab).

- var feedData = {};

+ */

- chrome.extension.onRequest.addListener(function(request, sender) {

- if (request.msg == "feedIcon") {

- // We have received a list of feed urls found on the page.

- // Enable the page action icon.

- feedData[sender.tab.id] = request.feeds;

- chrome.pageAction.setTitle(

- { tabId: sender.tab.id,

- title: chrome.i18n.getMessage("rss_subscription_action_title")});

- chrome.pageAction.show(sender.tab.id);

- } else if (request.msg == "feedDocument") {

- // We received word from the content script that this document

- // is an RSS feed (not just a document linking to the feed).

- // So, we go straight to the subscribe page in a new tab and

- // navigate back on the current page (to get out of the xml page).

- // We don't want to navigate in-place because trying to go back

- // from the subscribe page takes us back to the xml page, which

- // will redirect to the subscribe page again (we don't support a

- // location.replace equivalant in the Tab navigation system).

- chrome.tabs.executeScript(sender.tab.id,

- {code: "if (history.length > 1) " +

- "history.go(-1); else window.close();"});

- var url = "subscribe.html?" + encodeURIComponent(request.href);

- url = chrome.extension.getURL(url);

- chrome.tabs.create({url: url, index: sender.tab.index});

+// A dictionary keyed off of tabId that keeps track of data per tab (for

+// example what feedUrl was detected in the tab).

+var feedData = {};

+chrome.extension.onRequest.addListener(function(request, sender) {

+ if (request.msg == "feedIcon") {

+ // First validate that all the URLs have the right schema.

+ var input = [];

+ for (var i = 0; i < request.feeds.length; ++i) {

+ var a = document.createElement('a');

+ a.href = request.feeds[i].href;

+ if (a.protocol == "http:" || a.protocol == "https:") {

+ input.push(request.feeds[i]);

+ } else {

+ console.log('Warning: feed source rejected (wrong protocol): ' +

+ request.feeds[i].href);

+ }

}

- });

- chrome.tabs.onRemoved.addListener(function(tabId) {

- delete feedData[tabId];

- });

+ if (input.length == 0)

+ return; // We've rejected all the input, so abort.

- // On Linux, popups aren't supported yet, so Chrome will call into us

- // when the user clicks on the icon in the OmniBox.

- chrome.pageAction.onClicked.addListener(function(tab) {

- chrome.windows.get(tab.windowId, function(window) {

- // We need to know if we are the active window, because the tab may

- // have moved to another window and we don't want to execute this

- // action multiple times.

- if (window.focused) {

- // Create a new tab showing the subscription page with the right

- // feed URL.

- var url = "subscribe.html?" +

- encodeURIComponent(feedData[tab.id][0].href);

- chrome.tabs.create({url: url, windowId: window.id});

- }

- });

-</script>

-</head>

-</html>

+ // We have received a list of feed urls found on the page.

+ // Enable the page action icon.

+ feedData[sender.tab.id] = input;

+ chrome.pageAction.setTitle(

+ { tabId: sender.tab.id,

+ title: chrome.i18n.getMessage("rss_subscription_action_title")

+ });

+ chrome.pageAction.show(sender.tab.id);

+ } else if (request.msg == "feedDocument") {

+ // We received word from the content script that this document

+ // is an RSS feed (not just a document linking to the feed).

+ // So, we go straight to the subscribe page in a new tab and

+ // navigate back on the current page (to get out of the xml page).

+ // We don't want to navigate in-place because trying to go back

+ // from the subscribe page takes us back to the xml page, which

+ // will redirect to the subscribe page again (we don't support a

+ // location.replace equivalant in the Tab navigation system).

+ chrome.tabs.executeScript(sender.tab.id,

+ { code: "if (history.length > 1) " +

+ "history.go(-1); else window.close();"

+ });

+ var url = "subscribe.html?" + encodeURIComponent(request.href);

+ url = chrome.extension.getURL(url);

+ chrome.tabs.create({ url: url, index: sender.tab.index });

+ }

+});

+chrome.tabs.onRemoved.addListener(function(tabId) {

+ delete feedData[tabId];

+});