Skip to content

Commit

Permalink
Deprecate returning a GURL from GURL::GetOrigin().
Browse files Browse the repository at this point in the history
This patch renames `GURL::GetOrigin()` to
`GURL::DeprecatedGetOriginAsURL()`. A subsequent patch will
reintroduce a `GURL::GetOrigin()` that returns a `url::Origin`
rather than a `GURL`.

This patch was brought to you by sed, followed by a little manual
work to deal with look-alike methods (most notably
`CascadePriority::GetOrigin()` and `NudgeTracker::GetOrigin()`) and
two or three spots that held pointers to GURLs:

```
git grep -n -l ".GetOrigin(" | \
  xargs -L1 sed -i '' \
    -e 's/\.GetOrigin(/.DeprecatedGetOriginAsURL(/g'
```

Bug: 512374
Change-Id: I03fb290650b8759d9849af0911cad9bdc92a5132
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3220292
Commit-Queue: Mike West <mkwst@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Owners-Override: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/main@{#931431}
  • Loading branch information
mikewest authored and Chromium LUCI CQ committed Oct 14, 2021
1 parent 0feb660 commit 800532c
Show file tree
Hide file tree
Showing 453 changed files with 1,727 additions and 1,286 deletions.
6 changes: 3 additions & 3 deletions android_webview/browser/aw_contents.cc
Original file line number Diff line number Diff line change
Expand Up @@ -593,7 +593,7 @@ void AwContents::ShowGeolocationPrompt(const GURL& requesting_frame,
PermissionCallback callback) {
DCHECK_CURRENTLY_ON(BrowserThread::UI);

GURL origin = requesting_frame.GetOrigin();
GURL origin = requesting_frame.DeprecatedGetOriginAsURL();
bool show_prompt = pending_geolocation_prompts_.empty();
pending_geolocation_prompts_.emplace_back(origin, std::move(callback));
if (show_prompt) {
Expand All @@ -612,7 +612,7 @@ void AwContents::InvokeGeolocationCallback(
return;

GURL callback_origin(base::android::ConvertJavaStringToUTF16(env, origin));
if (callback_origin.GetOrigin() ==
if (callback_origin.DeprecatedGetOriginAsURL() ==
pending_geolocation_prompts_.front().first) {
std::move(pending_geolocation_prompts_.front().second).Run(value);
pending_geolocation_prompts_.pop_front();
Expand All @@ -628,7 +628,7 @@ void AwContents::HideGeolocationPrompt(const GURL& origin) {
bool removed_current_outstanding_callback = false;
std::list<OriginCallback>::iterator it = pending_geolocation_prompts_.begin();
while (it != pending_geolocation_prompts_.end()) {
if ((*it).first == origin.GetOrigin()) {
if ((*it).first == origin.DeprecatedGetOriginAsURL()) {
if (it == pending_geolocation_prompts_.begin()) {
removed_current_outstanding_callback = true;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ void PermissionRequestHandler::PreauthorizePermission(const GURL& origin,
if (!resources)
return;

std::string key = origin.GetOrigin().spec();
std::string key = origin.DeprecatedGetOriginAsURL().spec();
if (key.empty()) {
LOG(ERROR) << "The origin of preauthorization is empty, ignore it.";
return;
Expand Down Expand Up @@ -133,7 +133,7 @@ void PermissionRequestHandler::PruneRequests() {
bool PermissionRequestHandler::Preauthorized(const GURL& origin,
int64_t resources) {
std::map<std::string, int64_t>::iterator i =
preauthorized_permission_.find(origin.GetOrigin().spec());
preauthorized_permission_.find(origin.DeprecatedGetOriginAsURL().spec());

return i != preauthorized_permission_.end() &&
(resources & i->second) == resources;
Expand Down
4 changes: 2 additions & 2 deletions chrome/browser/android/shortcut_helper.cc
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ void ShortcutHelper::StoreWebappSplashImage(const std::string& webapp_id,

// static
bool ShortcutHelper::DoesOriginContainAnyInstalledWebApk(const GURL& origin) {
DCHECK_EQ(origin, origin.GetOrigin());
DCHECK_EQ(origin, origin.DeprecatedGetOriginAsURL());
JNIEnv* env = base::android::AttachCurrentThread();
base::android::ScopedJavaLocalRef<jstring> java_origin =
base::android::ConvertUTF8ToJavaString(env, origin.spec());
Expand All @@ -145,7 +145,7 @@ bool ShortcutHelper::DoesOriginContainAnyInstalledWebApk(const GURL& origin) {

bool ShortcutHelper::DoesOriginContainAnyInstalledTrustedWebActivity(
const GURL& origin) {
DCHECK_EQ(origin, origin.GetOrigin());
DCHECK_EQ(origin, origin.DeprecatedGetOriginAsURL());
JNIEnv* env = base::android::AttachCurrentThread();
base::android::ScopedJavaLocalRef<jstring> java_origin =
base::android::ConvertUTF8ToJavaString(env, origin.spec());
Expand Down
9 changes: 6 additions & 3 deletions chrome/browser/apps/guest_view/web_view_browsertest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -3991,7 +3991,8 @@ IN_PROC_BROWSER_TEST_F(WebViewTest, ReloadWebviewAccessibleResource) {
ASSERT_TRUE(web_view_contents);

GURL embedder_url(embedder_contents->GetLastCommittedURL());
GURL webview_url(embedder_url.GetOrigin().spec() + "assets/foo.html");
GURL webview_url(embedder_url.DeprecatedGetOriginAsURL().spec() +
"assets/foo.html");

EXPECT_EQ(webview_url, web_view_contents->GetLastCommittedURL());
}
Expand All @@ -4009,7 +4010,8 @@ IN_PROC_BROWSER_TEST_F(WebViewTest, BlobInWebviewAccessibleResource) {
ASSERT_TRUE(web_view_contents);

GURL embedder_url(embedder_contents->GetLastCommittedURL());
GURL webview_url(embedder_url.GetOrigin().spec() + "assets/foo.html");
GURL webview_url(embedder_url.DeprecatedGetOriginAsURL().spec() +
"assets/foo.html");

EXPECT_EQ(webview_url, web_view_contents->GetLastCommittedURL());

Expand Down Expand Up @@ -4040,7 +4042,8 @@ IN_PROC_BROWSER_TEST_F(WebViewTest, LoadWebviewInaccessibleResource) {
// Check that the webview stays at the first page that it loaded (foo.html),
// and does not commit inaccessible.html.
GURL embedder_url(embedder_contents->GetLastCommittedURL());
GURL foo_url(embedder_url.GetOrigin().spec() + "assets/foo.html");
GURL foo_url(embedder_url.DeprecatedGetOriginAsURL().spec() +
"assets/foo.html");

EXPECT_EQ(foo_url, web_view_contents->GetLastCommittedURL());
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -118,8 +118,9 @@ GURL RedirectUrlIfSwa(Profile* profile,

// Projector:
if (app_id == chromeos::kChromeUITrustedProjectorSwaAppId &&
url.GetOrigin() ==
GURL(chromeos::kChromeUIUntrustedProjectorPwaUrl).GetOrigin()) {
url.DeprecatedGetOriginAsURL() ==
GURL(chromeos::kChromeUIUntrustedProjectorPwaUrl)
.DeprecatedGetOriginAsURL()) {
std::string override_url = chromeos::kChromeUITrustedProjectorAppUrl;
if (url.path().length() > 1)
override_url += url.path().substr(1);
Expand Down
4 changes: 2 additions & 2 deletions chrome/browser/ash/attestation/attestation_ca_client.cc
Original file line number Diff line number Diff line change
Expand Up @@ -255,8 +255,8 @@ void AttestationCAClient::CheckIfAnyProxyPresent(
g_browser_process->system_network_context_manager()->GetContext();
}

CAProxyLookupClient::LookUpProxyForURL(network_context, url.GetOrigin(),
std::move(callback));
CAProxyLookupClient::LookUpProxyForURL(
network_context, url.DeprecatedGetOriginAsURL(), std::move(callback));
}

} // namespace attestation
Expand Down
6 changes: 4 additions & 2 deletions chrome/browser/ash/crosapi/url_handler_ash.cc
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,8 @@ void UrlHandlerAsh::BindReceiver(

void UrlHandlerAsh::OpenUrl(const GURL& url) {
// Settings will be handled.
if (url.GetOrigin() == GURL(chrome::kChromeUIOSSettingsURL).GetOrigin()) {
if (url.DeprecatedGetOriginAsURL() ==
GURL(chrome::kChromeUIOSSettingsURL).DeprecatedGetOriginAsURL()) {
chrome::SettingsWindowManager* settings_window_manager =
chrome::SettingsWindowManager::GetInstance();
settings_window_manager->ShowChromePageForProfile(
Expand All @@ -36,7 +37,8 @@ void UrlHandlerAsh::OpenUrl(const GURL& url) {
}

// TODO(crbug/1256481): Only accept URL's from the Ash supplied allow list.
if (url.GetOrigin() == GURL(chrome::kChromeUIFlagsURL).GetOrigin()) {
if (url.DeprecatedGetOriginAsURL() ==
GURL(chrome::kChromeUIFlagsURL).DeprecatedGetOriginAsURL()) {
if (!url_window_manager_) {
url_window_manager_ = std::make_unique<ChromeUrlWindowManager>();
url_window_observer_ =
Expand Down
2 changes: 1 addition & 1 deletion chrome/browser/ash/crostini/crostini_terminal.cc
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ void LaunchTerminal(Profile* profile,
void LaunchTerminalWithUrl(Profile* profile,
int64_t display_id,
const GURL& url) {
if (url.GetOrigin() != chrome::kChromeUIUntrustedTerminalURL) {
if (url.DeprecatedGetOriginAsURL() != chrome::kChromeUIUntrustedTerminalURL) {
LOG(ERROR) << "Trying to launch terminal with an invalid url: " << url;
return;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2282,7 +2282,7 @@ void FileManagerBrowserTestBase::OnCommand(const std::string& name,
// Obtain the mock CWS widget container URL and URL.origin.
const GURL url = embedded_test_server()->GetURL(
"/chromeos/file_manager/cws_container_mock/index.html");
std::string origin = url.GetOrigin().spec();
std::string origin = url.DeprecatedGetOriginAsURL().spec();
if (*origin.rbegin() == '/') // Strip origin trailing '/'.
origin.resize(origin.length() - 1);

Expand Down
3 changes: 2 additions & 1 deletion chrome/browser/ash/file_manager/file_watcher_unittest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,8 @@ TEST_F(FileManagerFileWatcherTest, AddSameExtensionMultipleTimes) {
const char kExtensionId[] = "extension-id";
GURL source_url =
extensions::Extension::GetBaseURLFromExtensionId(kExtensionId);
url::Origin origin_1 = url::Origin::Create(source_url.GetOrigin());
url::Origin origin_1 =
url::Origin::Create(source_url.DeprecatedGetOriginAsURL());
url::Origin origin_2 = url::Origin::Create(source_url);

FileWatcher file_watcher(kVirtualPath);
Expand Down
2 changes: 1 addition & 1 deletion chrome/browser/ash/file_manager/fileapi_util.cc
Original file line number Diff line number Diff line change
Expand Up @@ -517,7 +517,7 @@ const GURL GetFileManagerURL() {
}

bool IsFileManagerURL(const GURL& source_url) {
return GetFileManagerURL() == source_url.GetOrigin();
return GetFileManagerURL() == source_url.DeprecatedGetOriginAsURL();
}

storage::FileSystemContext* GetFileManagerFileSystemContext(Profile* profile) {
Expand Down
6 changes: 3 additions & 3 deletions chrome/browser/ash/login/arc_terms_of_service_browsertest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -252,9 +252,9 @@ class ArcTermsOfServiceScreenTest : public OobeBaseTest {
// The string will have the format "http://127.0.0.1:${PORT_NUMBER}" where
// PORT_NUMBER is a randomly assigned port number.
std::string TestServerBaseUrl() {
return std::string(
base::TrimString(embedded_test_server()->base_url().GetOrigin().spec(),
"/", base::TrimPositions::TRIM_TRAILING));
return std::string(base::TrimString(
embedded_test_server()->base_url().DeprecatedGetOriginAsURL().spec(),
"/", base::TrimPositions::TRIM_TRAILING));
}

// Handles both Terms of Service and Privacy policy requests.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -65,8 +65,9 @@ absl::optional<std::string> ReadFileToOptionalString(
}

std::string TestServerBaseUrl(net::EmbeddedTestServer* server) {
return std::string(base::TrimString(server->base_url().GetOrigin().spec(),
"/", base::TrimPositions::TRIM_TRAILING));
return std::string(
base::TrimString(server->base_url().DeprecatedGetOriginAsURL().spec(),
"/", base::TrimPositions::TRIM_TRAILING));
}

// Returns a successful `BasicHttpResponse` with `content`.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -227,7 +227,8 @@ const em::AppInfo AppInfoGenerator::ConvertToAppInfo(
// For web apps, publisher id is the start url.
GURL start_url(update.PublisherId());
DCHECK(start_url.is_valid());
const std::string launch_origin = start_url.GetOrigin().spec();
const std::string launch_origin =
start_url.DeprecatedGetOriginAsURL().spec();
info.set_app_id(launch_origin);
info.set_app_name(launch_origin);
}
Expand Down
7 changes: 4 additions & 3 deletions chrome/browser/autofill/captured_sites_test_utils.cc
Original file line number Diff line number Diff line change
Expand Up @@ -436,7 +436,8 @@ void IFrameWaiter::RenderFrameCreated(
run_loop_.Quit();
break;
case ORIGIN:
if (render_frame_host->GetLastCommittedURL().GetOrigin() == origin_)
if (render_frame_host->GetLastCommittedURL().DeprecatedGetOriginAsURL() ==
origin_)
run_loop_.Quit();
break;
case URL:
Expand All @@ -454,7 +455,7 @@ void IFrameWaiter::DidFinishLoad(content::RenderFrameHost* render_frame_host,
return;
switch (query_type_) {
case ORIGIN:
if (validated_url.GetOrigin() == origin_)
if (validated_url.DeprecatedGetOriginAsURL() == origin_)
run_loop_.Quit();
break;
case URL:
Expand Down Expand Up @@ -483,7 +484,7 @@ void IFrameWaiter::FrameNameChanged(content::RenderFrameHost* render_frame_host,
bool IFrameWaiter::FrameHasOrigin(const GURL& origin,
content::RenderFrameHost* frame) {
GURL url = frame->GetLastCommittedURL();
return (url.GetOrigin() == origin.GetOrigin());
return (url.DeprecatedGetOriginAsURL() == origin.DeprecatedGetOriginAsURL());
}

// TestRecipeReplayer ---------------------------------------------------------
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -426,7 +426,9 @@ CreditCardAccessoryControllerImpl::GetPromoCodeOffers() const {
return std::vector<const AutofillOfferData*>();

return personal_data_manager_->GetActiveAutofillPromoCodeOffersForOrigin(
autofill_manager->client()->GetLastCommittedURL().GetOrigin());
autofill_manager->client()
->GetLastCommittedURL()
.DeprecatedGetOriginAsURL());
}

base::WeakPtr<ManualFillingController>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ int BackgroundSyncDelegateImpl::GetSiteEngagementPenalty(const GURL& url) {
site_engagement_service_->GetEngagementLevel(url);
if (engagement_level == blink::mojom::EngagementLevel::NONE) {
suspended_periodic_sync_origins_.insert(
url::Origin::Create(url.GetOrigin()));
url::Origin::Create(url.DeprecatedGetOriginAsURL()));
}

switch (engagement_level) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,8 @@ class BackgroundSyncMetricsBrowserTest : public InProcessBrowserTest {
IN_PROC_BROWSER_TEST_F(BackgroundSyncMetricsBrowserTest,
OneShotBackgroundSyncUkmEventsAreRecorded) {
background_sync_metrics_->MaybeRecordOneShotSyncRegistrationEvent(
url::Origin::Create(embedded_test_server()->base_url().GetOrigin()),
url::Origin::Create(
embedded_test_server()->base_url().DeprecatedGetOriginAsURL()),
/* can_fire= */ true,
/* is_reregistered= */ false);
WaitForUkm();
Expand All @@ -79,7 +80,8 @@ IN_PROC_BROWSER_TEST_F(BackgroundSyncMetricsBrowserTest,
}

background_sync_metrics_->MaybeRecordOneShotSyncCompletionEvent(
url::Origin::Create(embedded_test_server()->base_url().GetOrigin()),
url::Origin::Create(
embedded_test_server()->base_url().DeprecatedGetOriginAsURL()),
/* status_code= */ blink::ServiceWorkerStatusCode::kOk,
/* num_attempts= */ 2, /* max_attempts= */ 5);
WaitForUkm();
Expand All @@ -102,7 +104,8 @@ IN_PROC_BROWSER_TEST_F(BackgroundSyncMetricsBrowserTest,
IN_PROC_BROWSER_TEST_F(BackgroundSyncMetricsBrowserTest,
PeriodicBackgroundSyncUkmEventsAreRecorded) {
background_sync_metrics_->MaybeRecordPeriodicSyncRegistrationEvent(
url::Origin::Create(embedded_test_server()->base_url().GetOrigin()),
url::Origin::Create(
embedded_test_server()->base_url().DeprecatedGetOriginAsURL()),
/* min_interval= */ 1000,
/* is_reregistered= */ false);
WaitForUkm();
Expand All @@ -123,7 +126,8 @@ IN_PROC_BROWSER_TEST_F(BackgroundSyncMetricsBrowserTest,
}

background_sync_metrics_->MaybeRecordPeriodicSyncEventCompletion(
url::Origin::Create(embedded_test_server()->base_url().GetOrigin()),
url::Origin::Create(
embedded_test_server()->base_url().DeprecatedGetOriginAsURL()),
/* status_code= */ blink::ServiceWorkerStatusCode::kOk,
/* num_attempts= */ 2, /* max_attempts= */ 5);
WaitForUkm();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ base::flat_set<GURL> GetDeletedOrigins(

bool Contains(const base::flat_set<GURL>& deleted_origins,
const GURL& template_gurl) {
return deleted_origins.contains(template_gurl.GetOrigin());
return deleted_origins.contains(template_gurl.DeprecatedGetOriginAsURL());
}

void DeleteTemplateUrlsForTimeRange(TemplateURLService* keywords_model,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2813,7 +2813,7 @@ TEST_F(ChromeBrowsingDataRemoverDelegateTest, OriginTypeMasks) {

auto mock_policy = base::MakeRefCounted<MockExtensionSpecialStoragePolicy>();
// Protect |kOriginProtected|.
mock_policy->AddProtected(kOriginProtected.GetOrigin());
mock_policy->AddProtected(kOriginProtected.DeprecatedGetOriginAsURL());

EXPECT_FALSE(Match(kOriginProtected, kUnprotected, mock_policy.get()));
EXPECT_TRUE(Match(kOriginUnprotected, kUnprotected, mock_policy.get()));
Expand Down
3 changes: 2 additions & 1 deletion chrome/browser/browsing_data/cookies_tree_model.cc
Original file line number Diff line number Diff line change
Expand Up @@ -277,7 +277,8 @@ CookieTreeNode::DetailedInfo& CookieTreeNode::DetailedInfo::InitSharedWorker(
const browsing_data::SharedWorkerHelper::SharedWorkerInfo* shared_worker) {
Init(TYPE_SHARED_WORKER);
shared_worker_info = shared_worker;
origin = url::Origin::Create(shared_worker_info->worker.GetOrigin());
origin = url::Origin::Create(
shared_worker_info->worker.DeprecatedGetOriginAsURL());
return *this;
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1912,7 +1912,8 @@ AutotestPrivateGetRegisteredSystemWebAppsFunction::Run() {
api::autotest_private::SystemApp system_app;
web_app::SystemWebAppDelegate* delegate = type_and_info.second.get();
system_app.internal_name = delegate->GetInternalName();
system_app.url = delegate->GetInstallUrl().GetOrigin().spec();
system_app.url =
delegate->GetInstallUrl().DeprecatedGetOriginAsURL().spec();
result.push_back(std::move(system_app));
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1232,7 +1232,7 @@ void FileManagerPrivateInternalGetRecentFilesFunction::OnGetRecentFiles(
file_manager::util::ConvertFileDefinitionListToEntryDefinitionList(
file_manager::util::GetFileSystemContextForSourceURL(profile,
source_url()),
url::Origin::Create(source_url().GetOrigin()),
url::Origin::Create(source_url().DeprecatedGetOriginAsURL()),
file_definition_list, // Safe, since copied internally.
base::BindOnce(&FileManagerPrivateInternalGetRecentFilesFunction::
OnConvertFileDefinitionListToEntryDefinitionList,
Expand Down
5 changes: 3 additions & 2 deletions chrome/browser/client_hints/client_hints_browsertest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -3151,7 +3151,8 @@ class ThirdPartyUaReducedOriginTrialBrowserTest
private:
// URLLoaderInterceptor callback
bool InterceptRequest(URLLoaderInterceptor::RequestParams* params) {
if (params->url_request.url.GetOrigin() != GURL(kFirstPartyOriginUrl)) {
if (params->url_request.url.DeprecatedGetOriginAsURL() !=
GURL(kFirstPartyOriginUrl)) {
return false;
}
if (params->url_request.url.path() !=
Expand Down Expand Up @@ -3417,7 +3418,7 @@ class ThirdPartyAcceptChUaReducedOriginTrialBrowserTest
// URLLoaderInterceptor callback. Handles the third-party embeds and
// subresource requests.
bool InterceptRequest(URLLoaderInterceptor::RequestParams* params) {
const GURL origin = params->url_request.url.GetOrigin();
const GURL origin = params->url_request.url.DeprecatedGetOriginAsURL();
// The interceptor does not handle requests to the EmbeddedTestServer.
// Requests also get sent to https://accounts.google.com/ (not sure from
// where), so we ignore them as well.
Expand Down
6 changes: 3 additions & 3 deletions chrome/browser/commerce/coupons/coupon_db.cc
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ CouponDB::CouponDB(content::BrowserContext* browser_context)
CouponDB::~CouponDB() = default;

void CouponDB::LoadCoupon(const GURL& origin, LoadCallback callback) {
DCHECK_EQ(origin.GetOrigin(), origin);
DCHECK_EQ(origin.DeprecatedGetOriginAsURL(), origin);
proto_db_->LoadOneEntry(origin.spec(), std::move(callback));
}

Expand All @@ -25,14 +25,14 @@ void CouponDB::LoadAllCoupons(LoadCallback callback) {

void CouponDB::AddCoupon(const GURL& origin,
const coupon_db::CouponContentProto& proto) {
DCHECK_EQ(origin.GetOrigin(), origin);
DCHECK_EQ(origin.DeprecatedGetOriginAsURL(), origin);
proto_db_->InsertContent(origin.spec(), proto,
base::BindOnce(&CouponDB::OnOperationFinished,
weak_ptr_factory_.GetWeakPtr()));
}

void CouponDB::DeleteCoupon(const GURL& origin) {
DCHECK_EQ(origin.GetOrigin(), origin);
DCHECK_EQ(origin.DeprecatedGetOriginAsURL(), origin);
proto_db_->DeleteOneEntry(origin.spec(),
base::BindOnce(&CouponDB::OnOperationFinished,
weak_ptr_factory_.GetWeakPtr()));
Expand Down
Loading

0 comments on commit 800532c

Please sign in to comment.