Merge pull request #3865 from deepak1556/visibility_api_patch

browser: fix document.hidden value

atom/renderer/lib/override.coffee

@@ -118,6 +118,12 @@ Object.defineProperty window.history, 'length',
   get: ->
     getHistoryOperation 'length'
 
-# Make document.hidden return the correct value.
+# Make document.hidden and document.visibilityState return the correct value.
 Object.defineProperty document, 'hidden',
-  get: -> !remote.getCurrentWindow().isVisible()
+  get: ->
+    currentWindow = remote.getCurrentWindow()
+    currentWindow.isMinimized() || !currentWindow.isVisible()
+
+Object.defineProperty document, 'visibilityState',
+  get: ->
+    if document.hidden then "hidden" else "visible"

spec/chromium-spec.coffee

@@ -45,6 +45,14 @@ describe 'chromium feature', ->
         done()
       w.loadURL url
 
+    it 'is set correctly when window is inactive', (done) ->
+      w = new BrowserWindow(show:false)
+      w.webContents.on 'ipc-message', (event, args) ->
+        assert.deepEqual args, ['hidden', false]
+        done()
+      w.showInactive()
+      w.loadURL url
+
   xdescribe 'navigator.webkitGetUserMedia', ->
     it 'calls its callbacks', (done) ->
       @timeout 5000