Fix zoom level on browser restore

and remove deprecated zoom level site setting.

fix #7832

Test Plan:
1. launch clean instance of brave
2. go to example.com and change the zoom level
3. close brave
4. reopen brave
5. example.com should load with the previous zoom setting

docs/state.md

@@ -262,7 +262,7 @@ AppStore
       savePasswords: boolean, // only false or undefined/null
       shieldsUp: boolean,
       widevine: (number|boolean), // false = block widevine, 0 = allow once, 1 = allow always
-      zoomLevel: number
+      zoomLevel: number // deprecated
     }
   },
   sync: {

js/components/frame.js

@@ -13,6 +13,7 @@ const cx = require('../lib/classSet')
 const siteUtil = require('../state/siteUtil')
 const FrameStateUtil = require('../state/frameStateUtil')
 const UrlUtil = require('../lib/urlutil')
+const {getZoomLevel} = require('../lib/zoom')
 const messages = require('../constants/messages')
 const contextMenus = require('../contextMenus')
 const ipc = require('electron').ipcRenderer
@@ -336,7 +337,18 @@ class Frame extends ImmutableComponent {
           this.runOnDomReady()
           delete this.runOnDomReady
         }
+        let zoomCallback = (e) => {
+          if (!e.isMainFrame) {
+            return
+          }
+          this.webview.removeEventListener(e.type, zoomCallback)
+          const zoomPercentage = this.frame && this.frame.get('lastZoomPercentage')
+          if (zoomPercentage !== this.webview.getZoomPercent()) {
+            this.webview.setZoomLevel(getZoomLevel(zoomPercentage))
+          }
+        }
         this.webview.addEventListener('did-attach', eventCallback)
+        this.webview.addEventListener('load-commit', zoomCallback)
       }
 
       webviewAdded = true
@@ -368,12 +380,6 @@ class Frame extends ImmutableComponent {
     this.updateWebview(this.onPropsChanged)
   }
 
-  get zoomLevel () {
-    const zoom = this.props.frameSiteSettings && this.props.frameSiteSettings.get('zoomLevel')
-    appActions.removeSiteSetting(this.origin, 'zoomLevel', this.props.isPrivate)
-    return zoom
-  }
-
   zoomIn () {
     if (this.webview) {
       this.webview.zoomIn()

js/lib/zoom.js

@@ -8,6 +8,11 @@ const {zoom} = require('../constants/config')
 module.exports.getZoomValuePercentage = (zoomLevel) =>
   100 + (20 * zoomLevel)
 
+// Convert zoom percentage to zoom level
+module.exports.getZoomLevel = (percentage) => {
+  return (percentage - 100) / 20
+}
+
 module.exports.getNextZoomLevel = (currentZoom, zoomIn) => {
   const zoomLevels = zoom.zoomLevels
   // First find the closet value to what we allow

js/state/frameStateUtil.js

@@ -343,7 +343,6 @@ function addFrame (windowState, tabs, frameOpts, newKey, partitionNumber, active
   }
 
   const frame = Immutable.fromJS(Object.assign({
-    zoomLevel: config.zoom.defaultValue,
     audioMuted: false, // frame is muted
     location,
     aboutDetails: undefined,

js/state/syncUtil.js

@@ -29,7 +29,7 @@ module.exports.CATEGORY_MAP = CATEGORY_MAP
 
 const siteSettingDefaults = {
   hostPattern: '',
-  zoomLevel: 0,
+  zoomLevel: 0, // deprecated
   shieldsUp: true,
   adControl: 1,
   cookieControl: 0,

test/unit/lib/zoomTest.js

@@ -1,6 +1,6 @@
 /* global describe, it */
 
-const {getZoomValuePercentage, getNextZoomLevel} = require('../../../js/lib/zoom')
+const {getZoomValuePercentage, getZoomLevel, getNextZoomLevel} = require('../../../js/lib/zoom')
 const {zoom} = require('../../../js/constants/config')
 const assert = require('assert')
 
@@ -16,6 +16,17 @@ describe('zoom', function () {
       assert.equal(getZoomValuePercentage(-3.75), 25)
     })
   })
+  describe('getZoomLevel', function () {
+    it('formats 100 to 0', function * () {
+      assert.equal(getZoomLevel(100), 0)
+    })
+    it('formats positive value', function * () {
+      assert.equal(getZoomLevel(120), 1)
+    })
+    it('formats negative value', function * () {
+      assert.equal(getZoomLevel(25), -3.75)
+    })
+  })
   describe('getNextZoomLevel', function () {
     it('zoomOut respects minimum', function * () {
       assert.equal(getNextZoomLevel(-200, false), zoom.zoomLevels[0])