Bug 117013

Summary:

If there is no Japanese character in a same line, a Japanese character inside ruby is overflowed from background of ruby.

Product:

WebKit

Reporter:

Yuki Sekiguchi <yuki.sekiguchi>

Component:

Layout and Rendering

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ahmad.saleem792, bdakin, buildbot, commit-queue, enrica, esprehn+autocc, glenn, hyatt, koivisto, rniwa, webkit-bug-importer, zalan

Priority:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
reproduced html content	none
Patch	beidson: review-, buildbot: commit-queue-
Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2	none
Reference Rendering across browsers	none

Yuki Sekiguchi

Reported 2013-05-29 19:52:39 PDT

Created attachment 203303 [details] reproduced html content In attached content, the Japanese character "国" is overflowed from background of ruby. If we added a Japanese character in a same line, it works correct.

Attachments
reproduced html content (555 bytes, text/html) 2013-05-29 19:52 PDT, Yuki Sekiguchi	no flags	Details
Patch (6.98 KB, patch) 2013-05-29 20:19 PDT, Yuki Sekiguchi	beidson: review- buildbot: commit-queue-	Details Formatted Diff Diff
Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2 (531.94 KB, application/zip) 2013-05-30 09:19 PDT, Build Bot	no flags	Details
Reference Rendering across browsers (262.21 KB, image/png) 2024-03-10 18:50 PDT, Ahmad Saleem	no flags	Details
View All Add attachment proposed patch, testcase, etc.

Yuki Sekiguchi

Comment 1 2013-05-29 20:19:43 PDT

Created attachment 203304 [details] Patch

Build Bot

Comment 2 2013-05-30 09:19:33 PDT

Comment on attachment 203304 [details] Patch Attachment 203304 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/697253 New failing tests: fast/dom/location-new-window-no-crash.html

Build Bot

Comment 3 2013-05-30 09:19:36 PDT

Created attachment 203364 [details] Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-12 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.3

Brady Eidson

Comment 4 2016-05-24 21:38:37 PDT

Comment on attachment 203304 [details] Patch r- to clear stale patches from the review queue

Ahmad Saleem

Comment 5 2024-03-10 18:50:56 PDT

Created attachment 470279 [details] Reference Rendering across browsers Took test from attached patch: Test Case - https://jsfiddle.net/cp1j634L/show Test Case Expectation - https://jsfiddle.net/xmj915ed/show ^ matches rendering-wise in Safari 17.4. (I have removed -webkit-). Now rendering of Safari don't match with Chrome Canary 124 and also with Firefox Nightly 124 and all browser differ.

alan

Comment 6 2024-03-10 19:00:12 PDT

Thank you, Ahmad. This looks to have progressed with IFC.

Ahmad Saleem

Comment 7 2024-03-10 19:16:07 PDT

(In reply to zalan from comment #6) > Thank you, Ahmad. This looks to have progressed with IFC. Thanks Alan - should we add this test from coverage perspective?

alan

Comment 8 2024-03-11 07:01:35 PDT

(In reply to Ahmad Saleem from comment #7) > (In reply to zalan from comment #6) > > Thank you, Ahmad. This looks to have progressed with IFC. > > Thanks Alan - should we add this test from coverage perspective? I think we should.

Ahmad Saleem

Comment 9 2024-03-11 15:51:29 PDT

Reopened to do PR for importing test - https://github.com/WebKit/WebKit/pull/25726

EWS

Comment 10 2024-03-11 17:35:51 PDT

Committed 275942@main (f5aca8555eac): <https://commits.webkit.org/275942@main> Reviewed commits have been landed. Closing PR #25726 and removing active labels.

Radar WebKit Bug Importer

Comment 11 2024-03-11 17:36:14 PDT

<rdar://problem/124417999>

Note You need to log in before you can comment on or make changes to this bug.