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

Unified Diff: android_webview/java/src/org/chromium/android_webview/AwSettings.java

Issue 98853009: Fixing Java style issues in android_webview. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Bo's nits Created 7 years 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: android_webview/java/src/org/chromium/android_webview/AwSettings.java
diff --git a/android_webview/java/src/org/chromium/android_webview/AwSettings.java b/android_webview/java/src/org/chromium/android_webview/AwSettings.java
index a8a4c888a42c0ccb858c7b3b5fba4d0292ee5c73..d829b8e6aece89ee6750ecff631eb0fd58021554 100644
--- a/android_webview/java/src/org/chromium/android_webview/AwSettings.java
+++ b/android_webview/java/src/org/chromium/android_webview/AwSettings.java
@@ -1,4 +1,4 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright 2012 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
@@ -137,19 +137,19 @@ public class AwSettings {
void bindUiThread() {
if (mHandler != null) return;
mHandler = new Handler(ThreadUtils.getUiThreadLooper()) {
- @Override
- public void handleMessage(Message msg) {
- switch (msg.what) {
- case UPDATE_WEBKIT_PREFERENCES:
- synchronized (mAwSettingsLock) {
- updateWebkitPreferencesOnUiThreadLocked();
- mIsUpdateWebkitPrefsMessagePending = false;
- mAwSettingsLock.notifyAll();
- }
- break;
- }
+ @Override
+ public void handleMessage(Message msg) {
+ switch (msg.what) {
+ case UPDATE_WEBKIT_PREFERENCES:
+ synchronized (mAwSettingsLock) {
+ updateWebkitPreferencesOnUiThreadLocked();
+ mIsUpdateWebkitPrefsMessagePending = false;
+ mAwSettingsLock.notifyAll();
+ }
+ break;
}
- };
+ }
+ };
}
void maybeRunOnUiThreadBlocking(Runnable r) {
@@ -396,7 +396,7 @@ public class AwSettings {
* See {@link android.webkit.WebSettings#setNeedInitialFocus}.
*/
public boolean shouldFocusFirstNode() {
- synchronized(mAwSettingsLock) {
+ synchronized (mAwSettingsLock) {
return mShouldFocusFirstNode;
}
}
@@ -1188,9 +1188,9 @@ public class AwSettings {
* See {@link android.webkit.WebSettings#getDomStorageEnabled}.
*/
public boolean getDomStorageEnabled() {
- synchronized (mAwSettingsLock) {
- return mDomStorageEnabled;
- }
+ synchronized (mAwSettingsLock) {
+ return mDomStorageEnabled;
+ }
}
@CalledByNative
@@ -1214,9 +1214,9 @@ public class AwSettings {
* See {@link android.webkit.WebSettings#getDatabaseEnabled}.
*/
public boolean getDatabaseEnabled() {
- synchronized (mAwSettingsLock) {
- return mDatabaseEnabled;
- }
+ synchronized (mAwSettingsLock) {
+ return mDatabaseEnabled;
+ }
}
@CalledByNative

Powered by Google App Engine
This is Rietveld 408576698