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

Unified Diff: lib/icon.js

Issue 29363445: Issue 4577 - Remove Safari support (Closed)
Patch Set: Created Nov. 17, 2016, 11:36 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 | « include.preload.js ('k') | lib/polyfills/promise.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/icon.js
diff --git a/lib/icon.js b/lib/icon.js
index 0d4fae007eb8ae645a9afd564d0e7254f9c73549..621f374eeb9f9c247c8e8d45699ad4632e7319db 100644
--- a/lib/icon.js
+++ b/lib/icon.js
@@ -25,7 +25,6 @@ const frameOpacities = [0.0, 0.1, 0.2, 0.3, 0.4, 0.5, 0.6, 0.7, 0.8, 0.9,
1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0,
0.9, 0.8, 0.7, 0.6, 0.5, 0.4, 0.3, 0.2, 0.1, 0.0];
const numberOfFrames = frameOpacities.length;
-const safariPlatform = require("info").platform == "safari";
let stopRequested = false;
let canUpdateIcon = true;
@@ -52,7 +51,7 @@ function loadImage(url)
function setIcon(page, notificationType, opacity, frames)
{
opacity = opacity || 0;
- let whitelisted = !!whitelistedState.get(page) && !safariPlatform;
+ let whitelisted = !!whitelistedState.get(page);
if (!notificationType || !frames)
{
@@ -81,9 +80,6 @@ FilterNotifier.on("page.WhitelistingStateRevalidate", (page, filter) =>
function renderFrames(notificationType)
{
- if (safariPlatform)
- return Promise.resolve(null);
-
return Promise.all([
loadImage("icons/abp-16.png"),
loadImage("icons/abp-16-whitelisted.png"),
« no previous file with comments | « include.preload.js ('k') | lib/polyfills/promise.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld