morgan/helium
1
0
Fork 0
mirror of https://github.com/morgan9e/helium synced 2026-04-14 00:14:20 +09:00
Code Issues Packages Projects Releases Wiki Activity

patches: refresh for chromium 143.0.7499.109

Browse Source 
removed upstream-fixes/fix-macos-fullscreen-ui-breakage.patch because it's been merged into M143 upstream :D
This commit is contained in:
wukko
2025-12-12 14:35:35 +06:00
parent 7c51e1f0f6
commit 4cfbfe2710
10 changed files with 16 additions and 147 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
patches/helium/core/exclude-irrelevant-flags.patch
View File

@@ -1,6 +1,6 @@
--- a/chrome/browser/about_flags.cc
+++ b/chrome/browser/about_flags.cc
@@ -13643,11 +13643,328 @@ const FeatureEntry kFeatureEntries[] = {
@@ -13652,11 +13652,328 @@ const FeatureEntry kFeatureEntries[] = {
// AboutFlagsHistogramTest unit test to verify this process).
};
@@ -330,7 +330,7 @@
FlagsStateSingleton(const FlagsStateSingleton&) = delete;
FlagsStateSingleton& operator=(const FlagsStateSingleton&) = delete;
~FlagsStateSingleton() override = default;
@@ -13666,7 +13983,7 @@ class FlagsStateSingleton : public flags
@@ -13675,7 +13992,7 @@ class FlagsStateSingleton : public flags
void RestoreDefaultState() {
flags_state_ =
@@ -339,7 +339,7 @@
}
private:
@@ -13990,7 +14307,7 @@ base::span<const FeatureEntry> GetFeatur
@@ -13999,7 +14316,7 @@ base::span<const FeatureEntry> GetFeatur
!entries_for_testing->empty()) {
return *entries_for_testing;
}

2
patches/helium/core/flags-setup.patch
View File

@@ -15,7 +15,7 @@
#endif // USE_AURA
#if defined(USE_AURA)
@@ -5200,11 +5200,13 @@ const FeatureEntry::FeatureVariation kPr
@@ -5207,11 +5207,13 @@ const FeatureEntry::FeatureVariation kPr
// calculate and verify checksum.
//
// When adding a new choice, add it to the end of the list.

2
patches/helium/core/prefer-https-by-default.patch
View File

