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

Side by Side Diff: build/android/findbugs_filter/findbugs_known_bugs.txt

Issue 406023002: Restructuring NavigationController functionalities from ContentViewCore to NavigationController (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased the patch with findbugs_known_bugs.txt changes. Created 6 years, 3 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 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeArrayCoercionTest.java 1 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeArrayCoercionTest.java
2 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeArrayTest.java 2 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeArrayTest.java
3 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeBasicsTest.java 3 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeBasicsTest.java
4 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeChildFrameTest.java 4 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeChildFrameTest.java
5 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeCoercionTest.java 5 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeCoercionTest.java
6 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeFieldsTest.java 6 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeFieldsTest.java
7 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeReturnValuesTest.java 7 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeReturnValuesTest.java
8 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeTestBase.java 8 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At JavaBridgeTestBase.java
9 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At PerfTraceEvent.java 9 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At PerfTraceEvent.java
10 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At SimpleSynchronizedMethod.java 10 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At SimpleSynchronizedMethod.java
11 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At SimpleSynchronizedStaticMethod.java 11 M C CSM: Shouldn't use synchronized method, please narrow down the synchronizati on scope. At SimpleSynchronizedStaticMethod.java
12 M C CST: Shouldn't use synchronized(this), please narrow down the synchronizatio n scope. At SimpleSynchronizedThis.java 12 M C CST: Shouldn't use synchronized(this), please narrow down the synchronizatio n scope. At SimpleSynchronizedThis.java
13 M D DMI: Hard coded reference to an absolute pathname in org.chromium.android_we bview.test.ArchiveTest.testAutoBadPath() At ArchiveTest.java 13 M D DMI: Hard coded reference to an absolute pathname in org.chromium.android_we bview.test.ArchiveTest.testAutoBadPath() At ArchiveTest.java
14 M D DMI: Hard coded reference to an absolute pathname in org.chromium.android_we bview.test.ArchiveTest.testExplicitBadPath() At ArchiveTest.java 14 M D DMI: Hard coded reference to an absolute pathname in org.chromium.android_we bview.test.ArchiveTest.testExplicitBadPath() At ArchiveTest.java
15 M D SF: Switch statement found in org.chromium.chrome.browser.ChromeBrowserProvi der.insert(Uri, ContentValues) where one case falls through to the next case At ChromeBrowserProvider.java 15 M D SF: Switch statement found in org.chromium.chrome.browser.ChromeBrowserProvi der.insert(Uri, ContentValues) where one case falls through to the next case At ChromeBrowserProvider.java
16 M M UG: org.chromium.content.browser.JavaBridgeReturnValuesTest$TestObject.getBo oleanValue() is unsynchronized, org.chromium.content.browser.JavaBridgeReturnVal uesTest$TestObject.setBooleanValue(boolean) is synchronized At JavaBridgeReturn ValuesTest.java 16 M M UG: org.chromium.content.browser.JavaBridgeReturnValuesTest$TestObject.getBo oleanValue() is unsynchronized, org.chromium.content.browser.JavaBridgeReturnVal uesTest$TestObject.setBooleanValue(boolean) is synchronized At JavaBridgeReturn ValuesTest.java
17 M M UG: org.chromium.content.browser.JavaBridgeReturnValuesTest$TestObject.getSt ringValue() is unsynchronized, org.chromium.content.browser.JavaBridgeReturnValu esTest$TestObject.setStringValue(String) is synchronized At JavaBridgeReturnVal uesTest.java 17 M M UG: org.chromium.content.browser.JavaBridgeReturnValuesTest$TestObject.getSt ringValue() is unsynchronized, org.chromium.content.browser.JavaBridgeReturnValu esTest$TestObject.setStringValue(String) is synchronized At JavaBridgeReturnVal uesTest.java
18 M V EI2: org.chromium.chrome.browser.ChromeBrowserProvider$BookmarkNode.setFavic on(byte[]) may expose internal representation by storing an externally mutable o bject into ChromeBrowserProvider$BookmarkNode.mFavicon At ChromeBrowserProvider .java 18 M V EI2: org.chromium.chrome.browser.ChromeBrowserProvider$BookmarkNode.setFavic on(byte[]) may expose internal representation by storing an externally mutable o bject into ChromeBrowserProvider$BookmarkNode.mFavicon At ChromeBrowserProvider .java
19 M V EI2: org.chromium.chrome.browser.ChromeBrowserProvider$BookmarkNode.setThumb nail(byte[]) may expose internal representation by storing an externally mutable object into ChromeBrowserProvider$BookmarkNode.mThumbnail At ChromeBrowserProv ider.java 19 M V EI2: org.chromium.chrome.browser.ChromeBrowserProvider$BookmarkNode.setThumb nail(byte[]) may expose internal representation by storing an externally mutable object into ChromeBrowserProvider$BookmarkNode.mThumbnail At ChromeBrowserProv ider.java
20 M V EI2: org.chromium.content.browser.LoadUrlParams.setPostData(byte[]) may expo se internal representation by storing an externally mutable object into LoadUrlP arams.mPostData At LoadUrlParams.java
21 M V EI: org.chromium.chrome.browser.ChromeBrowserProvider$BookmarkNode.favicon() may expose internal representation by returning ChromeBrowserProvider$BookmarkN ode.mFavicon At ChromeBrowserProvider.java 20 M V EI: org.chromium.chrome.browser.ChromeBrowserProvider$BookmarkNode.favicon() may expose internal representation by returning ChromeBrowserProvider$BookmarkN ode.mFavicon At ChromeBrowserProvider.java
22 M V EI: org.chromium.chrome.browser.ChromeBrowserProvider$BookmarkNode.thumbnail () may expose internal representation by returning ChromeBrowserProvider$Bookmar kNode.mThumbnail At ChromeBrowserProvider.java 21 M V EI: org.chromium.chrome.browser.ChromeBrowserProvider$BookmarkNode.thumbnail () may expose internal representation by returning ChromeBrowserProvider$Bookmar kNode.mThumbnail At ChromeBrowserProvider.java
23 M V EI: org.chromium.content.browser.LoadUrlParams.getPostData() may expose inte rnal representation by returning LoadUrlParams.mPostData At LoadUrlParams.java
24 M M LI: Incorrect lazy initialization of static field org.chromium.chrome.browse r.sync.ProfileSyncService.sSyncSetupManager in org.chromium.chrome.browser.sync. ProfileSyncService.get(Context) At ProfileSyncService.java 22 M M LI: Incorrect lazy initialization of static field org.chromium.chrome.browse r.sync.ProfileSyncService.sSyncSetupManager in org.chromium.chrome.browser.sync. ProfileSyncService.get(Context) At ProfileSyncService.java
25 M V EI: org.chromium.android_webview.AwWebResourceResponse.getResponseHeaderName s() may expose internal representation by returning AwWebResourceResponse.mRespo nseHeaderNames At AwWebResourceResponse.java 23 M V EI: org.chromium.android_webview.AwWebResourceResponse.getResponseHeaderName s() may expose internal representation by returning AwWebResourceResponse.mRespo nseHeaderNames At AwWebResourceResponse.java
26 M V EI: org.chromium.android_webview.AwWebResourceResponse.getResponseHeaderValu es() may expose internal representation by returning AwWebResourceResponse.mResp onseHeaderValues At AwWebResourceResponse.java 24 M V EI: org.chromium.android_webview.AwWebResourceResponse.getResponseHeaderValu es() may expose internal representation by returning AwWebResourceResponse.mResp onseHeaderValues At AwWebResourceResponse.java
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698