mozilla
/
gecko-dev
зеркало из https://github.com/mozilla/gecko-dev.git
1
0
Форкнуть 0
Код Задачи Пакеты Проекты Релизы Вики Активность

Backed out changeset 163d24542c19 (bug 1853216) for causing reftest failures on 1853216-1.html.

This commit is contained in:
Iulian Moraru 2023-09-16 05:14:15 +03:00
Родитель 80c9122018
Коммит 01b45bcc95
8 изменённых файлов: 0 добавлений и 32 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

7
gfx/tests/reftest/1853216-1-ref.html
Просмотреть файл

@ -1,7 +0,0 @@
<!DOCTYPE html>
<meta charset=utf-8>
<div style="font-size: 50px; color: transparent; text-shadow: 0px 0px 0 rgba(0,0,0,0.2)">
&#x1f550;
&#x1f550;
&#x1f550;
</div>

9
gfx/tests/reftest/1853216-1.html
Просмотреть файл

@ -1,9 +0,0 @@
<!DOCTYPE html>
<meta charset=utf-8>
<div style="font-size: 50px; color: transparent; text-shadow: 0px 0px 0 rgba(0,0,0,0.2)">
<!-- all 3 of these codepoints are clock faces, just with different hand positions,
so we expect their shadows to be identical -->
&#x1f550;
&#x1f552;
&#x1f558;
</div>

2
gfx/tests/reftest/reftest.list
Просмотреть файл

@ -43,5 +43,3 @@ pref(layout.css.backdrop-filter.enabled,true) == 1765862.html 1765862-ref.html
skip-if(!cocoaWidget) != 1806140.html 1806140-notref.html
fuzzy(0-1,0-240) == 1812341.html 1812341-ref.html
random-if(gtkWidget) fuzzy-if(Android,0-125,0-106) == 1845828-1.html 1845828-1-ref.html # Result on Linux depends on font configuration/hinting/etc, affecting whether subpixel positioning is used
fuzzy(0-1,0-2) == 1853216-1.html 1853216-1-ref.html

6
gfx/thebes/gfxFont.cpp
Просмотреть файл

@ -2313,7 +2313,6 @@ void gfxFont::Draw(const gfxTextRun* aTextRun, uint32_t aStart, uint32_t aEnd,
fontParams.obliqueSkew = SkewForSyntheticOblique();
fontParams.haveSVGGlyphs = GetFontEntry()->TryGetSVGData(this);
fontParams.haveColorGlyphs = GetFontEntry()->TryGetColorGlyphs();
fontParams.hasTextShadow = aRunParams.hasTextShadow;
fontParams.contextPaint = aRunParams.runContextPaint;
if (fontParams.haveColorGlyphs && !UseNativeColrFontSupport()) {
@ -2650,11 +2649,6 @@ bool gfxFont::RenderColorGlyph(DrawTarget* aDrawTarget, gfxContext* aContext,
layout::TextDrawTarget* aTextDrawer,
const FontDrawParams& aFontParams,
const Point& aPoint, uint32_t aGlyphId) {
if (aTextDrawer && aFontParams.hasTextShadow) {
aTextDrawer->FoundUnsupportedFeature();
return true;
}
if (const auto* paintGraph =
COLRFonts::GetGlyphPaintGraph(GetFontEntry()->GetCOLR(), aGlyphId)) {
const auto* hbShaper = GetHarfBuzzShaper();

2
gfx/thebes/gfxFont.h
Просмотреть файл

@ -2320,7 +2320,6 @@ struct MOZ_STACK_CLASS TextRunDrawParams {
bool isRTL = false;
bool paintSVGGlyphs = true;
bool allowGDI = true;
bool hasTextShadow = false;
// MRU cache of color-font palettes being used by fonts in the run. We cache
// these in the TextRunDrawParams so that we can avoid re-creating a new
@ -2365,7 +2364,6 @@ struct MOZ_STACK_CLASS FontDrawParams {
bool isVerticalFont;
bool haveSVGGlyphs;
bool haveColorGlyphs;
bool hasTextShadow; // whether we're rendering with a text-shadow
};
struct MOZ_STACK_CLASS EmphasisMarkDrawParams {

1
gfx/thebes/gfxTextRun.cpp
Просмотреть файл

@ -624,7 +624,6 @@ void gfxTextRun::Draw(const Range aRange, const gfx::Point aPt,
params.textStrokePattern = aParams.textStrokePattern;
params.drawOpts = aParams.drawOpts;
params.drawMode = aParams.drawMode;
params.hasTextShadow = aParams.hasTextShadow;
params.callbacks = aParams.callbacks;
params.runContextPaint = aParams.contextPaint;
params.paintSVGGlyphs =

1
gfx/thebes/gfxTextRun.h
Просмотреть файл

@ -264,7 +264,6 @@ class gfxTextRun : public gfxShapedText {
mozilla::SVGContextPaint* contextPaint = nullptr;
gfxTextRunDrawCallbacks* callbacks = nullptr;
bool allowGDI = true;
bool hasTextShadow = false;
explicit DrawParams(gfxContext* aContext) : context(aContext) {}
};

4
layout/generic/nsTextFrame.cpp
Просмотреть файл

@ -223,7 +223,6 @@ struct nsTextFrame::DrawTextRunParams {
gfx::FontPaletteValueSet* paletteValueSet = nullptr;
float textStrokeWidth = 0.0f;
bool drawSoftHyphen = false;
bool hasTextShadow = false;
explicit DrawTextRunParams(gfxContext* aContext) : context(aContext) {}
};
@ -6071,7 +6070,6 @@ bool nsTextFrame::PaintTextWithSelectionColors(
params.glyphRange = aParams.glyphRange;
params.fontPalette = StyleFont()->GetFontPaletteAtom();
params.paletteValueSet = PresContext()->GetFontPaletteValueSet();
params.hasTextShadow = !StyleText()->mTextShadow.IsEmpty();
PaintShadowParams shadowParams(aParams);
shadowParams.provider = aParams.provider;
@ -6658,7 +6656,6 @@ void nsTextFrame::PaintText(const PaintTextParams& aParams,
params.glyphRange = range;
params.fontPalette = StyleFont()->GetFontPaletteAtom();
params.paletteValueSet = PresContext()->GetFontPaletteValueSet();
params.hasTextShadow = !StyleText()->mTextShadow.IsEmpty();
DrawText(range, textBaselinePt, params);
}
@ -6675,7 +6672,6 @@ static void DrawTextRun(const gfxTextRun* aTextRun,
params.fontPalette = aParams.fontPalette;
params.paletteValueSet = aParams.paletteValueSet;
params.callbacks = aParams.callbacks;
params.hasTextShadow = aParams.hasTextShadow;
if (aParams.callbacks) {
aParams.callbacks->NotifyBeforeText(aParams.textColor);
params.drawMode = DrawMode::GLYPH_PATH;