From e347ad03d97599588aa9be0f134f665f4b7b46de Mon Sep 17 00:00:00 2001
From: Vivien Nicolas <21@vingtetun.org>
Date: Tue, 18 Oct 2011 01:34:44 +0200
Subject: [PATCH 1/4] Fix a small broking mistake in fonts.js

---
 fonts.js | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/fonts.js b/fonts.js
index bcec5d2ac..2bcfa5e6e 100644
--- a/fonts.js
+++ b/fonts.js
@@ -2613,14 +2613,13 @@ var Type2CFF = (function type2CFF() {
         var glyph = charsets[i];
         var code = glyphMap[glyph] || 0;
 
-        var mapping = glyphs[code] || glyphs[glyph] || {};
+        var mapping = glyphs[code] || glyphs[glyph] || { width: defaultWidth };
         var unicode = mapping.unicode;
 
         if (unicode <= 0x1f || (unicode >= 127 && unicode <= 255))
           unicode += kCmapGlyphOffset;
 
-        var width = ('width' in mapping) && isNum(mapping.width) ? mapping.width
-                                                                 : defaultWidth;
+        var width = isNum(mapping.width) ? mapping.width : defaultWidth;
         properties.encoding[code] = {
           unicode: unicode,
           width: width

From b8a5b76db5cd20dfba5891e9f70de27fe46aa46f Mon Sep 17 00:00:00 2001
From: Vivien Nicolas <21@vingtetun.org>
Date: Tue, 18 Oct 2011 03:45:07 +0200
Subject: [PATCH 2/4] Fix a spacing issue

---
 fonts.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fonts.js b/fonts.js
index 2bcfa5e6e..2ba6e2f7a 100644
--- a/fonts.js
+++ b/fonts.js
@@ -1329,7 +1329,7 @@ var Font = (function Font() {
 
         // Character to glyphs mapping
         'cmap': createCMapTable(charstrings.slice(),
-                                ('glyphIds' in font) ? font.glyphIds: null),
+                                ('glyphIds' in font) ? font.glyphIds : null),
 
         // Font header
         'head': (function fontFieldsHead() {

From 20efacb3a1e2c137e7f7ef934bd8590331b21ece Mon Sep 17 00:00:00 2001
From: Vivien Nicolas <21@vingtetun.org>
Date: Tue, 18 Oct 2011 21:40:59 +0200
Subject: [PATCH 3/4] Use 'hidden' instead of display: none

---
 web/viewer.css  | 4 ++++
 web/viewer.html | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/web/viewer.css b/web/viewer.css
index 6c7b551e8..040b87bdf 100644
--- a/web/viewer.css
+++ b/web/viewer.css
@@ -8,6 +8,10 @@ body {
   padding: 0px;
 }
 
+[hidden] {
+  display: none;
+}
+
 /* === Toolbar === */
 #controls {
   background-color: #eee;
diff --git a/web/viewer.html b/web/viewer.html
index f761c94fe..4175d29fc 100644
--- a/web/viewer.html
+++ b/web/viewer.html
@@ -82,7 +82,7 @@
         <div id="sidebarScrollView">
           <div id="sidebarView"></div>
         </div>
-        <div id="outlineScrollView" style="display:none">
+        <div id="outlineScrollView" hidden='true'>
           <div id="outlineView"></div>
         </div>
         <div id="sidebarControls">

From 010b59cc54eb57dbe79f09b1447e56106ead2f77 Mon Sep 17 00:00:00 2001
From: Vivien Nicolas <21@vingtetun.org>
Date: Tue, 18 Oct 2011 23:39:26 +0200
Subject: [PATCH 4/4] Use 'hidden' instead of display: none

---
 web/viewer.js | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/web/viewer.js b/web/viewer.js
index 0808fbce9..071b7a1af 100644
--- a/web/viewer.js
+++ b/web/viewer.js
@@ -186,7 +186,7 @@ var PDFView = {
 
   load: function pdfViewLoad(data, scale) {
     var loadingIndicator = document.getElementById('loading');
-    loadingIndicator.style.display = 'none';
+    loadingIndicator.setAttribute('hidden', 'true');
 
     var sidebar = document.getElementById('sidebarView');
     sidebar.parentNode.scrollTop = 0;
@@ -282,14 +282,14 @@ var PDFView = {
     var outlineSwitchButton = document.getElementById('outlineSwitch');
     switch (view) {
       case 'thumbs':
-        thumbsScrollView.style.display = 'block';
-        outlineScrollView.style.display = 'none';
+        thumbsScrollView.removeAttribute('hidden');
+        outlineScrollView.setAttribute('hidden', 'true');
         thumbsSwitchButton.setAttribute('data-selected', true);
         outlineSwitchButton.removeAttribute('data-selected');
         break;
       case 'outline':
-        thumbsScrollView.style.display = 'none';
-        outlineScrollView.style.display = 'block';
+        thumbsScrollView.setAttribute('hidden', 'true');
+        outlineScrollView.removeAttribute('hidden');
         thumbsSwitchButton.removeAttribute('data-selected');
         outlineSwitchButton.setAttribute('data-selected', true);
         break;
@@ -592,7 +592,7 @@ window.addEventListener('load', function webViewerLoad(evt) {
   PDFView.open(params.file || kDefaultURL, parseFloat(scale));
 
   if (!window.File || !window.FileReader || !window.FileList || !window.Blob)
-    document.getElementById('fileInput').style.display = 'none';
+    document.getElementById('fileInput').setAttribute('hidden', 'true');
   else
     document.getElementById('fileInput').value = null;
 }, true);
@@ -677,7 +677,7 @@ window.addEventListener('change', function webViewerChange(evt) {
   document.title = file.name;
 
   // URL does not reflect proper document location - hiding bookmark icon.
-  document.getElementById('viewBookmark').style.display = 'none';
+  document.getElementById('viewBookmark').setAttribute('hidden', 'true');
 }, true);
 
 window.addEventListener('transitionend', function webViewerTransitionend(evt) {