From d8581a7d1b31117deb9a381f64227e1bfa18980a Mon Sep 17 00:00:00 2001 From: Artem Samoilenko Date: Tue, 17 Feb 2026 14:18:08 -0500 Subject: [PATCH] [cr147][Android] Clean up for LogoMediator Chromium changes: https://chromium.googlesource.com/chromium/src/+/c779c30da8f7ab1a9ceb5301aa57c267bc0083ed commit c779c30da8f7ab1a9ceb5301aa57c267bc0083ed Author: Xi Han Date: Fri Feb 13 16:20:09 2026 -0800 [Logo] Clean up LogoView refactor code. The feature AndroidLogoViewRefactor was enabled by default on M143, so it is safe to clean up now. Bug: 484388593, 478251814 Change-Id: I7524e3b248fd19e66291818772fd2e2795dad63c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7577045 Reviewed-by: Henrique Nakashima Auto-Submit: Xi Han Commit-Queue: Xi Han Reviewed-by: Xinyi Ji Cr-Commit-Position: refs/heads/main@{#1584984} --- .../javatests/org/chromium/chrome/browser/BytecodeTest.java | 3 --- .../org/chromium/chrome/browser/logo/BraveLogoMediator.java | 5 ----- 2 files changed, 8 deletions(-) diff --git a/android/javatests/org/chromium/chrome/browser/BytecodeTest.java b/android/javatests/org/chromium/chrome/browser/BytecodeTest.java index 7fa36264b5e..ba8d7808360 100644 --- a/android/javatests/org/chromium/chrome/browser/BytecodeTest.java +++ b/android/javatests/org/chromium/chrome/browser/BytecodeTest.java @@ -88,7 +88,6 @@ import org.chromium.chrome.browser.hub.ResourceButtonData; import org.chromium.chrome.browser.infobar.InfoBarContainerView; import org.chromium.chrome.browser.layouts.LayoutManager; import org.chromium.chrome.browser.lifecycle.ActivityLifecycleDispatcher; -import org.chromium.chrome.browser.logo.CachedTintedBitmap; import org.chromium.chrome.browser.logo.LogoCoordinator; import org.chromium.chrome.browser.metrics.StartupMetricsTracker; import org.chromium.chrome.browser.multiwindow.BraveMultiWindowUtils; @@ -1866,12 +1865,10 @@ public class BytecodeTest { constructorsMatch( "org/chromium/chrome/browser/logo/LogoMediator", "org/chromium/chrome/browser/logo/BraveLogoMediator", - Context.class, Callback.class, PropertyModel.class, Callback.class, LogoCoordinator.VisibilityObserver.class, - CachedTintedBitmap.class, Drawable.class)); Assert.assertTrue( constructorsMatch( diff --git a/browser/ui/android/logo/java/src/org/chromium/chrome/browser/logo/BraveLogoMediator.java b/browser/ui/android/logo/java/src/org/chromium/chrome/browser/logo/BraveLogoMediator.java index 1826aa81388..0323466219b 100644 --- a/browser/ui/android/logo/java/src/org/chromium/chrome/browser/logo/BraveLogoMediator.java +++ b/browser/ui/android/logo/java/src/org/chromium/chrome/browser/logo/BraveLogoMediator.java @@ -5,7 +5,6 @@ package org.chromium.chrome.browser.logo; -import android.content.Context; import android.graphics.drawable.Drawable; import org.chromium.base.Callback; @@ -21,20 +20,16 @@ public class BraveLogoMediator extends LogoMediator { private boolean mShouldShowLogo; public BraveLogoMediator( - Context context, Callback logoClickedCallback, PropertyModel logoModel, Callback onLogoAvailableCallback, @Nullable VisibilityObserver visibilityObserver, - CachedTintedBitmap defaultGoogleLogo, @Nullable Drawable defaultGoogleLogoDrawable) { super( - context, logoClickedCallback, logoModel, onLogoAvailableCallback, visibilityObserver, - defaultGoogleLogo, defaultGoogleLogoDrawable); }