[Pkg-javascript-commits] [pdf.js] 23/109: Don't reload after updating if a tab is in use

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository pdf.js.

commit 32e7a8adba33407ceff3c3520a76f063cc319b86
Author: Rob Wu <rob at robwu.nl>
Date:   Mon Jul 20 13:10:51 2015 +0200

    Don't reload after updating if a tab is in use
---
 extensions/chromium/pdfHandler.html    |  1 +
 extensions/chromium/suppress-update.js | 29 +++++++++++++++++++++++++++++
 2 files changed, 30 insertions(+)

diff --git a/extensions/chromium/pdfHandler.html b/extensions/chromium/pdfHandler.html
index 02a82fa..a9bdca1 100644
--- a/extensions/chromium/pdfHandler.html
+++ b/extensions/chromium/pdfHandler.html
@@ -22,3 +22,4 @@ limitations under the License.
 <script src="pdfHandler-v2.js"></script>
 <script src="pdfHandler-vcros.js"></script>
 <script src="pageAction/background.js"></script>
+<script src="suppress-update.js"></script>
diff --git a/extensions/chromium/suppress-update.js b/extensions/chromium/suppress-update.js
new file mode 100644
index 0000000..0f985f8
--- /dev/null
+++ b/extensions/chromium/suppress-update.js
@@ -0,0 +1,29 @@
+/* -*- Mode: Java; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
+/* vim: set shiftwidth=2 tabstop=2 autoindent cindent expandtab: */
+/*
+Copyright 2015 Mozilla Foundation
+
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+/* globals chrome */
+
+'use strict';
+
+// Do not reload the extension when an update becomes available, UNLESS the PDF
+// viewer is not displaying any PDF files. Otherwise the tabs would close, which
+// is quite disruptive (crbug.com/511670).
+chrome.runtime.onUpdateAvailable.addListener(function() {
+    if (chrome.extension.getViews({type: 'tab'}).length === 0) {
+        chrome.runtime.reload();
+    }
+});

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/pdf.js.git