Skip to content

Commit

Permalink
[ios] Create directory for Password Issues.
Browse files Browse the repository at this point in the history
This patch moves to a new directory files related to the Password
Issues surface in Bling's Password Manager. Unused deps were removed
from the original source sets of the moving files.

Fixed:1425170

Change-Id: I2be2952e6fbbccea8052298c96083f2996e2610c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4343521
Commit-Queue: Ernesto Izquierdo Clua <eic@google.com>
Auto-Submit: Ernesto Izquierdo Clua <eic@google.com>
Reviewed-by: Tommy Martino <tmartino@chromium.org>
Commit-Queue: Sergio Collazos <sczs@chromium.org>
Reviewed-by: Sergio Collazos <sczs@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1118422}
  • Loading branch information
Ernesto Izquierdo Clua authored and Chromium LUCI CQ committed Mar 16, 2023
1 parent db41689 commit fee5b52
Show file tree
Hide file tree
Showing 22 changed files with 177 additions and 102 deletions.
65 changes: 16 additions & 49 deletions ios/chrome/browser/ui/settings/password/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,6 @@
source_set("password") {
configs += [ "//build/config/compiler:enable_arc" ]
sources = [
"password_issues_coordinator.h",
"password_issues_coordinator.mm",
"password_issues_mediator.h",
"password_issues_mediator.mm",
"passwords_coordinator.h",
"passwords_coordinator.mm",
"passwords_mediator.h",
Expand All @@ -18,15 +14,13 @@ source_set("password") {
":common",
":password_ui",
"//base",
"//components/autofill/core/common",
"//components/google/core/common",
"//components/keyed_service/core",
"//components/password_manager/core/browser",
"//components/password_manager/core/common:features",
"//components/signin/public/identity_manager/objc",
"//components/strings",
"//components/sync",
"//components/sync/driver",
"//ios/chrome/app/strings",
"//ios/chrome/browser/application_context",
"//ios/chrome/browser/browser_state",
"//ios/chrome/browser/favicon",
"//ios/chrome/browser/main:public",
"//ios/chrome/browser/net:crurl",
Expand All @@ -35,23 +29,21 @@ source_set("password") {
"//ios/chrome/browser/shared/coordinator/chrome_coordinator",
"//ios/chrome/browser/shared/public/commands",
"//ios/chrome/browser/shared/ui/table_view",
"//ios/chrome/browser/shared/ui/table_view:utils",
"//ios/chrome/browser/shared/ui/util",
"//ios/chrome/browser/signin",
"//ios/chrome/browser/sync",
"//ios/chrome/browser/ui/alert_coordinator",
"//ios/chrome/browser/ui/settings/password/password_checkup",
"//ios/chrome/browser/ui/settings/password/password_checkup:password_checkup_constants",
"//ios/chrome/browser/ui/settings/password/password_checkup:password_checkup_ui",
"//ios/chrome/browser/ui/settings/password/password_checkup:password_checkup_utils",
"//ios/chrome/browser/ui/settings/password/password_details",
"//ios/chrome/browser/ui/settings/password/password_issues",
"//ios/chrome/browser/ui/settings/password/password_settings",
"//ios/chrome/browser/ui/settings/password/passwords_in_other_apps",
"//ios/chrome/browser/ui/settings/utils",
"//ios/chrome/common:string_util",
"//ios/chrome/common/ui/colors",
"//ios/chrome/common/ui/favicon:favicon_constants",
"//ios/chrome/common/ui/reauthentication",
"//net",
"//ui/base",
"//url",
]
Expand All @@ -78,16 +70,6 @@ source_set("password_ui") {
sources = [
"password_exporter.h",
"password_exporter.mm",
"password_issue.h",
"password_issue.mm",
"password_issue_content_item.h",
"password_issue_content_item.mm",
"password_issue_group.h",
"password_issue_group.mm",
"password_issues_consumer.h",
"password_issues_presenter.h",
"password_issues_table_view_controller.h",
"password_issues_table_view_controller.mm",
"password_manager_view_controller+private.h",
"password_manager_view_controller.h",
"password_manager_view_controller.mm",
Expand All @@ -103,54 +85,49 @@ source_set("password_ui") {
":password_constants",
":title_view",
"//base",
"//components/autofill/core/common",
"//components/google/core/common",
"//components/keyed_service/core",
"//components/password_manager/core/browser",
"//components/password_manager/core/common",
"//components/password_manager/core/common:features",
"//components/prefs",
"//components/strings",
"//components/url_formatter",
"//components/sync/base",
"//components/sync/driver",
"//ios/chrome/app/strings",
"//ios/chrome/browser/application_context",
"//ios/chrome/browser/browser_state",
"//ios/chrome/browser/flags:system_flags",
"//ios/chrome/browser/main:public",
"//ios/chrome/browser/net:crurl",
"//ios/chrome/browser/passwords",
"//ios/chrome/browser/shared/public/commands",
"//ios/chrome/browser/shared/ui/table_view",
"//ios/chrome/browser/shared/ui/table_view:constants",
"//ios/chrome/browser/shared/ui/table_view:styler",
"//ios/chrome/browser/shared/ui/table_view:utils",
"//ios/chrome/browser/shared/ui/table_view:views",
"//ios/chrome/browser/shared/ui/table_view/cells",
"//ios/chrome/browser/shared/ui/util",
"//ios/chrome/browser/signin",
"//ios/chrome/browser/sync",
"//ios/chrome/browser/ui/elements",
"//ios/chrome/browser/ui/icons:symbols",
"//ios/chrome/browser/ui/settings:settings_root",
"//ios/chrome/browser/ui/settings:settings_root_categories",
"//ios/chrome/browser/ui/settings/cells",
"//ios/chrome/browser/ui/settings/cells:public",
"//ios/chrome/browser/ui/settings/elements:enterprise_info_popover_view_controller",
"//ios/chrome/browser/ui/settings/password/password_checkup",
"//ios/chrome/browser/ui/settings/password/password_details",
"//ios/chrome/browser/ui/settings/password/resources:passwords_empty",
"//ios/chrome/browser/ui/settings/utils",
"//ios/chrome/browser/url:constants",
"//ios/chrome/common",
"//ios/chrome/common:constants",
"//ios/chrome/common:string_util",
"//ios/chrome/common/ui/colors",
"//ios/chrome/common/ui/elements:popover_label_view_controller",
"//ios/chrome/common/ui/favicon",
"//ios/chrome/common/ui/reauthentication",
"//ios/chrome/common/ui/table_view:cells_constants",
"//ios/chrome/common/ui/util",
"//ios/third_party/material_components_ios",
"//net",
"//ui/base",
"//ui/base/clipboard:clipboard_types",
"//url",
]
}

Expand Down Expand Up @@ -199,8 +176,6 @@ source_set("unit_tests") {
testonly = true
sources = [
"password_exporter_unittest.mm",
"password_issues_mediator_unittest.mm",
"password_issues_table_view_controller_unittest.mm",
"password_manager_view_controller_unittest.mm",
"passwords_mediator_unittest.mm",
]
Expand All @@ -211,36 +186,29 @@ source_set("unit_tests") {
":test_support",
"//base",
"//base/test:test_support",
"//components/autofill/core/common",
"//components/google/core/common",
"//components/keyed_service/core",
"//components/password_manager/core/browser",
"//components/password_manager/core/browser:password_form",
"//components/password_manager/core/browser:test_support",
"//components/password_manager/core/common",
"//components/password_manager/core/common:features",
"//components/prefs:test_support",
"//components/signin/public/identity_manager/objc",
"//components/strings",
"//ios/chrome/app/strings",
"//ios/chrome/browser/application_context",
"//ios/chrome/browser/browser_state",
"//ios/chrome/browser/browser_state:test_support",
"//ios/chrome/browser/favicon",
"//ios/chrome/browser/main:test_support",
"//ios/chrome/browser/net:crurl",
"//ios/chrome/browser/passwords",
"//ios/chrome/browser/passwords:save_passwords_consumer",
"//ios/chrome/browser/passwords:store_factory",
"//ios/chrome/browser/shared/ui/table_view:test_support",
"//ios/chrome/browser/shared/ui/table_view/cells",
"//ios/chrome/browser/shared/ui/util",
"//ios/chrome/browser/signin:signin",
"//ios/chrome/browser/signin:test_support",
"//ios/chrome/browser/sync:sync",
"//ios/chrome/browser/sync",
"//ios/chrome/browser/sync:test_support",
"//ios/chrome/browser/ui/settings/cells",
"//ios/chrome/browser/ui/settings/password/password_checkup:password_checkup_constants",
"//ios/chrome/browser/ui/settings/password/password_checkup:password_checkup_utils",
"//ios/chrome/browser/ui/settings/utils",
"//ios/chrome/common/ui/colors",
"//ios/chrome/common/ui/reauthentication",
"//ios/chrome/common/ui/table_view:cells_constants",
"//ios/chrome/test:test_support",
"//ios/chrome/test/app:test_support",
Expand All @@ -249,7 +217,6 @@ source_set("unit_tests") {
"//testing/gtest",
"//third_party/ocmock",
"//ui/base",
"//url",
]
}

Expand Down
104 changes: 104 additions & 0 deletions ios/chrome/browser/ui/settings/password/password_issues/BUILD.gn
Original file line number Diff line number Diff line change
@@ -0,0 +1,104 @@
# Copyright 2023 The Chromium Authors
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.

source_set("password_issues") {
configs += [ "//build/config/compiler:enable_arc" ]
sources = [
"password_issues_coordinator.h",
"password_issues_coordinator.mm",
"password_issues_mediator.h",
"password_issues_mediator.mm",
]
deps = [
":password_issues_ui",
"//base",
"//components/google/core/common",
"//components/password_manager/core/browser",
"//components/password_manager/core/common:features",
"//components/sync/driver",
"//ios/chrome/app/strings",
"//ios/chrome/browser/application_context",
"//ios/chrome/browser/favicon",
"//ios/chrome/browser/main:public",
"//ios/chrome/browser/net:crurl",
"//ios/chrome/browser/passwords",
"//ios/chrome/browser/shared/coordinator/chrome_coordinator",
"//ios/chrome/browser/shared/public/commands",
"//ios/chrome/browser/shared/ui/table_view",
"//ios/chrome/browser/shared/ui/table_view:utils",
"//ios/chrome/browser/sync",
"//ios/chrome/browser/ui/settings/password:common",
"//ios/chrome/browser/ui/settings/password/password_checkup:password_checkup_constants",
"//ios/chrome/browser/ui/settings/password/password_checkup:password_checkup_utils",
"//ios/chrome/browser/ui/settings/password/password_details",
"//ios/chrome/common/ui/favicon:favicon_constants",
"//ios/chrome/common/ui/reauthentication",
"//ui/base",
]
}

source_set("password_issues_ui") {
configs += [ "//build/config/compiler:enable_arc" ]
sources = [
"password_issue.h",
"password_issue.mm",
"password_issue_content_item.h",
"password_issue_content_item.mm",
"password_issue_group.h",
"password_issue_group.mm",
"password_issues_consumer.h",
"password_issues_presenter.h",
"password_issues_table_view_controller.h",
"password_issues_table_view_controller.mm",
]
deps = [
"//base",
"//components/password_manager/core/browser",
"//ios/chrome/app/strings",
"//ios/chrome/browser/net:crurl",
"//ios/chrome/browser/shared/ui/table_view",
"//ios/chrome/browser/shared/ui/table_view/cells",
"//ios/chrome/browser/ui/settings:settings_root",
"//ios/chrome/browser/ui/settings/password:password_constants",
"//ios/chrome/common/ui/favicon",
"//ui/base",
]
}

source_set("unit_tests") {
configs += [ "//build/config/compiler:enable_arc" ]
testonly = true
sources = [
"password_issues_mediator_unittest.mm",
"password_issues_table_view_controller_unittest.mm",
]
deps = [
":password_issues",
":password_issues_ui",
"//base",
"//base/test:test_support",
"//components/google/core/common",
"//components/keyed_service/core",
"//components/password_manager/core/browser",
"//components/password_manager/core/browser:test_support",
"//components/password_manager/core/common:features",
"//ios/chrome/app/strings",
"//ios/chrome/browser/application_context",
"//ios/chrome/browser/browser_state:test_support",
"//ios/chrome/browser/favicon",
"//ios/chrome/browser/main:test_support",
"//ios/chrome/browser/net:crurl",
"//ios/chrome/browser/passwords",
"//ios/chrome/browser/passwords:store_factory",
"//ios/chrome/browser/shared/ui/table_view:test_support",
"//ios/chrome/browser/sync",
"//ios/chrome/browser/sync:test_support",
"//ios/chrome/browser/ui/settings/password/password_checkup:password_checkup_constants",
"//ios/chrome/browser/ui/settings/password/password_checkup:password_checkup_utils",
"//ios/web/public/test",
"//testing/gmock",
"//testing/gtest",
"//ui/base",
]
}
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

#ifndef IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUE_H_
#define IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUE_H_
#ifndef IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUES_PASSWORD_ISSUE_H_
#define IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUES_PASSWORD_ISSUE_H_

#import <Foundation/Foundation.h>

Expand Down Expand Up @@ -32,4 +32,4 @@ struct CredentialUIEntry;

@end

#endif // IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUE_H_
#endif // IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUES_PASSWORD_ISSUE_H_
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

#import "ios/chrome/browser/ui/settings/password/password_issue.h"
#import "ios/chrome/browser/ui/settings/password/password_issues/password_issue.h"

#import "base/strings/sys_string_conversions.h"
#import "components/password_manager/core/browser/password_ui_utils.h"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

#ifndef IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUE_CONTENT_ITEM_H_
#define IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUE_CONTENT_ITEM_H_
#ifndef IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUES_PASSWORD_ISSUE_CONTENT_ITEM_H_
#define IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUES_PASSWORD_ISSUE_CONTENT_ITEM_H_

#import "ios/chrome/browser/shared/ui/table_view/cells/table_view_url_item.h"
#import "ios/chrome/browser/ui/settings/password/password_issue.h"
#import "ios/chrome/browser/ui/settings/password/password_issues/password_issue.h"

// Table view item used by `PasswordIssuesTableViewController`. It is created to
// hold `PasswordIssueWithForm`.
Expand All @@ -18,4 +18,4 @@

@end

#endif // IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUE_CONTENT_ITEM_H_
#endif // IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUES_PASSWORD_ISSUE_CONTENT_ITEM_H_
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

#import "ios/chrome/browser/ui/settings/password/password_issue_content_item.h"
#import "ios/chrome/browser/ui/settings/password/password_issues/password_issue_content_item.h"

#if !defined(__has_feature) || !__has_feature(objc_arc)
#error "This file requires ARC support."
Expand All @@ -11,8 +11,9 @@
@implementation PasswordIssueContentItem

- (void)setPassword:(PasswordIssue*)password {
if (_password == password)
if (_password == password) {
return;
}
_password = password;
self.title = password.website;
self.detailText = password.username;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,12 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

#ifndef IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUE_GROUP_H_
#define IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUE_GROUP_H_
#ifndef IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUES_PASSWORD_ISSUE_GROUP_H_
#define IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUES_PASSWORD_ISSUE_GROUP_H_

#import <Foundation/Foundation.h>

#import "ios/chrome/browser/ui/settings/password/password_issue.h"
#import "ios/chrome/browser/ui/settings/password/password_issues/password_issue.h"

// Data model for PasswordIssuesTableViewController.
// Represents a set of password issues displayed together in the UI with an
Expand All @@ -28,4 +28,4 @@

@end

#endif // IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUE_GROUP_H_
#endif // IOS_CHROME_BROWSER_UI_SETTINGS_PASSWORD_PASSWORD_ISSUES_PASSWORD_ISSUE_GROUP_H_
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

#import "ios/chrome/browser/ui/settings/password/password_issue_group.h"
#import "ios/chrome/browser/ui/settings/password/password_issues/password_issue_group.h"

#import "base/check.h"

Expand Down

0 comments on commit fee5b52

Please sign in to comment.