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

Side by Side Diff: components/autofill/content/browser/content_autofill_driver_factory.h

Issue 2672243002: Convert ContentAutofillDriverFactory to use the new navigation callbacks. (Closed)
Patch Set: review comment Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_AUTOFILL_CONTENT_BROWSER_CONTENT_AUTOFILL_DRIVER_FACTORY_H_ 5 #ifndef COMPONENTS_AUTOFILL_CONTENT_BROWSER_CONTENT_AUTOFILL_DRIVER_FACTORY_H_
6 #define COMPONENTS_AUTOFILL_CONTENT_BROWSER_CONTENT_AUTOFILL_DRIVER_FACTORY_H_ 6 #define COMPONENTS_AUTOFILL_CONTENT_BROWSER_CONTENT_AUTOFILL_DRIVER_FACTORY_H_
7 7
8 #include <map> 8 #include <map>
9 #include <memory> 9 #include <memory>
10 #include <string> 10 #include <string>
(...skipping 29 matching lines...) Expand all
40 mojom::AutofillDriverRequest request); 40 mojom::AutofillDriverRequest request);
41 41
42 // Gets the |ContentAutofillDriver| associated with |render_frame_host|. 42 // Gets the |ContentAutofillDriver| associated with |render_frame_host|.
43 // |render_frame_host| must be owned by |web_contents()|. 43 // |render_frame_host| must be owned by |web_contents()|.
44 ContentAutofillDriver* DriverForFrame( 44 ContentAutofillDriver* DriverForFrame(
45 content::RenderFrameHost* render_frame_host); 45 content::RenderFrameHost* render_frame_host);
46 46
47 // content::WebContentsObserver: 47 // content::WebContentsObserver:
48 void RenderFrameCreated(content::RenderFrameHost* render_frame_host) override; 48 void RenderFrameCreated(content::RenderFrameHost* render_frame_host) override;
49 void RenderFrameDeleted(content::RenderFrameHost* render_frame_host) override; 49 void RenderFrameDeleted(content::RenderFrameHost* render_frame_host) override;
50 void DidNavigateAnyFrame(
51 content::RenderFrameHost* render_frame_host,
52 const content::LoadCommittedDetails& details,
53 const content::FrameNavigateParams& params) override;
54 void DidFinishNavigation( 50 void DidFinishNavigation(
55 content::NavigationHandle* navigation_handle) override; 51 content::NavigationHandle* navigation_handle) override;
56 void WasHidden() override; 52 void WasHidden() override;
57 53
58 static const char kContentAutofillDriverFactoryWebContentsUserDataKey[]; 54 static const char kContentAutofillDriverFactoryWebContentsUserDataKey[];
59 55
60 protected: 56 protected:
61 ContentAutofillDriverFactory( 57 ContentAutofillDriverFactory(
62 content::WebContents* web_contents, 58 content::WebContents* web_contents,
63 AutofillClient* client, 59 AutofillClient* client,
64 const std::string& app_locale, 60 const std::string& app_locale,
65 AutofillManager::AutofillDownloadManagerState enable_download_manager); 61 AutofillManager::AutofillDownloadManagerState enable_download_manager);
66 62
67 private: 63 private:
68 AutofillClient* client_; 64 AutofillClient* client_;
69 std::string app_locale_; 65 std::string app_locale_;
70 AutofillManager::AutofillDownloadManagerState enable_download_manager_; 66 AutofillManager::AutofillDownloadManagerState enable_download_manager_;
71 67
72 std::map<content::RenderFrameHost*, std::unique_ptr<ContentAutofillDriver>> 68 std::map<content::RenderFrameHost*, std::unique_ptr<ContentAutofillDriver>>
73 frame_driver_map_; 69 frame_driver_map_;
74 }; 70 };
75 71
76 } // namespace autofill 72 } // namespace autofill
77 73
78 #endif // COMPONENTS_AUTOFILL_CONTENT_BROWSER_CONTENT_AUTOFILL_DRIVER_FACTORY_H _ 74 #endif // COMPONENTS_AUTOFILL_CONTENT_BROWSER_CONTENT_AUTOFILL_DRIVER_FACTORY_H _
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698