1772166 - defaultZoomFactor not affected by global/default zoom

Status: RESOLVED FIXED

(WebExtensions :: General, defect, P5)

Description

[robbendebiene]

Attachment: zoomdef.png

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:100.0) Gecko/20100101 Firefox/100.0

Steps to reproduce:

Set a default zoom value other than 100% in the Firefox settings page and run the following webextension code for a specific tab:

const zoomSettings = await browser.tabs.getZoomSettings(
  tabId
);
console.log(zoomSettings.defaultZoomFactor);

Actual results:

The returned defaultZoomFactor value is always 1 regardless of the value set in the Firefox settings page.

Expected results:

The defaultZoomFactor should reflect the value set in the Firefox settings page.

Comment 1

[Alex Cornestean]

Hello,

I reproduced the issue on the latest Nightly (103.0a1/20220602093647), Beta (102.0b3/20220602190016) and Release (101.0/20220526203855) under Windows 10 x64 and Ubuntu 16.04 LTS.

Running the code snippet will return the value 1, regardless of the default zoom value set in the Firefox settings page, as described in Comment 0.

Comment 2

[Tomislav Jovanovic :zombie]

Yeah, seems we never implemented this:
https://searchfox.org/mozilla-central/rev/4c3f6e8bf8/browser/components/extensions/parent/ext-tabs.js#1203-1212

Comment 3

[Gregory Pappas [:gregp]]

Attachment: Bug 1772166 - Implement 'ZoomSettings.defaultZoomFactor' r=#extension-reviewers

Comment 4

[Shane Caraveo (:mixedpuppy)]

There is some behavior differences between chrome and firefox we may want to add to the documentation.

Comment 5

[Sandor Molnar[:smolnar]]

Backed out for causing bc failures in browser/components/extensions/test/browser/browser_ext_webNavigation_onCreatedNavigationTarget.js

Backout link: https://hg.mozilla.org/integration/autoland/rev/8074340029a33f955bcb7ac2dba1619d33650c3a

Push with failures

Failure log

INFO - TEST-UNEXPECTED-FAIL | browser/components/extensions/test/browser/browser_ext_webNavigation_onCreatedNavigationTarget.js | Test timed out - 
[task 2022-12-05T16:06:16.608Z] 16:06:16     INFO - Not taking screenshot here: see the one that was previously logged
[task 2022-12-05T16:06:16.609Z] 16:06:16     INFO - TEST-UNEXPECTED-FAIL | browser/components/extensions/test/browser/browser_ext_webNavigation_onCreatedNavigationTarget.js | no tasks awaiting on messages - Got "[\"webNavOnCreated\"]", expected "[]"
[task 2022-12-05T16:06:16.609Z] 16:06:16     INFO - Stack trace:

Comment 6

[Gregory Pappas [:gregp]]

Thanks, looking.

Comment 7

[Gregory Pappas [:gregp]]

I'm having trouble reproducing this failure locally.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/470fffc6ff9a
Implement 'ZoomSettings.defaultZoomFactor' r=mixedpuppy,extension-reviewers

Comment 9

[Atila Butkovits]

Backed out for causing failures at browser_ext_webNavigation_onCreatedNavigationTarget.js.

Backout link: https://hg.mozilla.org/integration/autoland/rev/d62aaaa9665c159f9bb6fc79c5136777435cfc3d

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&selectedTaskRun=K4RI63o_RiqSWP5UOIsjdw.0&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=470fffc6ff9a65be7af6157d73c285c1789de3cf

Failure log: https://treeherder.mozilla.org/logviewer?job_id=398781206&repo=autoland&lineNumber=5470

Comment 10

[Gregory Pappas [:gregp]]

Still can't reproduce this locally. I have some theories but testing them would require access to Try...

Comment 11

[BugBot [:suhaib / :marco/ :calixte]]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:gregp, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit auto_nag documentation.

Comment 12

[Gregory Pappas [:gregp]]

Still working on this

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/878a8a888ce1
Implement 'ZoomSettings.defaultZoomFactor' r=mixedpuppy,extension-reviewers

Comment 14

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/878a8a888ce1

Comment 15

[rbloor]

See

• BCD #18511
• content: #23392