Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(760)

Issue 17382007: Move IPC reception handling from AutofillManager to AutofillDriverImpl. (Closed)

Created:
7 years, 6 months ago by blundell
Modified:
7 years, 6 months ago
CC:
chromium-reviews, Raman Kakilate, benquan, jam, ahutter, browser-components-watch_chromium.org, joi+watch-content_chromium.org, darin-cc_chromium.org, Dane Wallinga, dyu1, Albert Bodenhamer, estade+watch_chromium.org, Ilya Sherman, rouslan+autofillwatch_chromium.org, selim, benm (inactive)
Visibility:
Public.

Description

Move IPC reception handling from AutofillManager to AutofillDriverImpl. As AutofillManager will be shared by iOS, it should not receive IPC directly. Instead, AutofillDriverImpl now handles IPC reception and forwards to AutofillManager as appropriate. Similarly, this CL also abstracts the content-level concepts of handling frame navigation out of AutofillManager into AutofillDriverImpl. BUG=247015 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=207726

Patch Set 1 #

Patch Set 2 : Rebase to fix conflicts #

Patch Set 3 : Rebase after conflict #

Unified diffs Side-by-side diffs Delta from patch set Stats (+109 lines, -121 lines) Patch
M components/autofill/content/browser/autofill_driver_impl.cc View 1 2 chunks +58 lines, -4 lines 0 comments Download
M components/autofill/core/browser/autocomplete_history_manager.cc View 1 1 chunk +0 lines, -1 line 0 comments Download
M components/autofill/core/browser/autofill_manager.h View 1 2 7 chunks +51 lines, -63 lines 0 comments Download
M components/autofill/core/browser/autofill_manager.cc View 1 2 3 chunks +0 lines, -53 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
blundell
I am currently working on bringing up unit tests of AutofillDriverImpl for a follow-up CL.
7 years, 6 months ago (2013-06-18 16:20:58 UTC) #1
Ilya Sherman
LGTM. It would be great to update the documentation for all of these methods, and ...
7 years, 6 months ago (2013-06-18 23:44:42 UTC) #2
blundell
Agreed about the followup work re: updating documentation and reordering. Will do.
7 years, 6 months ago (2013-06-20 17:08:47 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/blundell@chromium.org/17382007/4001
7 years, 6 months ago (2013-06-20 17:11:43 UTC) #4
commit-bot: I haz the power
Commit queue failed due to new patchset.
7 years, 6 months ago (2013-06-20 22:28:37 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/blundell@chromium.org/17382007/4002
7 years, 6 months ago (2013-06-20 22:28:56 UTC) #6
commit-bot: I haz the power
7 years, 6 months ago (2013-06-21 06:04:49 UTC) #7
Message was sent while issue was closed.
Change committed as 207726

Powered by Google App Engine
This is Rietveld 408576698