Merge pull request #5691 from Snuffleupagus/viewer-shadow

Use the |shadow| function from src/shared/util.js in viewer.js

src/shared/util.js

@@ -325,6 +325,7 @@ function shadow(obj, prop, value) {
                                      writable: false });
   return value;
 }
+PDFJS.shadow = shadow;
 
 var PasswordResponses = PDFJS.PasswordResponses = {
   NEED_PASSWORD: 1,

web/viewer.js

@@ -299,12 +299,8 @@ var PDFViewerApplication = {
   get supportsPrinting() {
     var canvas = document.createElement('canvas');
     var value = 'mozPrintCallback' in canvas;
-    // shadow
+
-    Object.defineProperty(this, 'supportsPrinting', { value: value,
+    return PDFJS.shadow(this, 'supportsPrinting', value);
-                                                      enumerable: true,
-                                                      configurable: true,
-                                                      writable: false });
-    return value;
   },
 
   get supportsFullscreen() {
@@ -319,59 +315,40 @@ var PDFViewerApplication = {
       support = false;
     }
 
-    Object.defineProperty(this, 'supportsFullscreen', { value: support,
+    return PDFJS.shadow(this, 'supportsFullscreen', support);
-                                                        enumerable: true,
-                                                        configurable: true,
-                                                        writable: false });
-    return support;
   },
 
   get supportsIntegratedFind() {
     var support = false;
-//#if !(FIREFOX || MOZCENTRAL)
+//#if (FIREFOX || MOZCENTRAL)
-//#else
 //  support = FirefoxCom.requestSync('supportsIntegratedFind');
 //#endif
-    Object.defineProperty(this, 'supportsIntegratedFind', { value: support,
+
-                                                            enumerable: true,
+    return PDFJS.shadow(this, 'supportsIntegratedFind', support);
-                                                            configurable: true,
-                                                            writable: false });
-    return support;
   },
 
   get supportsDocumentFonts() {
     var support = true;
-//#if !(FIREFOX || MOZCENTRAL)
+//#if (FIREFOX || MOZCENTRAL)
-//#else
 //  support = FirefoxCom.requestSync('supportsDocumentFonts');
 //#endif
-    Object.defineProperty(this, 'supportsDocumentFonts', { value: support,
+
-                                                           enumerable: true,
+    return PDFJS.shadow(this, 'supportsDocumentFonts', support);
-                                                           configurable: true,
-                                                           writable: false });
-    return support;
   },
 
   get supportsDocumentColors() {
     var support = true;
-//#if !(FIREFOX || MOZCENTRAL)
+//#if (FIREFOX || MOZCENTRAL)
-//#else
 //  support = FirefoxCom.requestSync('supportsDocumentColors');
 //#endif
-    Object.defineProperty(this, 'supportsDocumentColors', { value: support,
+
-                                                            enumerable: true,
+    return PDFJS.shadow(this, 'supportsDocumentColors', support);
-                                                            configurable: true,
-                                                            writable: false });
-    return support;
   },
 
   get loadingBar() {
     var bar = new ProgressBar('#loadingBar', {});
-    Object.defineProperty(this, 'loadingBar', { value: bar,
+
-                                                enumerable: true,
+    return PDFJS.shadow(this, 'loadingBar', bar);
-                                                configurable: true,
-                                                writable: false });
-    return bar;
   },
 
 //#if (FIREFOX || MOZCENTRAL)