Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

REGRESSION (255903@main): iWork/Numbers/Pages Keynote attachments appear upside down #6882

Conversation

nmoucht
Copy link
Contributor

@nmoucht nmoucht commented Nov 28, 2022

f3dfd0c

REGRESSION (255903@main): iWork/Numbers/Pages Keynote attachments appear upside down
https://bugs.webkit.org/show_bug.cgi?id=248421
<rdar://102052562>

Unreviewed revert.

Revert 255903@main.

* LayoutTests/fast/attachment/mac/attachment-element-gpu-process-expected.html: Removed.
* LayoutTests/fast/attachment/mac/attachment-element-gpu-process.html: Removed.
* Source/WebCore/SourcesCocoa.txt:
* Source/WebCore/WebCore.xcodeproj/project.pbxproj:
* Source/WebCore/rendering/AttachmentLayout.h: Removed.
* Source/WebCore/rendering/AttachmentLayout.mm: Removed.
* Source/WebCore/rendering/RenderTheme.cpp:
(WebCore::RenderTheme::paint):
* Source/WebCore/rendering/RenderTheme.h:
(WebCore::RenderTheme::paintAttachmentText): Deleted.
* Source/WebCore/rendering/RenderThemeCocoa.h:
* Source/WebCore/rendering/RenderThemeCocoa.mm:
(WebCore::RenderThemeCocoa::attachmentBaseline const): Deleted.
(WebCore::RenderThemeCocoa::paintAttachmentText): Deleted.
* Source/WebCore/rendering/RenderThemeIOS.h:
* Source/WebCore/rendering/RenderThemeIOS.mm:
(WebCore::attachmentActionFont):
(WebCore::attachmentActionColor):
(WebCore::attachmentTitleFont):
(WebCore::shortCaptionPointSizeWithContentSizeCategory):
(WebCore::attachmentDynamicTypeScaleFactor):
(WebCore::attachmentTitleColor):
(WebCore::attachmentSubtitleFont):
(WebCore::attachmentSubtitleColor):
(WebCore::RenderAttachmentInfo::addLine):
(WebCore::RenderAttachmentInfo::buildWrappedLines):
(WebCore::RenderAttachmentInfo::buildSingleLine):
(WebCore::getAttachmentProgress):
(WebCore::RenderAttachmentInfo::RenderAttachmentInfo):
(WebCore::RenderThemeIOS::attachmentBaseline const):
(WebCore::paintAttachmentIcon):
(WebCore::paintAttachmentText):
(WebCore::paintAttachmentProgress):
(WebCore::attachmentBorderPath):
(WebCore::RenderThemeIOS::paintAttachment):
* Source/WebCore/rendering/RenderThemeMac.h:
* Source/WebCore/rendering/RenderThemeMac.mm:
(WebCore::titleTextColorForAttachment):
(WebCore::AttachmentLayout::addTitleLine):
(WebCore::AttachmentLayout::layOutTitle):
(WebCore::AttachmentLayout::layOutSubtitle):
(WebCore::AttachmentLayout::AttachmentLayout):
(WebCore::RenderThemeMac::attachmentBaseline const):
(WebCore::paintAttachmentIcon):
(WebCore::paintAttachmentTitle):
(WebCore::paintAttachmentSubtitle):
(WebCore::RenderThemeMac::paintAttachment):

Canonical link: https://commits.webkit.org/257095@main

1b3d366

Misc iOS, tvOS & watchOS macOS Linux Windows
βœ… πŸ§ͺ style   πŸ›  ios   πŸ›  mac   πŸ›  wpe   πŸ›  πŸ§ͺ win
βœ… πŸ§ͺ bindings βœ… πŸ›  ios-sim   πŸ›  mac-debug   πŸ›  gtk   πŸ›  wincairo
βœ… πŸ§ͺ webkitperl   πŸ§ͺ ios-wk2   πŸ›  mac-AS-debug   πŸ§ͺ gtk-wk2
  πŸ§ͺ api-ios   πŸ§ͺ api-mac   πŸ§ͺ api-gtk
  πŸ›  tv   πŸ§ͺ mac-wk1
  πŸ›  tv-sim   πŸ§ͺ mac-wk2
  πŸ›  watch   πŸ§ͺ mac-AS-debug-wk2
❌ πŸ›  πŸ§ͺ unsafe-merge βœ… πŸ›  watch-sim   πŸ§ͺ mac-wk2-stress

