Bug 38222 – "word-wrap: break-word" should not override "white-space: pre"

• Status: RESOLVED
• Resolution: CONFIGURATION CHANGED
• Component: Text

[Xianzhu Wang]

Other browsers tested:
Add OK or FAIL after other browsers where you have tested this issue:
     Chrome 5: FAIL
  Firefox 3.x: OK
         IE 8: OK

What steps will reproduce the problem?
1. Open the attached test case.

What is the expected result?
Line won't wrap.

What happens instead?
Line wraps

Please provide any additional information below. Attach a screenshot if
possible.

According to the CSS3 specification about word-wrap 
(http://www.w3.org/TR/css3-text/#word-wrap), the value of word-wrap only 
controls if the UA may break unbreakable words. It shouldn't force line-
wrapping, and shouldn't override "white-space: pre". Though the title says 
"force", the content doesn't say it.

"normal
  Lines may break only at allowed break points.
break-word
  An unbreakable "word" may be broken at an arbitrary point if there are no 
otherwise-acceptable break points in the line..."

http://crbug.com/42680

[Xianzhu Wang]

Created attachment 54586 [details]
testcase

[Chris Rebert]

Current analogous Chrome bug seems to be
https://code.google.com/p/chromium/issues/detail?id=373502

[Ahmad Saleem]

Changing this:

https://github.com/WebKit/WebKit/blob/4032d1d4de923b3fa6b16e5174c4aa88041f0107/Source/WebCore/rendering/style/RenderStyle.h#L2334

Line:

```

    return wordBreak() == WordBreak::BreakWord || overflowWrap() == OverflowWrap::BreakWord || overflowWrap() == OverflowWrap::Anywhere;

```

to:

```

    return (wordBreak() == WordBreak::BreakWord || overflowWrap() == OverflowWrap::BreakWord || overflowWrap() == OverflowWrap::Anywhere) && whiteSpace() != WhiteSpace::Pre && whiteSpace() != WhiteSpace::NoWrap;

_____

Just wanted to update.

[Ahmad Saleem]

Although it seems - Safari Technology Preview 156 passes all WPT Tests cases:

https://wpt.fyi/results/css/css-text?label=master&label=experimental&aligned&view=subtest&q=break-word

and even attached test case seems to render same in Safari 16.1 as Chrome Canary 109 and Firefox Nightly 108.

Do we need to do any changes? Thanks!

[zalan]

IFC surely gets this right and in legacy line layout this must have been addressed in BreakingContext (and not in the RenderStyle helper function).