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

Unified Diff: chrome/browser/android/chromium_application.cc

Issue 978653002: Upstream FirstRunActivity and friends. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix nits + remove MobileFreFinishState as it is no longer relevant Created 5 years, 9 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/chromium_application.cc
diff --git a/chrome/browser/android/chromium_application.cc b/chrome/browser/android/chromium_application.cc
index 0def8e8f4aafcab4fdfc93d6b0917713c7074556..cebc06a1da3c04a513b1e9aca7aa57934eb28217 100644
--- a/chrome/browser/android/chromium_application.cc
+++ b/chrome/browser/android/chromium_application.cc
@@ -6,17 +6,58 @@
#include "base/android/jni_android.h"
#include "base/android/jni_string.h"
+#include "base/prefs/pref_service.h"
#include "chrome/browser/android/tab_android.h"
+#include "chrome/browser/browser_process.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_manager.h"
#include "chrome/common/chrome_content_client.h"
+#include "content/public/browser/browser_thread.h"
#include "content/public/browser/web_contents.h"
#include "jni/ChromiumApplication_jni.h"
+#include "net/cookies/cookie_monster.h"
+#include "net/url_request/url_request_context.h"
+#include "net/url_request/url_request_context_getter.h"
using base::android::ConvertUTF8ToJavaString;
+namespace {
+void FlushCookiesOnIOThread(
+ scoped_refptr<net::URLRequestContextGetter> getter) {
+ DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::IO));
+ getter->GetURLRequestContext()
+ ->cookie_store()
+ ->GetCookieMonster()
+ ->FlushStore(base::Closure());
+}
+
+void CommitPendingWritesForProfile(Profile* profile) {
+ // Both of these calls are asynchronous. They may not finish (and may not
+ // even start!) before the Android OS kills our process. But we can't wait
+ // for them to finish because blocking the UI thread is illegal.
+ profile->GetPrefs()->CommitPendingWrite();
+ content::BrowserThread::PostTask(
+ content::BrowserThread::IO, FROM_HERE,
+ base::Bind(&FlushCookiesOnIOThread,
+ make_scoped_refptr(profile->GetRequestContext())));
+}
+} // namespace
+
static jstring GetBrowserUserAgent(JNIEnv* env, jclass clazz) {
return ConvertUTF8ToJavaString(env, GetUserAgent()).Release();
}
+static void FlushPersistentData(JNIEnv* env, jclass obj) {
+ // Commit the prending writes for all the loaded profiles.
+ std::vector<Profile*> loaded_profiles =
+ g_browser_process->profile_manager()->GetLoadedProfiles();
+ std::for_each(loaded_profiles.begin(), loaded_profiles.end(),
+ CommitPendingWritesForProfile);
+
+ if (g_browser_process->local_state())
+ g_browser_process->local_state()->CommitPendingWrite();
+}
+
namespace chrome {
namespace android {

Powered by Google App Engine
This is Rietveld 408576698