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

Unified Diff: chrome/browser/android/ntp/most_visited_sites_bridge.cc

Issue 2237943002: Remove now-unnecessary .obj() in Java method calls. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@switch-context
Patch Set: Rebase *again* :( Created 4 years, 4 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/android/ntp/most_visited_sites_bridge.cc
diff --git a/chrome/browser/android/ntp/most_visited_sites_bridge.cc b/chrome/browser/android/ntp/most_visited_sites_bridge.cc
index 802adee3ba4c804cca7505986ef6c3ef11c6c07b..7c0676f30d28ac3eb89531dda8ec52e4e06c726b 100644
--- a/chrome/browser/android/ntp/most_visited_sites_bridge.cc
+++ b/chrome/browser/android/ntp/most_visited_sites_bridge.cc
@@ -133,10 +133,10 @@ void MostVisitedSitesBridge::JavaObserver::OnMostVisitedURLsAvailable(
sources.emplace_back(static_cast<int>(tile.source));
}
Java_MostVisitedURLsObserver_onMostVisitedURLsAvailable(
- env, observer_.obj(), ToJavaArrayOfStrings(env, titles).obj(),
- ToJavaArrayOfStrings(env, urls).obj(),
- ToJavaArrayOfStrings(env, whitelist_icon_paths).obj(),
- ToJavaIntArray(env, sources).obj());
+ env, observer_, ToJavaArrayOfStrings(env, titles),
+ ToJavaArrayOfStrings(env, urls),
+ ToJavaArrayOfStrings(env, whitelist_icon_paths),
+ ToJavaIntArray(env, sources));
}
void MostVisitedSitesBridge::JavaObserver::OnPopularURLsAvailable(
@@ -151,9 +151,9 @@ void MostVisitedSitesBridge::JavaObserver::OnPopularURLsAvailable(
large_icon_urls.emplace_back(site.large_icon_url.spec());
}
Java_MostVisitedURLsObserver_onPopularURLsAvailable(
- env, observer_.obj(), ToJavaArrayOfStrings(env, urls).obj(),
- ToJavaArrayOfStrings(env, favicon_urls).obj(),
- ToJavaArrayOfStrings(env, large_icon_urls).obj());
+ env, observer_, ToJavaArrayOfStrings(env, urls),
+ ToJavaArrayOfStrings(env, favicon_urls),
+ ToJavaArrayOfStrings(env, large_icon_urls));
}
MostVisitedSitesBridge::MostVisitedSitesBridge(Profile* profile)
« no previous file with comments | « chrome/browser/android/net/external_estimate_provider_android.cc ('k') | chrome/browser/android/ntp/ntp_snippets_bridge.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698