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

Unified Diff: chrome/browser/ui/cocoa/autofill/autofill_dialog_cocoa.mm

Issue 135933003: rAc: split TestableAutofillDialogView implementation into its own class (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: with mac nits fixed Created 6 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/autofill/autofill_dialog_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/autofill/autofill_dialog_cocoa.mm b/chrome/browser/ui/cocoa/autofill/autofill_dialog_cocoa.mm
index f07d4c8e6f178ffdcb061528b2f946b884c65812..e559f45300edc1350ee1f29829c77c4fe1c04590 100644
--- a/chrome/browser/ui/cocoa/autofill/autofill_dialog_cocoa.mm
+++ b/chrome/browser/ui/cocoa/autofill/autofill_dialog_cocoa.mm
@@ -134,68 +134,11 @@ void AutofillDialogCocoa::UpdateErrorBubble() {
[sheet_delegate_ updateErrorBubble];
}
-TestableAutofillDialogView* AutofillDialogCocoa::GetTestableView() {
- return this;
-}
-
void AutofillDialogCocoa::OnSignInResize(const gfx::Size& pref_size) {
[sheet_delegate_ onSignInResize:
NSMakeSize(pref_size.width(), pref_size.height())];
}
-void AutofillDialogCocoa::SubmitForTesting() {
- [sheet_delegate_ accept:nil];
-}
-
-void AutofillDialogCocoa::CancelForTesting() {
- [sheet_delegate_ cancel:nil];
-}
-
-base::string16 AutofillDialogCocoa::GetTextContentsOfInput(
- ServerFieldType type) {
- for (size_t i = SECTION_MIN; i <= SECTION_MAX; ++i) {
- DialogSection section = static_cast<DialogSection>(i);
- FieldValueMap contents;
- [sheet_delegate_ getInputs:&contents forSection:section];
- FieldValueMap::const_iterator it = contents.find(type);
- if (it != contents.end())
- return it->second;
- }
-
- NOTREACHED();
- return base::string16();
-}
-
-void AutofillDialogCocoa::SetTextContentsOfInput(
- ServerFieldType type,
- const base::string16& contents) {
- [sheet_delegate_ setTextContents:base::SysUTF16ToNSString(contents)
- forType:type];
-}
-
-void AutofillDialogCocoa::SetTextContentsOfSuggestionInput(
- DialogSection section,
- const base::string16& text) {
- [sheet_delegate_ setTextContents:base::SysUTF16ToNSString(text)
- ofSuggestionForSection:section];
-}
-
-void AutofillDialogCocoa::ActivateInput(ServerFieldType type) {
- [sheet_delegate_ activateFieldForType:type];
-}
-
-gfx::Size AutofillDialogCocoa::GetSize() const {
- return gfx::Size(NSSizeToCGSize([[sheet_delegate_ window] frame].size));
-}
-
-content::WebContents* AutofillDialogCocoa::GetSignInWebContents() {
- return [sheet_delegate_ getSignInWebContents];
-}
-
-
-bool AutofillDialogCocoa::IsShowingOverlay() const {
- return [sheet_delegate_ isShowingOverlay];
-}
void AutofillDialogCocoa::OnConstrainedWindowClosed(
ConstrainedWindowMac* window) {

Powered by Google App Engine
This is Rietveld 408576698