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

Side by Side Diff: chrome/browser/extensions/api/passwords_private/passwords_private_apitest.cc

Issue 1558633002: Cleanup: Remove double semicolons. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Convert CP code to a while loop, fix nit Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 #include <stddef.h> 5 #include <stddef.h>
6 6
7 #include <sstream> 7 #include <sstream>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 50
51 // A test PasswordsPrivateDelegate implementation which uses mock data. 51 // A test PasswordsPrivateDelegate implementation which uses mock data.
52 // TestDelegate starts out with kNumMocks mocks of each type (saved password 52 // TestDelegate starts out with kNumMocks mocks of each type (saved password
53 // and password exception) and removes one mock each time RemoveSavedPassword() 53 // and password exception) and removes one mock each time RemoveSavedPassword()
54 // or RemovePasswordException() is called. 54 // or RemovePasswordException() is called.
55 class TestDelegate : public PasswordsPrivateDelegate { 55 class TestDelegate : public PasswordsPrivateDelegate {
56 public: 56 public:
57 TestDelegate() : observers_(new base::ObserverListThreadSafe<Observer>()) { 57 TestDelegate() : observers_(new base::ObserverListThreadSafe<Observer>()) {
58 // Create mock data. 58 // Create mock data.
59 for (size_t i = 0; i < kNumMocks; i++) { 59 for (size_t i = 0; i < kNumMocks; i++) {
60 current_entries_.push_back(CreateEntry(i));; 60 current_entries_.push_back(CreateEntry(i));
61 current_exceptions_.push_back(CreateException(i)); 61 current_exceptions_.push_back(CreateException(i));
62 } 62 }
63 } 63 }
64 ~TestDelegate() override {} 64 ~TestDelegate() override {}
65 65
66 void AddObserver(Observer* observer) override { 66 void AddObserver(Observer* observer) override {
67 observers_->AddObserver(observer); 67 observers_->AddObserver(observer);
68 SendSavedPasswordsList(); 68 SendSavedPasswordsList();
69 SendPasswordExceptionsList(); 69 SendPasswordExceptionsList();
70 } 70 }
(...skipping 116 matching lines...) Expand 10 before | Expand all | Expand 10 after
187 187
188 IN_PROC_BROWSER_TEST_F(PasswordsPrivateApiTest, RemovePasswordException) { 188 IN_PROC_BROWSER_TEST_F(PasswordsPrivateApiTest, RemovePasswordException) {
189 EXPECT_TRUE(RunPasswordsSubtest("removePasswordException")) << message_; 189 EXPECT_TRUE(RunPasswordsSubtest("removePasswordException")) << message_;
190 } 190 }
191 191
192 IN_PROC_BROWSER_TEST_F(PasswordsPrivateApiTest, RequestPlaintextPassword) { 192 IN_PROC_BROWSER_TEST_F(PasswordsPrivateApiTest, RequestPlaintextPassword) {
193 EXPECT_TRUE(RunPasswordsSubtest("requestPlaintextPassword")) << message_; 193 EXPECT_TRUE(RunPasswordsSubtest("requestPlaintextPassword")) << message_;
194 } 194 }
195 195
196 } // namespace extensions 196 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698