Commit 1e3c4c96 authored by Robert O'Callahan's avatar Robert O'Callahan
Browse files

Bug 506615. XUL decks should not have wididgets. r=dbaron 

--HG--
extra : rebase_source : 5d6728b11a121565f828f27e8abed5e0172aa069
parent 4bc4c470

layout/generic/nsContainerFrame.cpp

+4 −5
Original line number Diff line number Diff line
@@ -617,11 +617,10 @@ nsContainerFrame::SyncFrameViewProperties(nsPresContext* aPresContext, 


    if (!aStyleContext->GetStyleVisibility()->IsVisible() &&

        !aFrame->SupportsVisibilityHidden()) {

      // If it's a scrollable frame that can't hide its scrollbars,

      // hide the view. This means that child elements can't override

      // their parent's visibility, but it's not practical to leave it

      // visible in all cases because the scrollbars will be showing

      // XXXldb Does the view system really enforce this correctly?

      // If it's a subdocument frame or a plugin, hide the view and

      // any associated widget.

      // These are leaf elements so this is OK, no descendant can be

      // visibility:visible.

      viewIsVisible = PR_FALSE;

    } else if (IsMenuPopup(aFrame)) {

      // if the view is for a popup, don't show the view if the popup is closed

layout/generic/nsFrame.cpp

+0 −9
Original line number Diff line number Diff line
@@ -6717,15 +6717,6 @@ nsFrame::SetParent(const nsIFrame* aParent) 
  else if (wasBoxWrapped && !IsBoxWrapped())

    DeleteProperty(nsGkAtoms::boxMetricsProperty);



  if (aParent && aParent->IsBoxFrame()) {

    if (aParent->ChildrenMustHaveWidgets()) {

        nsHTMLContainerFrame::CreateViewForFrame(this, PR_TRUE);

        nsIView* view = GetView();

        if (!view->HasWidget())

          CreateWidgetForView(view);

    }

  }



  return NS_OK;

}

layout/generic/nsIFrame.h

+2 −6
Original line number Diff line number Diff line
@@ -107,10 +107,9 @@ struct nsMargin; 
typedef class nsIFrame nsIBox;



// IID for the nsIFrame interface

// 7b437d20-a34e-11dd-ad8b-0800200c9a66

#define NS_IFRAME_IID \

  { 0x7b437d20, 0xa34e, 0x11dd, \

    { 0xad, 0x8b, 0x08, 0x00, 0x20, 0x0c, 0x9a, 0x66 } }

  { 0x7e9018b5, 0x5405, 0x4e2b, \

    { 0x87, 0x67, 0xe2, 0xb4, 0xb1, 0x3e, 0xc1, 0x69 } }



/**

 * Indication of how the frame can be split. This is used when doing runaround
@@ -2286,9 +2285,6 @@ NS_PTR_TO_INT32(frame->GetProperty(nsGkAtoms::embeddingLevel)) 
  NS_IMETHOD DumpBox(FILE* out)=0;

#endif



  // Only nsDeckFrame requires that all its children have widgets

  virtual PRBool ChildrenMustHaveWidgets() const { return PR_FALSE; }



  /**

   * @return PR_TRUE if this text frame ends with a newline character.  It

   * should return PR_FALSE if this is not a text frame.

layout/xul/base/src/nsBoxFrame.cpp

+0 −12
Original line number Diff line number Diff line
@@ -205,18 +205,6 @@ nsBoxFrame::Init(nsIContent* aContent, 


  MarkIntrinsicWidthsDirty();



  // see if we need a widget

  if (aParent && aParent->IsBoxFrame()) {

    if (aParent->ChildrenMustHaveWidgets()) {

        rv = nsHTMLContainerFrame::CreateViewForFrame(this, PR_TRUE);

        NS_ENSURE_SUCCESS(rv, rv);



        nsIView* view = GetView();

        if (!view->HasWidget())

           view->CreateWidget(kWidgetCID);   

    }

  }



  CacheAttributes();



#ifdef DEBUG_LAYOUT

layout/xul/base/src/nsDeckFrame.h

+0 −4
Original line number Diff line number Diff line
@@ -75,10 +75,6 @@ public: 


  virtual nsIAtom* GetType() const;



#ifndef MOZ_GFX_OPTIMIZE_MOBILE

  virtual PRBool ChildrenMustHaveWidgets() const { return PR_TRUE; }

#endif



#ifdef NS_DEBUG

  NS_IMETHOD GetFrameName(nsAString& aResult) const

  {