author Doug Thayer <dothayer@mozilla.com>
Fri, 01 Mar 2019 18:29:09 +0000
changeset 520116 8b3fe0426ffc1b3a2ad044ef6cdde6c4f736f8e2
parent 505383 6f3709b3878117466168c40affa7bca0b60cf75b
child 531502 59633bc3f7619ec93c0a978861c2f466d7fd3433
permissions -rw-r--r--
Bug 1442694 - Fix failures due to removing selected tab r=Gijs This adds test which reproduce the failure as well as the fix. Essentially, if we hit the edited case in SessionStore with `tab` equal to `tabbrowser.tabs[t]`, we remove the tab and then try to pin it, which obviously blows up. Note: the additional method in SessionStore.jsm was largely to get around complexity requirements inside restoreWindow. Cleaner solutions welcome. Differential Revision: https://phabricator.services.mozilla.com/D21383

/* -*- Mode: C++; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*-
 * This Source Code Form is subject to the terms of the Mozilla Public
 * License, v. 2.0. If a copy of the MPL was not distributed with this
 * file, You can obtain one at http://mozilla.org/MPL/2.0/. */

#ifndef __nsIRollupListener_h__
#define __nsIRollupListener_h__

#include "nsTArray.h"
#include "nsPoint.h"

class nsIContent;
class nsIWidget;

class nsIRollupListener {
   * Notifies the object to rollup, optionally returning the node that
   * was just rolled up.
   * If aFlush is true, then views should be flushed after the rollup.
   * aPoint is the mouse pointer position where the event that triggered the
   * rollup occurred, which may be nullptr.
   * aCount is the number of popups in a chain to close. If this is
   * UINT32_MAX, then all popups are closed.
   * If aLastRolledUp is non-null, it will be set to the last rolled up popup,
   * if this is supported. aLastRolledUp is not addrefed.
   * Returns true if the event that the caller is processing should be consumed.
  virtual bool Rollup(uint32_t aCount, bool aFlush, const nsIntPoint* aPoint,
                      nsIContent** aLastRolledUp) = 0;

   * Asks the RollupListener if it should rollup on mouse wheel events
  virtual bool ShouldRollupOnMouseWheelEvent() = 0;

   * Asks the RollupListener if it should consume mouse wheel events
  virtual bool ShouldConsumeOnMouseWheelEvent() = 0;

   * Asks the RollupListener if it should rollup on mouse activate, eg. X-Mouse
  virtual bool ShouldRollupOnMouseActivate() = 0;

   * Retrieve the widgets for open menus and store them in the array
   * aWidgetChain. The number of menus of the same type should be returned,
   * for example, if a context menu is open, return only the number of menus
   * that are part of the context menu chain. This allows closing up only
   * those menus in different situations. The returned value should be exactly
   * the same number of widgets added to aWidgetChain.
  virtual uint32_t GetSubmenuWidgetChain(
      nsTArray<nsIWidget*>* aWidgetChain) = 0;

   * Notify the RollupListener that the widget did a Move or Resize.
  virtual void NotifyGeometryChange() = 0;

  virtual nsIWidget* GetRollupWidget() = 0;

#endif /* __nsIRollupListener_h__ */