Don't send PageMsg_WasShown from RenderFrameHostManager::CommitPending()

This was done to show the Page which was hidden accidentally when
hiding the main frame's RenderWidgetHost when detaching a main
frame. Detaching a main frame doesn't destroy the RenderWidgetHost
because the renderer's RenderWidget's lifetime is tied to
RenderView.

Now the Page is no longer hiding with the main frame RenderWidget
so this code does not need to show the Page when creating a subframe
in a tree that was empty but used to hold a local main frame.

Lots of investigation/details in crbug.com/908582 from #15 to #19.

R=alexmos@chromium.org, creis@chromium.org

Change-Id: Ie03115c94883c2a19840f7b9e16fc4c94789ede2
Bug: 419087, 908582, 638375
Reviewed-on: https://chromium-review.googlesource.com/c/1357629
Commit-Queue: danakj <danakj@chromium.org>
Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Lucas Gadani <lfg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#613740}
1 file changed