@@ -1,6 +1,6 @@
--- a/chrome/common/chrome_features.cc
+++ b/chrome/common/chrome_features.cc
@@ -1181,7 +1181,7 @@ BASE_FEATURE(kHttpsFirstBalancedMode, ba
@@ -1185,7 +1185,7 @@ BASE_FEATURE(kHttpsFirstBalancedMode, ba
// Automatically enables HTTPS-First Mode in a balanced configuration when
// possible.
BASE_FEATURE(kHttpsFirstBalancedModeAutoEnable,

2
patches/helium/core/ublock-reconfigure-defaults.patch
View File

@@ -21,7 +21,7 @@
location <= mojom::ManifestLocation::kMaxValue;
--- a/chrome/browser/chrome_content_browser_client.cc
+++ b/chrome/browser/chrome_content_browser_client.cc
@@ -6194,7 +6194,8 @@ void AddChromeSchemeFactories(
@@ -6212,7 +6212,8 @@ void AddChromeSchemeFactories(
std::vector<std::string> allowed_webui_hosts;
// Support for chrome:// scheme if appropriate.
if ((extension->is_extension() || extension->is_platform_app()) &&

2
patches/helium/ui/bangs-ui.patch
View File

@@ -58,7 +58,7 @@
#include "components/search_engines/template_url_service.h"
#include "components/search_engines/template_url_starter_pack_data.h"
#include "components/strings/grit/components_strings.h"
@@ -1944,7 +1945,21 @@ void AutocompleteController::UpdateKeywo
@@ -1939,7 +1940,21 @@ void AutocompleteController::UpdateKeywo
// - For contextual search matches, the description indicates the
// alternative UX because they're opened in the side panel.
i->description = template_url->AdjustedShortNameForLocaleDirection();

6
patches/helium/ui/clean-new-tab-page.patch
View File

@@ -102,8 +102,8 @@
${this.lazyRender_ ? html`
${this.shortcutsEnabled_ ? html`
<cr-most-visited id="mostVisited" .theme="${this.theme_?.mostVisited || null}"
@@ -93,54 +6,7 @@ import {nothing} from '//resources/lit/v
?expandable-tiles-enabled="${this.ntpNextFeaturesEnabled_}">
@@ -94,54 +7,7 @@ import {nothing} from '//resources/lit/v
max-tiles-before-show-more="${this.maxTilesBeforeShowMore_}">
</cr-most-visited>
` : ''}
- ${this.middleSlotPromoEnabled_ ? html`
@@ -157,7 +157,7 @@
<ntp-customize-buttons id="customizeButtons"
?modules-shown-to-user="${this.modulesShownToUser}"
?show-background-image="${this.showBackgroundImage_}"
@@ -162,23 +28,3 @@ import {nothing} from '//resources/lit/v
@@ -163,23 +29,3 @@ import {nothing} from '//resources/lit/v
` : ''}
<div id="contentBottomSpacer"></div>
</div>

2
patches/helium/ui/experiments/compact-action-toolbar.patch
View File

@@ -107,7 +107,7 @@
// Reparent PWA views that were moved for immersive mode.
if (web_app_frame_toolbar_) {
@@ -5196,6 +5202,12 @@ void BrowserView::AddedToWidget() {
@@ -5197,6 +5203,12 @@ void BrowserView::AddedToWidget() {
toolbar_->Init();

10
patches/helium/ui/restyle-ntp-tiles.patch
View File

@@ -1,6 +1,6 @@
--- a/ui/webui/resources/cr_components/most_visited/most_visited.ts
+++ b/ui/webui/resources/cr_components/most_visited/most_visited.ts
@@ -719,7 +719,7 @@ export class MostVisitedElement extends
@@ -717,7 +717,7 @@ export class MostVisitedElement extends
protected getFaviconUrl_(url: Url): string {
const faviconUrl = new URL('chrome://favicon2/');
@@ -38,7 +38,7 @@
display: inline-block;
height: 100%;
outline: none;
@@ -184,7 +184,7 @@
@@ -187,7 +187,7 @@
.tile-icon {
align-items: center;
background-color: var(--tile-background-color);
@@ -47,7 +47,7 @@
display: flex;
flex-shrink: 0;
height: var(--icon-size);
@@ -194,8 +194,8 @@
@@ -197,8 +197,8 @@
}
.tile-icon img {
@@ -58,7 +58,7 @@
}
.managed-tile-icon {
@@ -218,7 +218,6 @@ cr-policy-indicator {
@@ -221,7 +221,6 @@ cr-policy-indicator {
.tile-title {
align-items: center;
@@ -66,7 +66,7 @@
color: var(--most-visited-text-color);
display: flex;
height: var(--title-height);
@@ -263,7 +262,7 @@ cr-icon-button {
@@ -266,7 +265,7 @@ cr-icon-button {
--cr-icon-button-fill-color: var(--icon-button-color);
--cr-icon-button-size: 28px;
--cr-icon-button-transition: none;

1
patches/series
View File

@@ -2,7 +2,6 @@ upstream-fixes/missing-dependencies.patch
upstream-fixes/fix-crash-without-enterprise-cloud-content-analysis.patch
upstream-fixes/fix-python-codecs-deprecation.patch
upstream-fixes/fix-macos-widget-rect.patch
upstream-fixes/fix-macos-fullscreen-ui-breakage.patch
inox-patchset/fix-building-without-safebrowsing.patch
inox-patchset/disable-autofill-download-manager.patch

130
patches/upstream-fixes/fix-macos-fullscreen-ui-breakage.patch
View File

@@ -1,130 +0,0 @@
From 1e69bde8508191f45357b8c027bcd63a58d9ea8d Mon Sep 17 00:00:00 2001
From: Dana Fried <dfried@chromium.org>
Date: Fri, 31 Oct 2025 11:49:22 -0700
Subject: [PATCH] [Browser Frame] Abandon reading Mac caption button positions
These were not reliable during fullscreen transitions in a way that was
not possible to accurately filter out. This now uses the old fallback
hard-coded constants that were being used in all PWA windows anyway.
Include-Ci-Only-Tests: chromium.mac:mac15-x64-rel-tests|browser_tests
Bug: 453717426
Change-Id: Id102a321834c3dac0f56edcacda933c88db0b82a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7107360
Reviewed-by: Darryl James <dljames@chromium.org>
Commit-Queue: Dana Fried <dfried@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1538756}
---
--- a/chrome/browser/ui/views/frame/browser_frame_view_mac.h
+++ b/chrome/browser/ui/views/frame/browser_frame_view_mac.h
@@ -90,9 +90,6 @@ class BrowserFrameViewMac : public Brows
FRIEND_TEST_ALL_PREFIXES(BrowserFrameViewMacTest,
GetCaptionButtonPlaceholderBounds);
- // Fetches the caption button bounds from the buttons themselves.
- BoundsAndMargins GetCaptionButtonBoundsNative() const;
-
// Creates an inset from the caption button size which controls for which edge
// the captions buttons exists on. Used to position elements like the tabstrip
// that are adjacent to the caption buttons.
--- a/chrome/browser/ui/views/frame/browser_frame_view_mac.mm
+++ b/chrome/browser/ui/views/frame/browser_frame_view_mac.mm
@@ -458,96 +458,9 @@ void BrowserFrameViewMac::PaintChildren(
}
BrowserFrameViewMac::BoundsAndMargins
-BrowserFrameViewMac::GetCaptionButtonBoundsNative() const {
- BoundsAndMargins result;
-
- // Verify that this is not an out-of-process window.
- const auto native_window = GetWidget()->GetNativeWindow();
- if (auto* const host =
- views::NativeWidgetMacNSWindowHost::GetFromNativeWindow(
- native_window);
- host && host->application_host()) {
- return result;
- }
-
- // Verify that there is a valid NSWindow.
- NSWindow* ns_window = native_window.GetNativeNSWindow();
- if (!ns_window) {
- return result;
- }
-
- // Build a list of caption button bounds.
- std::vector<gfx::RectF> button_rects;
-
- // Also track some other data that will be useful for calculating the visual
- // margins of the buttons against the window border.
- float min_x = width();
- float max_x = 0;
- float min_y = height();
-
- // Chrome coordinates are reversed in RTL but not in the Mac API, so we might
- // have to flip them.
- const bool is_rtl = base::i18n::IsRTL();
-
- // Build the list. If any of the buttons are not present or are zero size,
- // abort (this will fall back to previous hard-coded values).
- for (NSButton* button :
- {[ns_window standardWindowButton:NSWindowCloseButton],
- [ns_window standardWindowButton:NSWindowMiniaturizeButton],
- [ns_window standardWindowButton:NSWindowZoomButton]}) {
- if (!button) {
- return result;
- }
- NSRect ns_rect = [button convertRect:[button bounds] toView:nil];
-
- // When converting from Mac to Chrome coordinates:
- // - Y axis is inverted (Mac coordinates start at bottom-left).
- // - X axis is inverted in RTL mode only (Mac coordinates are invariant
- // while Chrome reverses them).
- button_rects.emplace_back(
- is_rtl ? width() - (ns_rect.origin.x + ns_rect.size.width)
- : ns_rect.origin.x,
- height() - (ns_rect.origin.y + ns_rect.size.height), ns_rect.size.width,
- ns_rect.size.height);
- const auto& rect = button_rects.back();
- if (rect.IsEmpty()) {
- return result;
- }
- min_x = std::min(min_x, rect.x());
- max_x = std::max(max_x, rect.right());
- min_y = std::min(min_y, rect.y());
- }
-
- // Calculate the margins that the buttons are using.
- const float block_margin = min_y;
- const float inline_margin =
- CaptionButtonsOnLeadingEdge() ? min_x : width() - max_x;
-
- // Accumulate the button bounds.
- for (const auto& rect : button_rects) {
- result.bounds.Union(rect);
- }
-
- // Apply the margins on the exterior of the region, so that the padding around
- // the buttons appears visually symmetrical.
- result.margins = gfx::OutsetsF::VH(block_margin, inline_margin);
-
- return result;
-}
-
-BrowserFrameViewMac::BoundsAndMargins
BrowserFrameViewMac::GetCaptionButtonBounds() const {
- BoundsAndMargins result = GetCaptionButtonBoundsNative();
- if (!result.bounds.IsEmpty()) {
- return result;
- }
+ BoundsAndMargins result;
- // If that doesn't work, fall back to some hard-coded constants. These will
- // apply for app windows, since the app is out of process and the caption
- // buttons can't be retrieved. Note that the app window actually positions its
- // buttons slightly differently from a standard browser window (it's unclear
- // why).
- //
// These are empirically determined; feel free to change them if they're
// not precise.
if (@available(macOS 26, *)) {