Skip to content

Commit

Permalink
Bug 1299001 - Ensure that the toolbox resets the preferences set when…
Browse files Browse the repository at this point in the history
… it is closed. r=ochameau

This patch fixes a regression which has been likely introduced in the refactoring
we did in Bug 1222047 on Firefox 63.

Differential Revision: https://phabricator.services.mozilla.com/D7523

UltraBlame original commit: 5e5d6abb39afe0a23555db526a5bbbf3d59b4cd9
  • Loading branch information
marco-c committed Oct 3, 2019
1 parent 2de9461 commit ea573cd
Show file tree
Hide file tree
Showing 2 changed files with 107 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -402,6 +402,32 @@ onReadyForOpenPopup
let
onPopupCustomMessage
;
is
(
Services
.
prefs
.
getBoolPref
(
"
ui
.
popup
.
disable_autohide
"
)
false
"
disable_autohide
shoult
be
initially
false
"
)
;
Management
.
on
Expand Down Expand Up @@ -1569,16 +1595,47 @@ API
await
onToolboxClose
;
ok
info
(
true
"
Addon
toolbox
closed
"
)
;
is
(
Services
.
prefs
.
getBoolPref
(
"
ui
.
popup
.
disable_autohide
"
)
false
"
disable_autohide
should
be
reset
to
false
when
the
toolbox
is
closed
"
)
;
await
uninstallAddon
(
Expand Down
49 changes: 48 additions & 1 deletion devtools/client/framework/toolbox.js
Original file line number Diff line number Diff line change
Expand Up @@ -15560,7 +15560,9 @@ preferenceFront
(
)
{
return
const
frontPromise
=
this
.
target
Expand All @@ -15576,6 +15578,51 @@ preference
"
)
;
frontPromise
.
then
(
front
=
>
{
/
/
Set
the
_preferenceFront
property
to
allow
the
resetPreferences
toolbox
method
/
/
to
cleanup
the
preference
set
when
the
toolbox
is
closed
.
this
.
_preferenceFront
=
front
;
}
)
;
return
frontPromise
;
}
/
/
Expand Down

0 comments on commit ea573cd

Please sign in to comment.