зеркало из https://github.com/mozilla/gecko-dev.git
Bug 1703220 - Remove unnecessary caching in windows lookandfeel. r=cmartin
After bug 1697607 and the subsequent pref removal this code only runs in the parent process. After bug 1699088 the caller takes care of caching. So there should be no need to cache stuff ourselves. Differential Revision: https://phabricator.services.mozilla.com/D110912
This commit is contained in:
Emilio Cobos Álvarez
Родитель
830a1bba6e
Коммит
c817989481
|
|
@ -91,10 +91,6 @@ static nsresult SystemWantsDarkTheme(int32_t& darkThemeEnabled) {
|
|||
|
||||
nsLookAndFeel::nsLookAndFeel()
|
||||
: nsXPLookAndFeel(),
|
||||
mUseAccessibilityTheme(0),
|
||||
mUseDefaultTheme(0),
|
||||
mNativeThemeId(eWindowsTheme_Generic),
|
||||
mCaretBlinkTime(-1),
|
||||
mHasColorMenuHoverText(false),
|
||||
mHasColorAccent(false),
|
||||
mHasColorAccentText(false),
|
||||
|
|
@ -112,16 +108,7 @@ void nsLookAndFeel::NativeInit() { EnsureInit(); }
|
|||
/* virtual */
|
||||
void nsLookAndFeel::RefreshImpl() {
|
||||
nsXPLookAndFeel::RefreshImpl();
|
||||
|
||||
for (auto e = mSystemFontCache.begin(), end = mSystemFontCache.end();
|
||||
e != end; ++e) {
|
||||
e->mCacheValid = false;
|
||||
}
|
||||
mCaretBlinkTime = -1;
|
||||
|
||||
mCacheValidBits.reset();
|
||||
|
||||
mInitialized = false;
|
||||
mInitialized = false; // Fetch system colors next time they're used.
|
||||
}
|
||||
|
||||
nsresult nsLookAndFeel::NativeGetColor(ColorID aID, ColorScheme,
|
||||
|
|
@ -380,12 +367,7 @@ nsresult nsLookAndFeel::NativeGetInt(IntID aID, int32_t& aResult) {
|
|||
aResult = 3;
|
||||
break;
|
||||
case IntID::CaretBlinkTime:
|
||||
// IntID::CaretBlinkTime is often called by updating editable text
|
||||
// that has focus. So it should be cached to improve performance.
|
||||
if (mCaretBlinkTime < 0) {
|
||||
mCaretBlinkTime = static_cast<int32_t>(::GetCaretBlinkTime());
|
||||
}
|
||||
aResult = mCaretBlinkTime;
|
||||
aResult = static_cast<int32_t>(::GetCaretBlinkTime());
|
||||
break;
|
||||
case IntID::CaretWidth:
|
||||
aResult = 1;
|
||||
|
|
@ -424,16 +406,7 @@ nsresult nsLookAndFeel::NativeGetInt(IntID aID, int32_t& aResult) {
|
|||
// High contrast is a misnomer under Win32 -- any theme can be used with
|
||||
// it, e.g. normal contrast with large fonts, low contrast, etc. The high
|
||||
// contrast flag really means -- use this theme and don't override it.
|
||||
if (XRE_IsContentProcess()) {
|
||||
// If we're running in the content process, then the parent should
|
||||
// have sent us the accessibility state when nsLookAndFeel
|
||||
// initialized, and stashed it in the mUseAccessibilityTheme cache.
|
||||
aResult = mUseAccessibilityTheme;
|
||||
} else {
|
||||
// Otherwise, we can ask the OS to see if we're using High Contrast
|
||||
// mode.
|
||||
aResult = nsUXThemeData::IsHighContrastOn();
|
||||
}
|
||||
break;
|
||||
case IntID::ScrollArrowStyle:
|
||||
aResult = eScrollArrowStyle_Single;
|
||||
|
|
@ -460,20 +433,11 @@ nsresult nsLookAndFeel::NativeGetInt(IntID aID, int32_t& aResult) {
|
|||
aResult = !nsUXThemeData::IsAppThemed();
|
||||
break;
|
||||
case IntID::WindowsDefaultTheme:
|
||||
if (XRE_IsContentProcess()) {
|
||||
aResult = mUseDefaultTheme;
|
||||
} else {
|
||||
aResult = nsUXThemeData::IsDefaultWindowTheme();
|
||||
}
|
||||
break;
|
||||
case IntID::WindowsThemeIdentifier:
|
||||
if (XRE_IsContentProcess()) {
|
||||
aResult = mNativeThemeId;
|
||||
} else {
|
||||
aResult = nsUXThemeData::GetNativeThemeId();
|
||||
}
|
||||
break;
|
||||
|
||||
case IntID::OperatingSystemVersionIdentifier: {
|
||||
aResult = int32_t(GetOperatingSystemVersion());
|
||||
break;
|
||||
|
|
@ -605,25 +569,13 @@ nsresult nsLookAndFeel::NativeGetInt(IntID aID, int32_t& aResult) {
|
|||
break;
|
||||
}
|
||||
case IntID::PrimaryPointerCapabilities: {
|
||||
if (!mCacheValidBits[PrimaryPointerCapabilitiesKind] &&
|
||||
!XRE_IsContentProcess()) {
|
||||
mPrimaryPointerCapabilities = static_cast<int32_t>(
|
||||
aResult = static_cast<int32_t>(
|
||||
widget::WinUtils::GetPrimaryPointerCapabilities());
|
||||
mCacheValidBits[PrimaryPointerCapabilitiesKind] = true;
|
||||
}
|
||||
|
||||
aResult = mPrimaryPointerCapabilities;
|
||||
break;
|
||||
}
|
||||
case IntID::AllPointerCapabilities: {
|
||||
if (!mCacheValidBits[AllPointerCapabilitiesKind] &&
|
||||
!XRE_IsContentProcess()) {
|
||||
mAllPointerCapabilities =
|
||||
aResult =
|
||||
static_cast<int32_t>(widget::WinUtils::GetAllPointerCapabilities());
|
||||
mCacheValidBits[AllPointerCapabilitiesKind] = true;
|
||||
}
|
||||
|
||||
aResult = mAllPointerCapabilities;
|
||||
break;
|
||||
}
|
||||
default:
|
||||
|
|
@ -714,10 +666,6 @@ LookAndFeelFont nsLookAndFeel::GetLookAndFeelFontInternal(
|
|||
}
|
||||
|
||||
LookAndFeelFont nsLookAndFeel::GetLookAndFeelFont(LookAndFeel::FontID anID) {
|
||||
if (XRE_IsContentProcess()) {
|
||||
return mFontCache[anID];
|
||||
}
|
||||
|
||||
LookAndFeelFont result{};
|
||||
|
||||
result.haveFont() = false;
|
||||
|
|
@ -771,54 +719,10 @@ LookAndFeelFont nsLookAndFeel::GetLookAndFeelFont(LookAndFeel::FontID anID) {
|
|||
return result;
|
||||
}
|
||||
|
||||
bool nsLookAndFeel::GetSysFont(LookAndFeel::FontID anID, nsString& aFontName,
|
||||
bool nsLookAndFeel::NativeGetFont(LookAndFeel::FontID anID, nsString& aFontName,
|
||||
gfxFontStyle& aFontStyle) {
|
||||
LookAndFeelFont font = GetLookAndFeelFont(anID);
|
||||
|
||||
if (!font.haveFont()) {
|
||||
return false;
|
||||
}
|
||||
|
||||
aFontName = std::move(font.name());
|
||||
|
||||
aFontStyle.size = font.size();
|
||||
|
||||
// FIXME: What about oblique?
|
||||
aFontStyle.style =
|
||||
font.italic() ? FontSlantStyle::Italic() : FontSlantStyle::Normal();
|
||||
|
||||
aFontStyle.weight = FontWeight(font.weight());
|
||||
|
||||
// FIXME: Set aFontStyle->stretch correctly!
|
||||
aFontStyle.stretch = FontStretch::Normal();
|
||||
|
||||
aFontStyle.systemFont = true;
|
||||
|
||||
return true;
|
||||
}
|
||||
|
||||
bool nsLookAndFeel::NativeGetFont(FontID anID, nsString& aFontName,
|
||||
gfxFontStyle& aFontStyle) {
|
||||
CachedSystemFont& cacheSlot = mSystemFontCache[anID];
|
||||
|
||||
bool status;
|
||||
if (cacheSlot.mCacheValid) {
|
||||
status = cacheSlot.mHaveFont;
|
||||
if (status) {
|
||||
aFontName = cacheSlot.mFontName;
|
||||
aFontStyle = cacheSlot.mFontStyle;
|
||||
}
|
||||
} else {
|
||||
status = GetSysFont(anID, aFontName, aFontStyle);
|
||||
|
||||
cacheSlot.mCacheValid = true;
|
||||
cacheSlot.mHaveFont = status;
|
||||
if (status) {
|
||||
cacheSlot.mFontName = aFontName;
|
||||
cacheSlot.mFontStyle = aFontStyle;
|
||||
}
|
||||
}
|
||||
return status;
|
||||
return LookAndFeelFontToStyle(font, aFontName, aFontStyle);
|
||||
}
|
||||
|
||||
/* virtual */
|
||||
|
|
|
|||
|
|
@ -60,12 +60,6 @@ class nsLookAndFeel final : public nsXPLookAndFeel {
|
|||
char16_t GetPasswordCharacterImpl() override;
|
||||
|
||||
private:
|
||||
enum CachedValueKind {
|
||||
PrimaryPointerCapabilitiesKind,
|
||||
AllPointerCapabilitiesKind,
|
||||
CachedValueKindMax = AllPointerCapabilitiesKind,
|
||||
};
|
||||
|
||||
/**
|
||||
* Fetches the Windows accent color from the Windows settings if
|
||||
* the accent color is set to apply to the title bar, otherwise
|
||||
|
|
@ -89,25 +83,6 @@ class nsLookAndFeel final : public nsXPLookAndFeel {
|
|||
|
||||
LookAndFeelFont GetLookAndFeelFont(LookAndFeel::FontID anID);
|
||||
|
||||
bool GetSysFont(LookAndFeel::FontID anID, nsString& aFontName,
|
||||
gfxFontStyle& aFontStyle);
|
||||
|
||||
// Content process cached values that get shipped over from the browser
|
||||
// process.
|
||||
int32_t mUseAccessibilityTheme;
|
||||
int32_t mUseDefaultTheme; // is the current theme a known default?
|
||||
int32_t mNativeThemeId; // see LookAndFeel enum 'WindowsTheme'
|
||||
|
||||
// Information about whether pointers exist, and whether they are fine
|
||||
// (like a mouse) or coarse (like a VR peripheral), and whether they
|
||||
// support the concept of "hovering" over something
|
||||
//
|
||||
// See the CSS "@media pointer" query for more info
|
||||
int32_t mPrimaryPointerCapabilities;
|
||||
int32_t mAllPointerCapabilities;
|
||||
|
||||
int32_t mCaretBlinkTime;
|
||||
|
||||
// Cached colors and flags indicating success in their retrieval.
|
||||
nscolor mColorMenuHoverText;
|
||||
bool mHasColorMenuHoverText;
|
||||
|
|
@ -126,28 +101,7 @@ class nsLookAndFeel final : public nsXPLookAndFeel {
|
|||
|
||||
void EnsureInit();
|
||||
|
||||
struct CachedSystemFont {
|
||||
CachedSystemFont() : mCacheValid(false) {}
|
||||
|
||||
bool mCacheValid;
|
||||
bool mHaveFont;
|
||||
nsString mFontName;
|
||||
gfxFontStyle mFontStyle;
|
||||
};
|
||||
|
||||
mozilla::EnumeratedArray<FontID, FontID::End, CachedSystemFont>
|
||||
mSystemFontCache;
|
||||
|
||||
using FontCache =
|
||||
mozilla::EnumeratedArray<FontID, FontID::End, LookAndFeelFont>;
|
||||
FontCache mFontCache;
|
||||
|
||||
nsCOMPtr<nsIWindowsRegKey> mDwmKey;
|
||||
|
||||
// A bitmap of which cached values are currently valid (ignored in content
|
||||
// process, since all cached values in content may only be updated from
|
||||
// one valid value to another, otherwise layout will not function properly)
|
||||
std::bitset<CachedValueKindMax + 1> mCacheValidBits;
|
||||
};
|
||||
|
||||
#endif
|
||||
|
|
|
|||
Загрузка…
Ссылка в новой задаче