Remove redundant calls to FrameLoader::frameDetached().

Remove redundant calls to FrameLoader::frameDetached().

Page::willBeDestroyed() detachs any frames remaining in the frame tree,
so there's no need to do this before calling willBeDestroyed().

In addition, since frameDetached() recursively detaches all frames,
eliminate the loop inside Page::willBeDestroyed() to detach all frames.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=176173

[dcheng, 2014-06-16 00:02:32]

Theoretically, frame detach is recursive as well, so the loop inside Page
shouldn't be needed, but I'd like to defer that cleanup until we have a better
story for detaching trees of mixed local and remote frames.

[abarth-chromium, 2014-06-16 03:05:23]

lgtm

[dcheng, 2014-06-16 03:56:59]

The CQ bit was checked by dcheng@chromium.org

[commit-bot: I haz the power, 2014-06-16 03:57:37]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dcheng@chromium.org/339493002/20001

[commit-bot: I haz the power, 2014-06-16 04:59:29]

Change committed as 176173