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

Side by Side Diff: chrome/browser/browsing_data/browsing_data_cookie_helper_unittest.cc

Issue 625113002: replace OVERRIDE and FINAL with override and final in chrome/browser/[a-i]* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix newly added OVERRIDEs Created 6 years, 2 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "chrome/browser/browsing_data/browsing_data_cookie_helper.h" 5 #include "chrome/browser/browsing_data/browsing_data_cookie_helper.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/run_loop.h" 8 #include "base/run_loop.h"
9 #include "chrome/test/base/testing_profile.h" 9 #include "chrome/test/base/testing_profile.h"
10 #include "content/public/test/test_browser_thread_bundle.h" 10 #include "content/public/test/test_browser_thread_bundle.h"
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 bool ExpectationIsMatched(const CookieExpectation& expectation) { 57 bool ExpectationIsMatched(const CookieExpectation& expectation) {
58 return expectation.matched_; 58 return expectation.matched_;
59 } 59 }
60 60
61 class BrowsingDataCookieHelperTest : public testing::Test { 61 class BrowsingDataCookieHelperTest : public testing::Test {
62 public: 62 public:
63 BrowsingDataCookieHelperTest() 63 BrowsingDataCookieHelperTest()
64 : testing_profile_(new TestingProfile()) { 64 : testing_profile_(new TestingProfile()) {
65 } 65 }
66 66
67 virtual void SetUp() OVERRIDE { cookie_expectations_.clear(); } 67 virtual void SetUp() override { cookie_expectations_.clear(); }
68 68
69 // Adds an expectation for a cookie that satisfies the given parameters. 69 // Adds an expectation for a cookie that satisfies the given parameters.
70 void AddCookieExpectation(const char* source, 70 void AddCookieExpectation(const char* source,
71 const char* domain, 71 const char* domain,
72 const char* path, 72 const char* path,
73 const char* name, 73 const char* name,
74 const char* value) { 74 const char* value) {
75 CookieExpectation matcher; 75 CookieExpectation matcher;
76 if (source) 76 if (source)
77 matcher.source_ = source; 77 matcher.source_ = source;
(...skipping 416 matching lines...) Expand 10 before | Expand all | Expand 10 after
494 // "A=1; 494 // "A=1;
495 // "A=3; Domain=www.google.com" 495 // "A=3; Domain=www.google.com"
496 // Add a domain cookie and check if it increases the cookie count. 496 // Add a domain cookie and check if it increases the cookie count.
497 helper->AddChangedCookie(frame2_url, frame1_url, 497 helper->AddChangedCookie(frame2_url, frame1_url,
498 cookie_pair4 + "; Domain=" + cookie_domain, 498 cookie_pair4 + "; Domain=" + cookie_domain,
499 net::CookieOptions()); 499 net::CookieOptions());
500 EXPECT_EQ(5U, helper->GetCookieCount()); 500 EXPECT_EQ(5U, helper->GetCookieCount());
501 } 501 }
502 502
503 } // namespace 503 } // namespace
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698