Certain uses of iframe elements in WebKit clients can cause undesirable printing behavior. WebKit attempts to keep all contents of an iframe together on the same page, but in some cases that does not produce good looking print output. This bug provides a new setting that allows the default printing behavior to be overridden.
<rdar://problem/14912763>
Created attachment 246537 [details] Patch
Created attachment 246541 [details] Patch
Comment on attachment 246541 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=246541&action=review > Source/WebCore/rendering/RenderBlockFlow.cpp:1617 > +static const AtomicString& messageContainerName() > +{ > + static NeverDestroyed<AtomicString> messageContainerNameString("messageContentContainer", AtomicString::ConstructFromLiteral); > + return messageContainerNameString; > +} It's bad to use AtomicString inside NeverDestroyed since they are different for different threads. I'd just use a literal in the function below instead.
Created attachment 246545 [details] Patch
(In reply to comment #4) > Comment on attachment 246541 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=246541&action=review > > > Source/WebCore/rendering/RenderBlockFlow.cpp:1617 > > +static const AtomicString& messageContainerName() > > +{ > > + static NeverDestroyed<AtomicString> messageContainerNameString("messageContentContainer", AtomicString::ConstructFromLiteral); > > + return messageContainerNameString; > > +} > > It's bad to use AtomicString inside NeverDestroyed since they are different > for different threads. I'd just use a literal in the function below instead. OK!
Committed r180076: <http://trac.webkit.org/changeset/180076>