mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-10-15 22:35:43 +00:00
Fix for 93111, Make table row and col frame invalid as direct targets of mouseevents. Mouseevents on table should hit cells or the table itself instead. r:dbaron, sr:jst
This commit is contained in:
parent
a5cda6ec41
commit
af9379992c
@ -161,15 +161,6 @@ nsTableColFrame::GetFrameForPoint(nsIPresContext* aPresContext,
|
|||||||
nsFramePaintLayer aWhichLayer,
|
nsFramePaintLayer aWhichLayer,
|
||||||
nsIFrame** aFrame)
|
nsIFrame** aFrame)
|
||||||
{
|
{
|
||||||
if ((aWhichLayer == NS_FRAME_PAINT_LAYER_BACKGROUND) &&
|
|
||||||
(mRect.Contains(aPoint))) {
|
|
||||||
const nsStyleVisibility* vis =
|
|
||||||
(const nsStyleVisibility*)mStyleContext->GetStyleData(eStyleStruct_Visibility);
|
|
||||||
if (vis->IsVisible()) {
|
|
||||||
*aFrame = this;
|
|
||||||
return NS_OK;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
return NS_ERROR_FAILURE;
|
return NS_ERROR_FAILURE;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -437,7 +437,7 @@ nsTableColGroupFrame::GetFrameForPoint(nsIPresContext* aPresContext,
|
|||||||
nsIFrame** aFrame)
|
nsIFrame** aFrame)
|
||||||
{
|
{
|
||||||
// this should act like a block, so we need to override
|
// this should act like a block, so we need to override
|
||||||
return GetFrameForPointUsing(aPresContext, aPoint, nsnull, aWhichLayer, (aWhichLayer == NS_FRAME_PAINT_LAYER_BACKGROUND), aFrame);
|
return GetFrameForPointUsing(aPresContext, aPoint, nsnull, aWhichLayer, PR_FALSE, aFrame);
|
||||||
}
|
}
|
||||||
|
|
||||||
NS_METHOD nsTableColGroupFrame::Reflow(nsIPresContext* aPresContext,
|
NS_METHOD nsTableColGroupFrame::Reflow(nsIPresContext* aPresContext,
|
||||||
|
@ -707,15 +707,6 @@ nsTableRowFrame::GetFrameForPoint(nsIPresContext* aPresContext,
|
|||||||
return NS_OK;
|
return NS_OK;
|
||||||
}
|
}
|
||||||
|
|
||||||
if ( inThisFrame && (aWhichLayer == NS_FRAME_PAINT_LAYER_BACKGROUND)) {
|
|
||||||
const nsStyleVisibility* vis =
|
|
||||||
(const nsStyleVisibility*)mStyleContext->GetStyleData(eStyleStruct_Visibility);
|
|
||||||
if (vis->IsVisible()) {
|
|
||||||
*aFrame = this;
|
|
||||||
return NS_OK;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
return NS_ERROR_FAILURE;
|
return NS_ERROR_FAILURE;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -305,7 +305,7 @@ nsTableRowGroupFrame::GetFrameForPoint(nsIPresContext* aPresContext,
|
|||||||
nsIFrame** aFrame)
|
nsIFrame** aFrame)
|
||||||
{
|
{
|
||||||
// this should act like a block, so we need to override
|
// this should act like a block, so we need to override
|
||||||
return GetFrameForPointUsing(aPresContext, aPoint, nsnull, aWhichLayer, (aWhichLayer == NS_FRAME_PAINT_LAYER_BACKGROUND), aFrame);
|
return GetFrameForPointUsing(aPresContext, aPoint, nsnull, aWhichLayer, PR_FALSE, aFrame);
|
||||||
}
|
}
|
||||||
|
|
||||||
// Position and size aKidFrame and update our reflow state. The origin of
|
// Position and size aKidFrame and update our reflow state. The origin of
|
||||||
|
@ -161,15 +161,6 @@ nsTableColFrame::GetFrameForPoint(nsIPresContext* aPresContext,
|
|||||||
nsFramePaintLayer aWhichLayer,
|
nsFramePaintLayer aWhichLayer,
|
||||||
nsIFrame** aFrame)
|
nsIFrame** aFrame)
|
||||||
{
|
{
|
||||||
if ((aWhichLayer == NS_FRAME_PAINT_LAYER_BACKGROUND) &&
|
|
||||||
(mRect.Contains(aPoint))) {
|
|
||||||
const nsStyleVisibility* vis =
|
|
||||||
(const nsStyleVisibility*)mStyleContext->GetStyleData(eStyleStruct_Visibility);
|
|
||||||
if (vis->IsVisible()) {
|
|
||||||
*aFrame = this;
|
|
||||||
return NS_OK;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
return NS_ERROR_FAILURE;
|
return NS_ERROR_FAILURE;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -437,7 +437,7 @@ nsTableColGroupFrame::GetFrameForPoint(nsIPresContext* aPresContext,
|
|||||||
nsIFrame** aFrame)
|
nsIFrame** aFrame)
|
||||||
{
|
{
|
||||||
// this should act like a block, so we need to override
|
// this should act like a block, so we need to override
|
||||||
return GetFrameForPointUsing(aPresContext, aPoint, nsnull, aWhichLayer, (aWhichLayer == NS_FRAME_PAINT_LAYER_BACKGROUND), aFrame);
|
return GetFrameForPointUsing(aPresContext, aPoint, nsnull, aWhichLayer, PR_FALSE, aFrame);
|
||||||
}
|
}
|
||||||
|
|
||||||
NS_METHOD nsTableColGroupFrame::Reflow(nsIPresContext* aPresContext,
|
NS_METHOD nsTableColGroupFrame::Reflow(nsIPresContext* aPresContext,
|
||||||
|
@ -707,15 +707,6 @@ nsTableRowFrame::GetFrameForPoint(nsIPresContext* aPresContext,
|
|||||||
return NS_OK;
|
return NS_OK;
|
||||||
}
|
}
|
||||||
|
|
||||||
if ( inThisFrame && (aWhichLayer == NS_FRAME_PAINT_LAYER_BACKGROUND)) {
|
|
||||||
const nsStyleVisibility* vis =
|
|
||||||
(const nsStyleVisibility*)mStyleContext->GetStyleData(eStyleStruct_Visibility);
|
|
||||||
if (vis->IsVisible()) {
|
|
||||||
*aFrame = this;
|
|
||||||
return NS_OK;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
return NS_ERROR_FAILURE;
|
return NS_ERROR_FAILURE;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -305,7 +305,7 @@ nsTableRowGroupFrame::GetFrameForPoint(nsIPresContext* aPresContext,
|
|||||||
nsIFrame** aFrame)
|
nsIFrame** aFrame)
|
||||||
{
|
{
|
||||||
// this should act like a block, so we need to override
|
// this should act like a block, so we need to override
|
||||||
return GetFrameForPointUsing(aPresContext, aPoint, nsnull, aWhichLayer, (aWhichLayer == NS_FRAME_PAINT_LAYER_BACKGROUND), aFrame);
|
return GetFrameForPointUsing(aPresContext, aPoint, nsnull, aWhichLayer, PR_FALSE, aFrame);
|
||||||
}
|
}
|
||||||
|
|
||||||
// Position and size aKidFrame and update our reflow state. The origin of
|
// Position and size aKidFrame and update our reflow state. The origin of
|
||||||
|
Loading…
Reference in New Issue
Block a user