@nmoucht nmoucht self-assigned this Nov 28, 2022
@nmoucht nmoucht changed the title Need a short description (OOPS!). REGRESSION (255903@main): iWork/Numbers/Pages Keynote attachments appear upside down Nov 28, 2022
@nmoucht nmoucht force-pushed the eng/REGRESSION-255903main-iWorkNumbersPages-Keynote-attachments-appear-upside-down branch from ffd5810 to 43a5a21 Compare November 28, 2022 21:51
@nmoucht nmoucht added the New Bugs Unclassified bugs are placed in this component until the correct component can be determined. label Nov 28, 2022
@nmoucht nmoucht requested a review from smfr November 28, 2022 22:05
@nmoucht nmoucht force-pushed the eng/REGRESSION-255903main-iWorkNumbersPages-Keynote-attachments-appear-upside-down branch from 43a5a21 to 3879e0b Compare November 28, 2022 22:22
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Nov 28, 2022
@nmoucht nmoucht removed the merging-blocked Applied to prevent a change from being merged label Nov 28, 2022
@nmoucht nmoucht force-pushed the eng/REGRESSION-255903main-iWorkNumbersPages-Keynote-attachments-appear-upside-down branch from 3879e0b to 8154390 Compare November 28, 2022 23:39
@nmoucht nmoucht force-pushed the eng/REGRESSION-255903main-iWorkNumbersPages-Keynote-attachments-appear-upside-down branch from 8154390 to 1b3d366 Compare November 29, 2022 00:48
@nmoucht nmoucht added the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Nov 29, 2022
@webkit-early-warning-system webkit-early-warning-system force-pushed the eng/REGRESSION-255903main-iWorkNumbersPages-Keynote-attachments-appear-upside-down branch from 1b3d366 to 5bd5d88 Compare November 29, 2022 00:50
…ear upside down

https://bugs.webkit.org/show_bug.cgi?id=248421
<rdar://102052562>

Unreviewed revert.

Revert 255903@main.

* LayoutTests/fast/attachment/mac/attachment-element-gpu-process-expected.html: Removed.
* LayoutTests/fast/attachment/mac/attachment-element-gpu-process.html: Removed.
* Source/WebCore/SourcesCocoa.txt:
* Source/WebCore/WebCore.xcodeproj/project.pbxproj:
* Source/WebCore/rendering/AttachmentLayout.h: Removed.
* Source/WebCore/rendering/AttachmentLayout.mm: Removed.
* Source/WebCore/rendering/RenderTheme.cpp:
(WebCore::RenderTheme::paint):
* Source/WebCore/rendering/RenderTheme.h:
(WebCore::RenderTheme::paintAttachmentText): Deleted.
* Source/WebCore/rendering/RenderThemeCocoa.h:
* Source/WebCore/rendering/RenderThemeCocoa.mm:
(WebCore::RenderThemeCocoa::attachmentBaseline const): Deleted.
(WebCore::RenderThemeCocoa::paintAttachmentText): Deleted.
* Source/WebCore/rendering/RenderThemeIOS.h:
* Source/WebCore/rendering/RenderThemeIOS.mm:
(WebCore::attachmentActionFont):
(WebCore::attachmentActionColor):
(WebCore::attachmentTitleFont):
(WebCore::shortCaptionPointSizeWithContentSizeCategory):
(WebCore::attachmentDynamicTypeScaleFactor):
(WebCore::attachmentTitleColor):
(WebCore::attachmentSubtitleFont):
(WebCore::attachmentSubtitleColor):
(WebCore::RenderAttachmentInfo::addLine):
(WebCore::RenderAttachmentInfo::buildWrappedLines):
(WebCore::RenderAttachmentInfo::buildSingleLine):
(WebCore::getAttachmentProgress):
(WebCore::RenderAttachmentInfo::RenderAttachmentInfo):
(WebCore::RenderThemeIOS::attachmentBaseline const):
(WebCore::paintAttachmentIcon):
(WebCore::paintAttachmentText):
(WebCore::paintAttachmentProgress):
(WebCore::attachmentBorderPath):
(WebCore::RenderThemeIOS::paintAttachment):
* Source/WebCore/rendering/RenderThemeMac.h:
* Source/WebCore/rendering/RenderThemeMac.mm:
(WebCore::titleTextColorForAttachment):
(WebCore::AttachmentLayout::addTitleLine):
(WebCore::AttachmentLayout::layOutTitle):
(WebCore::AttachmentLayout::layOutSubtitle):
(WebCore::AttachmentLayout::AttachmentLayout):
(WebCore::RenderThemeMac::attachmentBaseline const):
(WebCore::paintAttachmentIcon):
(WebCore::paintAttachmentTitle):
(WebCore::paintAttachmentSubtitle):
(WebCore::RenderThemeMac::paintAttachment):

Canonical link: https://commits.webkit.org/257095@main
@webkit-early-warning-system webkit-early-warning-system force-pushed the eng/REGRESSION-255903main-iWorkNumbersPages-Keynote-attachments-appear-upside-down branch from 5bd5d88 to f3dfd0c Compare November 29, 2022 00:51
@webkit-commit-queue
Copy link
Collaborator

Committed 257095@main (f3dfd0c): https://commits.webkit.org/257095@main

Reviewed commits have been landed. Closing PR #6882 and removing active labels.

@webkit-early-warning-system webkit-early-warning-system merged commit f3dfd0c into WebKit:main Nov 29, 2022
@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Bugs Unclassified bugs are placed in this component until the correct component can be determined.
Projects
None yet
4 participants