Remove logs

Resolve lint warnings
This commit is contained in:
Deep
2022-09-21 17:58:41 -04:00
parent e6e13a9618
commit fb08838bee
8 changed files with 16 additions and 14 deletions
@@ -32,8 +32,8 @@ public class CookieListOptInServiceFactory {
public CookieListOptInPageAndroidHandler getCookieListOptInPageAndroidHandler(
ConnectionErrorHandler connectionErrorHandler) {
int nativeHandle = CookieListOptInServiceFactoryJni.get()
.getInterfaceToCookieListOptInPageAndroidHandler();
int nativeHandle =
CookieListOptInServiceFactoryJni.get().getInterfaceToCookieListOptInService();
MessagePipeHandle handle = wrapNativeHandle(nativeHandle);
CookieListOptInPageAndroidHandler cookieListOptInPageAndroidHandler =
CookieListOptInPageAndroidHandler.MANAGER.attachProxy(handle, 0);
@@ -49,6 +49,6 @@ public class CookieListOptInServiceFactory {
@NativeMethods
interface Natives {
int getInterfaceToCookieListOptInPageAndroidHandler();
int getInterfaceToCookieListOptInService();
}
}
}
@@ -347,7 +347,6 @@ public abstract class BraveToolbarLayoutImpl extends ToolbarLayout
@Override
public void onConnectionError(MojoException e) {
Log.e("NTP", "MojoException : " + e.getMessage());
mCookieListOptInPageAndroidHandler = null;
initCookieListOptInPageAndroidHandler();
}
@@ -364,7 +363,6 @@ public abstract class BraveToolbarLayoutImpl extends ToolbarLayout
@Override
protected void onNativeLibraryReady() {
Log.e("NTP", "onNativeLibraryReady : ");
super.onNativeLibraryReady();
initCookieListOptInPageAndroidHandler();
mBraveShieldsContentSettings = BraveShieldsContentSettings.getInstance();
+2 -1
View File
@@ -563,7 +563,8 @@ constexpr char kBraveVerticalTabsDescription[] =
{"brave-adblock-cookie-list-opt-in", \
flag_descriptions::kBraveAdblockCookieListOptInName, \
flag_descriptions::kBraveAdblockCookieListOptInDescription, \
kOsDesktop | kOsAndroid, FEATURE_VALUE_TYPE(kBraveAdblockCookieListOptIn)}, \
kOsDesktop | kOsAndroid, \
FEATURE_VALUE_TYPE(kBraveAdblockCookieListOptIn)}, \
{"brave-adblock-cosmetic-filtering", \
flag_descriptions::kBraveAdblockCosmeticFilteringName, \
flag_descriptions::kBraveAdblockCosmeticFilteringDescription, kOsAll, \
@@ -2,7 +2,6 @@
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this file,
* You can obtain one at http://mozilla.org/MPL/2.0/. */
#include "base/android/jni_android.h"
#include "brave/browser/brave_shields/cookie_list_opt_in_service_factory.h"
#include "brave/build/android/jni_headers/CookieListOptInServiceFactory_jni.h"
@@ -13,7 +12,7 @@
namespace chrome {
namespace android {
static jint
JNI_CookieListOptInServiceFactory_GetInterfaceToCookieListOptInPageAndroidHandler(
JNI_CookieListOptInServiceFactory_GetInterfaceToCookieListOptInService(
JNIEnv* env) {
auto* profile = ProfileManager::GetActiveUserProfile();
auto pending = brave_shields::CookieListOptInServiceFactory::GetInstance()
@@ -23,4 +22,4 @@ JNI_CookieListOptInServiceFactory_GetInterfaceToCookieListOptInPageAndroidHandle
}
} // namespace android
} // namespace chrome
} // namespace chrome
@@ -5,6 +5,8 @@
#include "brave/browser/brave_shields/cookie_list_opt_in_service_factory.h"
#include <utility>
#include "brave/components/brave_shields/browser/cookie_list_opt_in_service.h"
#include "chrome/browser/profiles/incognito_helpers.h"
#include "components/keyed_service/content/browser_context_dependency_manager.h"
@@ -59,4 +61,4 @@ content::BrowserContext* CookieListOptInServiceFactory::GetBrowserContextToUse(
return chrome::GetBrowserContextRedirectedInIncognito(context);
}
} // namespace brave_shields
} // namespace brave_shields
@@ -46,4 +46,4 @@ class CookieListOptInServiceFactory : public BrowserContextKeyedServiceFactory {
} // namespace brave_shields
#endif // BRAVE_BROWSER_BRAVE_SHIELDS_COOKIE_LIST_OPT_IN_SERVICE_FACTORY_H_
#endif // BRAVE_BROWSER_BRAVE_SHIELDS_COOKIE_LIST_OPT_IN_SERVICE_FACTORY_H_
@@ -5,6 +5,8 @@
#include "brave/components/brave_shields/browser/cookie_list_opt_in_service.h"
#include <utility>
#include "base/feature_list.h"
#include "brave/browser/brave_browser_process.h"
#include "brave/components/brave_shields/browser/ad_block_regional_service_manager.h"
@@ -56,4 +58,4 @@ void CookieListOptInService::EnableFilter(bool shouldEnableFilter) {
->EnableFilterList(brave_shields::kCookieListUuid, shouldEnableFilter);
}
} // namespace brave_shields
} // namespace brave_shields
@@ -41,4 +41,4 @@ class CookieListOptInService : public KeyedService,
} // namespace brave_shields
#endif // BRAVE_COMPONENTS_BRAVE_SHIELDS_BROWSER_COOKIE_LIST_OPT_IN_SERVICE_H_
#endif // BRAVE_COMPONENTS_BRAVE_SHIELDS_BROWSER_COOKIE_LIST_OPT_IN_SERVICE_H_