Remove FrameDetached and FrameWillClose listeners from AutofillAgent.
commit1dbe123c5e7b2d41e517049c1b98d8ad75563b98
authorestade <estade@chromium.org>
Tue, 6 Jan 2015 22:45:04 +0000 (6 14:45 -0800)
committerCommit bot <commit-bot@chromium.org>
Tue, 6 Jan 2015 22:46:01 +0000 (6 22:46 +0000)
tree14772650568220fd4e139e225de5f00eb5e29ecd
parent4c559b468d544ddee9f545ad578949ebc02e3919
Remove FrameDetached and FrameWillClose listeners from AutofillAgent.

AutofillAgent::FrameDetached is no longer necessary because form_cache_ only holds the forms for a single frame.
AutofillAgent::FrameWillClose is only called on the actual closing frame (which might be a parent frame).
AutofillAgent::FrameDetached can't take the place of AutofillAgent::FrameWillClose because the RenderFrame disables sending IPCs during detachment.

Therefore, move the automatic closing of rAc from the renderer (autofill_agent.cc) to the browser (chrome_autofill_client.cc).

For PasswordAutofillAgent, add FrameDetached to RenderFrameHost.

BUG=433486

Review URL: https://codereview.chromium.org/796493004

Cr-Commit-Position: refs/heads/master@{#310162}
22 files changed:
android_webview/native/aw_autofill_client.cc
android_webview/native/aw_autofill_client.h
chrome/browser/ui/autofill/autofill_dialog_controller.h
chrome/browser/ui/autofill/autofill_dialog_controller_browsertest.cc
chrome/browser/ui/autofill/chrome_autofill_client.cc
chrome/browser/ui/autofill/chrome_autofill_client.h
chrome/renderer/autofill/autofill_renderer_browsertest.cc
components/autofill/content/browser/content_autofill_driver.cc
components/autofill/content/browser/request_autocomplete_manager.cc
components/autofill/content/browser/request_autocomplete_manager.h
components/autofill/content/browser/request_autocomplete_manager_unittest.cc
components/autofill/content/common/autofill_messages.h
components/autofill/content/renderer/autofill_agent.cc
components/autofill/content/renderer/autofill_agent.h
components/autofill/content/renderer/password_autofill_agent.cc
components/autofill/content/renderer/password_autofill_agent.h
components/autofill/core/browser/autofill_client.h
components/autofill/core/browser/autofill_manager_unittest.cc
components/autofill/core/browser/test_autofill_client.cc
components/autofill/core/browser/test_autofill_client.h
content/public/renderer/render_frame_observer.h
content/renderer/render_frame_impl.cc