Bug 1351777 - Remove unnecessary cast and more tightly scope a local var. r?dvander
MozReview-Commit-ID: JEfQOdQ2HNJ
--- a/dom/ipc/TabParent.cpp
+++ b/dom/ipc/TabParent.cpp
@@ -2485,27 +2485,25 @@ TabParent::DeallocPColorPickerParent(PCo
return true;
}
PRenderFrameParent*
TabParent::AllocPRenderFrameParent()
{
MOZ_ASSERT(ManagedPRenderFrameParent().IsEmpty());
RefPtr<nsFrameLoader> frameLoader = GetFrameLoader();
- uint64_t layersId = 0;
bool success = false;
- PRenderFrameParent* renderFrame =
+ RenderFrameParent* rfp =
new RenderFrameParent(frameLoader, &success);
if (success) {
- RenderFrameParent* rfp = static_cast<RenderFrameParent*>(renderFrame);
- layersId = rfp->GetLayersId();
+ uint64_t layersId = rfp->GetLayersId();
AddTabParentToTable(layersId, this);
}
- return renderFrame;
+ return rfp;
}
bool
TabParent::DeallocPRenderFrameParent(PRenderFrameParent* aFrame)
{
delete aFrame;
return true;
}