1676439 - New Import button shouldn't show on the toolbar if DisableProfileImport policy is set

Status: RESOLVED FIXED

(Firefox :: Bookmarks & History, defect, P1)

Description

[Mike Kaply [:mkaply]]

The new import button should not appear if the DisableProfileImport policy is set.

This can be checked with:

if (!Services.policies.isAllowed("profileImport")) {

Comment 1

[Mike Kaply [:mkaply]]

Jared:

Do you think I should do this in maybeAddImportButton or in BrowserGlue.jsm to prevent the call to maybeAddImportButton?

Comment 2

[Jared Wein [:jaws] (please needinfo? me)]

I would put this inside of maybeAddImportButton to limit the amount of code in the calling function in BrowserGlue.jsm. Also, maybeAddImportButton is already named for this occasion :)

Comment 3

[Mike Kaply [:mkaply]]

Attachment: Bug 1676439 - Don't show import button if DisableProfileImport policy is active. r?jaws!

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5c5c0f902674
Don't show import button if DisableProfileImport policy is active. r=jaws

Comment 5

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/5c5c0f902674