Clean up some detach logic in RenderFrameImpl.

Clean up some detach logic in RenderFrameImpl.

There's no reason to gate RenderWidget frame unregistration
on the kSitePerProcess flag; it should be gated using the
same policy as the creation of the RenderWidget.

Moving the RenderWidget cleanup out of the is_subframe
block also paves the way for when the main frame also has a
RenderWidget.

BUG=419087
R=creis@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/3403bc5413575b3b05842d79b8547d58308d0b7d

[dcheng, 2015-07-09 20:39:01]

dcheng@chromium.org changed reviewers:
+ creis@chromium.org

[Charlie Reis, 2015-07-09 21:06:19]

LGTM with nit.

https://codereview.chromium.org/1229083002/diff/1/content/renderer/render_fra...
File content/renderer/render_frame_impl.cc (right):

https://codereview.chromium.org/1229083002/diff/1/content/renderer/render_fra...
content/renderer/render_frame_impl.cc:2170: if (render_widget_) {
nit: No braces.  Same below.

[dcheng, 2015-07-09 21:13:23]

Committed patchset #2 (id:20001) manually as
3403bc5413575b3b05842d79b8547d58308d0b7d (presubmit successful).