From c414c76461355c76bfa4044b5836ab260043e504 Mon Sep 17 00:00:00 2001
From: Yury Delendik <ydelendik@mozilla.com>
Date: Fri, 17 Aug 2012 11:05:51 -0500
Subject: [PATCH 01/12] Fixes font loading concurency

---
 src/fonts.js | 143 +++++++++++++++++++++++++++++++--------------------
 1 file changed, 88 insertions(+), 55 deletions(-)

diff --git a/src/fonts.js b/src/fonts.js
index f0e1bd4d3..9a6dc37b5 100644
--- a/src/fonts.js
+++ b/src/fonts.js
@@ -404,28 +404,16 @@ function mapPrivateUseChars(code) {
 }
 
 var FontLoader = {
-  listeningForFontLoad: false,
+  loadingContext: {
+    pending: 0,
+    requests: [],
+    nextRequestId: 0
+  },
 
   bind: function fontLoaderBind(fonts, callback) {
-    function checkFontsLoaded() {
-      for (var i = 0, ii = fonts.length; i < ii; i++) {
-        var fontObj = fonts[i];
-        if (fontObj.loading) {
-          return false;
-        }
-      }
-
-      document.documentElement.removeEventListener(
-        'pdfjsFontLoad', checkFontsLoaded, false);
-
-      // Use timeout to fix chrome intermittent failures on font loading.
-      setTimeout(callback, 0);
-      return true;
-    }
-
-    var rules = [], names = [], fontsToLoad = [];
-    var fontCreateTimer = 0;
+    assert(!isWorker, 'bind() shall be called from main thread');
 
+    var rules = [], fontsToLoad = [];
     for (var i = 0, ii = fonts.length; i < ii; i++) {
       var font = fonts[i];
 
@@ -436,8 +424,6 @@ var FontLoader = {
       }
       font.attached = true;
 
-      fontsToLoad.push(font);
-
       var str = '';
       var data = font.data;
       if (data) {
@@ -448,28 +434,79 @@ var FontLoader = {
         var rule = font.bindDOM(str);
         if (rule) {
           rules.push(rule);
-          names.push(font.loadedName);
+          fontsToLoad.push(font);
         }
       }
     }
 
-    this.listeningForFontLoad = false;
-    if (!isWorker && rules.length) {
-      FontLoader.prepareFontLoadEvent(rules, names, fontsToLoad);
-    }
-
-    if (!checkFontsLoaded()) {
-      document.documentElement.addEventListener(
-        'pdfjsFontLoad', checkFontsLoaded, false);
+    var request = FontLoader.queueLoadingCallback(callback);
+    if (rules.length > 0) {
+      FontLoader.prepareFontLoadEvent(rules, fontsToLoad, request);
+    } else {
+      request.complete();
     }
   },
+
+  queueLoadingCallback: function FontLoader_queueLoadingCallback(callback) {
+    function LoadLoader_completeRequest() {
+      assert(!request.end, 'completeRequest() cannot be called twice');
+      request.end = Date.now();
+
+      if (context.pending <= 1) {
+        // it's simple completion for one request
+        context.pending = 0;
+        context.requests.pop();
+        callback();
+        return;
+      }
+
+      // calculating the load delay for all fonts and checking if all loaded
+      var totalTime = 0;
+      for (var i = 0, ii = context.requests.length; i < ii; i++) {
+        var otherRequest = context.requests[i];
+        if (!otherRequest.end)
+          return; // one more font to load, cancel the completion
+        totalTime += otherRequest.end - otherRequest.start;
+      }
+      var now = Date.now();
+      var startTime = context.requests[0].start;
+      var leftToWait = Math.max(totalTime - (now - startTime), 0);
+      context.timeout = setTimeout(function completeAllRequests() {
+        for (var i = 0, ii = context.requests.length; i < ii; i++) {
+          context.requests[i].callback();
+        }
+        context.pending = 0;
+        context.requests = [];
+        delete context.timeout;
+      }, leftToWait);
+    }
+
+    var context = FontLoader.loadingContext;
+    var requestId = 'pdfjs-font-loading-' + (context.nextRequestId++);
+    context.pending++;
+    var request = {
+      id: requestId,
+      complete: LoadLoader_completeRequest,
+      callback: callback,
+      started: Date.now()
+    };
+    context.requests.push(request);
+    if (context.timeout) {
+      // timeout for callbacks was set, removing that
+      clearTimeout(context.timeout);
+      delete context.timeout;
+    }
+    return request;
+  },
+
   // Set things up so that at least one pdfjsFontLoad event is
-  // dispatched when all the @font-face |rules| for |names| have been
+  // dispatched when all the @font-face |rules| for |fonts| have been
   // loaded in a subdocument.  It's expected that the load of |rules|
   // has already started in this (outer) document, so that they should
   // be ordered before the load in the subdocument.
-  prepareFontLoadEvent: function fontLoaderPrepareFontLoadEvent(rules, names,
-                                                                fonts) {
+  prepareFontLoadEvent: function fontLoaderPrepareFontLoadEvent(rules,
+                                                                fonts,
+                                                                request) {
       /** Hack begin */
       // There's no event when a font has finished downloading so the
       // following code is a dirty hack to 'guess' when a font is
@@ -493,6 +530,10 @@ var FontLoader = {
       // The postMessage() hackery was added to work around chrome bug
       // 82402.
 
+      var names = [];
+      for (var i = 0, ii = fonts.length; i < ii; i++)
+        names.push(fonts[i].loadedName);
+
       // Validate the names parameter -- the values can used to construct HTML.
       if (!/^\w+$/.test(names.join(''))) {
         error('Invalid font name(s): ' + names.join());
@@ -514,22 +555,19 @@ var FontLoader = {
       div.innerHTML = html;
       document.body.appendChild(div);
 
-      if (!this.listeningForFontLoad) {
-        window.addEventListener(
-          'message',
-          function fontLoaderMessage(e) {
-            var fontNames = JSON.parse(e.data);
-            for (var i = 0, ii = fonts.length; i < ii; ++i) {
-              var font = fonts[i];
-              font.loading = false;
-            }
-            var evt = document.createEvent('Events');
-            evt.initEvent('pdfjsFontLoad', true, false);
-            document.documentElement.dispatchEvent(evt);
-          },
-          false);
-        this.listeningForFontLoad = true;
-      }
+      var requestId = request.id;
+      window.addEventListener(
+        'message',
+        function fontLoaderMessage(e) {
+          if (e.data !== requestId)
+            return;
+          for (var i = 0, ii = fonts.length; i < ii; ++i) {
+            var font = fonts[i];
+            font.loading = false;
+          }
+          request.complete();
+        },
+        false);
 
       // XXX we should have a time-out here too, and maybe fire
       // pdfjsFontLoadFailed?
@@ -540,13 +578,8 @@ var FontLoader = {
       }
       src += '</style>';
       src += '<script type="application/javascript">';
-      var fontNamesArray = '';
-      for (var i = 0, ii = names.length; i < ii; ++i) {
-        fontNamesArray += '"' + names[i] + '", ';
-      }
-      src += '  var fontNames=[' + fontNamesArray + '];\n';
       src += '  window.onload = function fontLoaderOnload() {\n';
-      src += '    parent.postMessage(JSON.stringify(fontNames), "*");\n';
+      src += '    parent.postMessage("' + requestId + '", "*");\n';
       src += '  }';
       // Hack so the end script tag isn't counted if this is inline JS.
       src += '</scr' + 'ipt></head><body>';

From 3c1d63055d3dbb57cadb05006ba614d5e31b9491 Mon Sep 17 00:00:00 2001
From: Yury Delendik <ydelendik@mozilla.com>
Date: Fri, 17 Aug 2012 11:32:34 -0500
Subject: [PATCH 02/12] Invoking callback via setTimeout

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

diff --git a/src/fonts.js b/src/fonts.js
index 9a6dc37b5..421da92d9 100644
--- a/src/fonts.js
+++ b/src/fonts.js
@@ -456,7 +456,7 @@ var FontLoader = {
         // it's simple completion for one request
         context.pending = 0;
         context.requests.pop();
-        callback();
+        setTimeout(callback, 0);
         return;
       }
 

From 8459fef6c32b689e32d8094dfdbd6e8db3aa9910 Mon Sep 17 00:00:00 2001
From: Yury Delendik <ydelendik@mozilla.com>
Date: Fri, 17 Aug 2012 14:00:35 -0500
Subject: [PATCH 03/12] Cleanup: remove iframe and message handler

---
 src/fonts.js | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/fonts.js b/src/fonts.js
index 421da92d9..ef2051a93 100644
--- a/src/fonts.js
+++ b/src/fonts.js
@@ -566,6 +566,9 @@ var FontLoader = {
             font.loading = false;
           }
           request.complete();
+          // cleanup
+          document.body.removeChild(frame);
+          window.removeEventListener('message', fontLoaderMessage, false);
         },
         false);
 

From 52d229761a2f4eb001be3ebc8f0ceb0108eef275 Mon Sep 17 00:00:00 2001
From: Yury Delendik <ydelendik@mozilla.com>
Date: Fri, 17 Aug 2012 16:08:42 -0500
Subject: [PATCH 04/12] Removing sum of the iframe times, browser does it for
 us

---
 src/fonts.js | 37 ++++---------------------------------
 1 file changed, 4 insertions(+), 33 deletions(-)

diff --git a/src/fonts.js b/src/fonts.js
index ef2051a93..7f987c825 100644
--- a/src/fonts.js
+++ b/src/fonts.js
@@ -405,7 +405,6 @@ function mapPrivateUseChars(code) {
 
 var FontLoader = {
   loadingContext: {
-    pending: 0,
     requests: [],
     nextRequestId: 0
   },
@@ -452,38 +451,15 @@ var FontLoader = {
       assert(!request.end, 'completeRequest() cannot be called twice');
       request.end = Date.now();
 
-      if (context.pending <= 1) {
-        // it's simple completion for one request
-        context.pending = 0;
-        context.requests.pop();
-        setTimeout(callback, 0);
-        return;
+      // sending all completed requests in order how they were queued
+      while (context.requests.length > 0 && context.requests[0].end) {
+        var otherRequest = context.requests.shift();
+        setTimeout(otherRequest.callback, 0);
       }
-
-      // calculating the load delay for all fonts and checking if all loaded
-      var totalTime = 0;
-      for (var i = 0, ii = context.requests.length; i < ii; i++) {
-        var otherRequest = context.requests[i];
-        if (!otherRequest.end)
-          return; // one more font to load, cancel the completion
-        totalTime += otherRequest.end - otherRequest.start;
-      }
-      var now = Date.now();
-      var startTime = context.requests[0].start;
-      var leftToWait = Math.max(totalTime - (now - startTime), 0);
-      context.timeout = setTimeout(function completeAllRequests() {
-        for (var i = 0, ii = context.requests.length; i < ii; i++) {
-          context.requests[i].callback();
-        }
-        context.pending = 0;
-        context.requests = [];
-        delete context.timeout;
-      }, leftToWait);
     }
 
     var context = FontLoader.loadingContext;
     var requestId = 'pdfjs-font-loading-' + (context.nextRequestId++);
-    context.pending++;
     var request = {
       id: requestId,
       complete: LoadLoader_completeRequest,
@@ -491,11 +467,6 @@ var FontLoader = {
       started: Date.now()
     };
     context.requests.push(request);
-    if (context.timeout) {
-      // timeout for callbacks was set, removing that
-      clearTimeout(context.timeout);
-      delete context.timeout;
-    }
     return request;
   },
 

From 4f20f5f1bb6b07a40bb4246577a8746a2e577e61 Mon Sep 17 00:00:00 2001
From: Yury Delendik <ydelendik@mozilla.com>
Date: Fri, 17 Aug 2012 16:56:47 -0500
Subject: [PATCH 05/12] Verify the request id parameter

---
 src/fonts.js | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/fonts.js b/src/fonts.js
index 7f987c825..da9ab85f8 100644
--- a/src/fonts.js
+++ b/src/fonts.js
@@ -501,6 +501,16 @@ var FontLoader = {
       // The postMessage() hackery was added to work around chrome bug
       // 82402.
 
+      var requestId = request.id;
+      // Validate the requestId parameter -- the value used to construct HTML.
+      if (!/^[\w\-]+$/.test(requestId)) {
+        error('Invalid request id: ' + requestId);
+
+        // Normally the error-function throws. But if a malicious code
+        // intercepts the function call then the return is needed.
+        return;
+      }
+
       var names = [];
       for (var i = 0, ii = fonts.length; i < ii; i++)
         names.push(fonts[i].loadedName);
@@ -526,7 +536,6 @@ var FontLoader = {
       div.innerHTML = html;
       document.body.appendChild(div);
 
-      var requestId = request.id;
       window.addEventListener(
         'message',
         function fontLoaderMessage(e) {

From 2516be859bf3212d132beb22289e2de1d4ddde2e Mon Sep 17 00:00:00 2001
From: Kalervo Kujala <kkujala@com>
Date: Thu, 23 Aug 2012 21:20:21 +0300
Subject: [PATCH 06/12] Gjslint also the build related js files.

---
 make.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/make.js b/make.js
index df732ce1e..a42ae7fd2 100755
--- a/make.js
+++ b/make.js
@@ -697,7 +697,10 @@ target.lint = function() {
   echo();
   echo('### Linting JS files (this can take a while!)');
 
-  var LINT_FILES = ['src/*.js',
+  var LINT_FILES = ['make.js',
+                    'external/builder/*.js',
+                    'external/crlfchecker/*.js',
+                    'src/*.js',
                     'web/*.js',
                     'test/*.js',
                     'test/unit/*.js',

From ac0ae3e5fd94660bf90535bcdb12c544b9c3fd18 Mon Sep 17 00:00:00 2001
From: Yury Delendik <ydelendik@mozilla.com>
Date: Thu, 23 Aug 2012 16:12:33 -0500
Subject: [PATCH 07/12] Merges new jpgjs: fixes invalid JPEG marker.

---
 external/jpgjs/LICENSE | 31 ++++++++++++-------------------
 external/jpgjs/jpg.js  | 27 +++++++++++++++++++++++++--
 2 files changed, 37 insertions(+), 21 deletions(-)

diff --git a/external/jpgjs/LICENSE b/external/jpgjs/LICENSE
index f28619c6a..1e29c87c8 100644
--- a/external/jpgjs/LICENSE
+++ b/external/jpgjs/LICENSE
@@ -1,24 +1,17 @@
 
-Copyright (C) 2011 by notmasteryet
+   Copyright (C) 2011 notmasteryet
 
-Contributors: Yury Delendik
-              Brendan Dahl <bdahl@mozilla.com>
+   Contributors: Yury Delendik <ydelendik@mozilla.com>
+                 Brendan Dahl <bdahl@mozilla.com>
 
-Permission is hereby granted, free of charge, to any person obtaining
-a copy of this software and associated documentation files (the
-"Software"), to deal in the Software without restriction, including
-without limitation the rights to use, copy, modify, merge, publish,
-distribute, sublicense, and/or sell copies of the Software, and to
-permit persons to whom the Software is furnished to do so, subject to
-the following conditions:
+   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
 
-The above copyright notice and this permission notice shall be
-included in all copies or substantial portions of the Software.
+       http://www.apache.org/licenses/LICENSE-2.0
 
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
-EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
-MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
-NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
-LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
-OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
-WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+   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.
diff --git a/external/jpgjs/jpg.js b/external/jpgjs/jpg.js
index 103153d9d..941d01a87 100644
--- a/external/jpgjs/jpg.js
+++ b/external/jpgjs/jpg.js
@@ -1,5 +1,20 @@
 /* -*- Mode: Java; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*- /
 /* vim: set shiftwidth=2 tabstop=2 autoindent cindent expandtab: */
+/*
+   Copyright 2011 notmasteryet
+
+   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.
+*/
 
 // - The JPEG specification can be found in the ITU CCITT Recommendation T.81
 //   (www.w3.org/Graphics/JPEG/itu-t81.pdf)
@@ -627,8 +642,9 @@ var JpegImage = (function jpegImage() {
             break;
 
           case 0xFFDB: // DQT (Define Quantization Tables)
-            var quantizationTableCount = Math.floor((readUint16() - 2) / 65);
-            for (i = 0; i < quantizationTableCount; i++) {
+            var quantizationTablesLength = readUint16();
+            var quantizationTablesEnd = quantizationTablesLength + offset - 2;
+            while (offset < quantizationTablesEnd) {
               var quantizationTableSpec = data[offset++];
               var tableData = new Int32Array(64);
               if ((quantizationTableSpec >> 4) === 0) { // 8 bit values
@@ -721,6 +737,13 @@ var JpegImage = (function jpegImage() {
             offset += processed;
             break;
           default:
+            if (data[offset - 3] == 0xFF &&
+                data[offset - 2] >= 0xC0 && data[offset - 2] <= 0xFE) {
+              // could be incorrect encoding -- last 0xFF byte of the previous
+              // block was eaten by the encoder
+              offset -= 3;
+              break;
+            }
             throw "unknown JPEG marker " + fileMarker.toString(16);
         }
         fileMarker = readUint16();

From 2531515f15b455b4f0899a48c542e30ce6ff95eb Mon Sep 17 00:00:00 2001
From: Yury Delendik <ydelendik@mozilla.com>
Date: Fri, 24 Aug 2012 10:12:03 -0500
Subject: [PATCH 08/12] Replaces scrollIntoView

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

diff --git a/web/viewer.js b/web/viewer.js
index f93c5247b..213ae8cec 100644
--- a/web/viewer.js
+++ b/web/viewer.js
@@ -33,6 +33,17 @@ function getFileName(url) {
   return url.substring(url.lastIndexOf('/', end) + 1, end);
 }
 
+function scrollIntoView(element, spot) {
+  var parent = element.offsetParent, offsetY = element.offsetTop;
+  while (parent.clientHeight == parent.scrollHeight) {
+    offsetY += parent.offsetTop;
+    parent = parent.offsetParent;
+  }
+  if (spot)
+    offsetY += spot.top;
+  parent.scrollTop = offsetY;
+}
+
 var Cache = function cacheCache(size) {
   var data = [];
   this.push = function cachePush(view) {
@@ -1309,7 +1320,7 @@ var PageView = function pageView(container, pdfPage, id, scale,
 
   this.scrollIntoView = function pageViewScrollIntoView(dest) {
       if (!dest) {
-        div.scrollIntoView(true);
+        scrollIntoView(div);
         return;
       }
 
@@ -1368,16 +1379,7 @@ var PageView = function pageView(container, pdfPage, id, scale,
         var width = Math.abs(boundingRect[0][0] - boundingRect[1][0]);
         var height = Math.abs(boundingRect[0][1] - boundingRect[1][1]);
 
-        // using temporary div to scroll it into view
-        var tempDiv = document.createElement('div');
-        tempDiv.style.position = 'absolute';
-        tempDiv.style.left = Math.floor(x) + 'px';
-        tempDiv.style.top = Math.floor(y) + 'px';
-        tempDiv.style.width = Math.ceil(width) + 'px';
-        tempDiv.style.height = Math.ceil(height) + 'px';
-        div.appendChild(tempDiv);
-        tempDiv.scrollIntoView(true);
-        div.removeChild(tempDiv);
+        scrollIntoView(div, {left: x, top: y, width: width, height: height});
       }, 0);
   };
 
@@ -2078,7 +2080,7 @@ window.addEventListener('pagechange', function pagechange(evt) {
       var last = numVisibleThumbs > 1 ?
                   visibleThumbs.last.id : first;
       if (page <= first || page >= last)
-        thumbnail.scrollIntoView();
+        scrollIntoView(thumbnail);
     }
 
   }

From 1cbbb37fbc41a8044feeaea5c1cf7a68fc5691b6 Mon Sep 17 00:00:00 2001
From: Yury Delendik <ydelendik@mozilla.com>
Date: Mon, 27 Aug 2012 10:57:40 -0500
Subject: [PATCH 09/12] Making webkit fullscreen work as for moz

---
 web/viewer.css | 19 ++++---------------
 1 file changed, 4 insertions(+), 15 deletions(-)

diff --git a/web/viewer.css b/web/viewer.css
index 95691e818..439ec8ca3 100644
--- a/web/viewer.css
+++ b/web/viewer.css
@@ -29,28 +29,13 @@ select {
 
 #viewerContainer:-webkit-full-screen {
   top: 0px;
-  padding-top: 6px;
-  padding-bottom: 24px;
   background-color: #404040;
   background-image: url(images/texture.png);
   width: 100%;
   height: 100%;
-  overflow: auto;
-}
-
-:-webkit-full-screen #viewer {
-  margin: 0pt;
-  padding: 0pt;
-  height: 100%;
-  width: 100%;
   overflow: hidden;
 }
 
-:-webkit-full-screen .page {
-  margin: 0px auto;
-  margin-bottom: 10px;
-}
-
 #viewerContainer:-moz-full-screen {
   background-color: #404040;
   background-image: url(images/texture.png);
@@ -59,6 +44,10 @@ select {
   overflow: hidden;
 }
 
+:-webkit-full-screen .page:last-child {
+  margin-bottom: 40px;
+}
+
 :-moz-full-screen .page:last-child {
   margin-bottom: 40px;
 }

From df32b4393c5b993b6c35a6d559eef78d89f5a003 Mon Sep 17 00:00:00 2001
From: Artur Adib <arturadib@gmail.com>
Date: Mon, 27 Aug 2012 13:09:33 -0300
Subject: [PATCH 10/12] Update make.js

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

diff --git a/make.js b/make.js
index fa04b6c1d..a4b3f4f9e 100755
--- a/make.js
+++ b/make.js
@@ -625,7 +625,7 @@ target.chrome = function() {
 
   // If we're on a Darwin (Mac) OS, then let's check for an .app path
   if (process.platform === 'darwin' && executable.indexOf('.app') !== -1) {
-    executable = executable + '/Contents/MacOS/Google Chrome');
+    executable = executable + '/Contents/MacOS/Google Chrome';
   }
 
   // If the chrome executable doesn't exist

From 72bb0f5a4533d2344eb7117e8cb4379a5aae4aed Mon Sep 17 00:00:00 2001
From: Yury Delendik <ydelendik@mozilla.com>
Date: Mon, 27 Aug 2012 12:46:26 -0500
Subject: [PATCH 11/12] Add top padding for full screen mode

---
 web/viewer.css | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/web/viewer.css b/web/viewer.css
index 439ec8ca3..97e6c742e 100644
--- a/web/viewer.css
+++ b/web/viewer.css
@@ -29,6 +29,7 @@ select {
 
 #viewerContainer:-webkit-full-screen {
   top: 0px;
+  border-top: 5px solid transparent;
   background-color: #404040;
   background-image: url(images/texture.png);
   width: 100%;
@@ -37,6 +38,8 @@ select {
 }
 
 #viewerContainer:-moz-full-screen {
+  top: 0px;
+  border-top: 5px solid transparent;
   background-color: #404040;
   background-image: url(images/texture.png);
   width: 100%;

From c2cfa99cc8bde9a243780e735344adf7b0faa8f5 Mon Sep 17 00:00:00 2001
From: Yury Delendik <ydelendik@mozilla.com>
Date: Mon, 27 Aug 2012 14:23:42 -0500
Subject: [PATCH 12/12] Fixing new make.js lint errors

---
 make.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/make.js b/make.js
index 4c58545dd..7299bbf3c 100755
--- a/make.js
+++ b/make.js
@@ -618,7 +618,7 @@ target.chrome = function() {
   });
 
   // If there was no chrome entry in the browser manifest, exit
-  if(!executable) {
+  if (!executable) {
     echo('There was no \'chrome\' entry in the browser manifest');
     exit(1);
   }
@@ -629,7 +629,7 @@ target.chrome = function() {
   }
 
   // If the chrome executable doesn't exist
-  if(!test('-f', executable)) {
+  if (!test('-f', executable)) {
     echo('Incorrect executable path to chrome');
     exit(1);
   }