Bug 1456616 - Remove suppressFocusScroll from nsXULCommandDispatcher;r=bz

This attribute doesn't do anything.

MozReview-Commit-ID: FnshlyyAms5

--HG--
extra : rebase_source : 75e67479a3306a29b8ea45491984d08ceb4d6907

devtools/client/shared/widgets/VariablesView.jsm

@@ -767,7 +767,6 @@ VariablesView.prototype = {
     let currFocusedItem = null;
 
     do {
-      commandDispatcher.suppressFocusScroll = true;
       commandDispatcher[aDirection]();
 
       // Make sure the newly focused item is a part of this view.

devtools/client/shared/widgets/view-helpers.js

@@ -1133,7 +1133,6 @@ const WidgetMethods = exports.WidgetMethods = {
     let currFocusedElement;
 
     do {
-      commandDispatcher.suppressFocusScroll = true;
       commandDispatcher[direction]();
       currFocusedElement = commandDispatcher.focusedElement;
 

dom/interfaces/xul/nsIDOMXULCommandDispatcher.idl

@@ -30,7 +30,6 @@ interface nsIDOMXULCommandDispatcher : nsISupports
   void advanceFocus();
   void rewindFocus();
   void advanceFocusIntoSubtree(in nsIDOMElement elt);
-  attribute boolean suppressFocusScroll;
 
   // When locked, command updating is batched until unlocked. Always ensure that
   // lock and unlock is called in a pair.

dom/xul/nsXULCommandDispatcher.cpp

@@ -452,19 +452,6 @@ nsXULCommandDispatcher::GetControllerForCommand(const char *aCommand, nsIControl
                                        _retval);
 }
 
-NS_IMETHODIMP
-nsXULCommandDispatcher::GetSuppressFocusScroll(bool* aSuppressFocusScroll)
-{
-  *aSuppressFocusScroll = false;
-  return NS_OK;
-}
-
-NS_IMETHODIMP
-nsXULCommandDispatcher::SetSuppressFocusScroll(bool aSuppressFocusScroll)
-{
-  return NS_OK;
-}
-
 NS_IMETHODIMP
 nsXULCommandDispatcher::Lock()
 {