Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: safari/ext/background.js

Issue 29340749: Issue 3816 - Add hidden toggle for the new options page (Closed)
Patch Set: Created April 22, 2016, 9:32 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « popup.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: safari/ext/background.js
diff --git a/safari/ext/background.js b/safari/ext/background.js
index 38870c0564b31fca093a5cb0c36419a0063014a7..5b8d32d0cc54001ab6d04dfe382ab3d245d5717b 100644
--- a/safari/ext/background.js
+++ b/safari/ext/background.js
@@ -563,9 +563,10 @@
/* Options */
- ext.showOptions = function(callback)
+ ext.showOptions = function(newOptionsPage, callback)
{
- var optionsUrl = safari.extension.baseURI + "options.html";
+ var optionsUrl = safari.extension.baseURI +
+ (newOptionsPage ? "new-" : "") + "options.html";
for (var id in pages)
{
« no previous file with comments | « popup.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld