Upstream iOS-specific changes to WebCore/rendering.
Created attachment 218443 [details] Patch
Attachment 218443 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj', u'Source/WebCore/rendering/InlineBox.cpp', u'Source/WebCore/rendering/InlineBox.h', u'Source/WebCore/rendering/InlineTextBox.cpp', u'Source/WebCore/rendering/RenderBlock.cpp', u'Source/WebCore/rendering/RenderBlock.h', u'Source/WebCore/rendering/RenderBox.cpp', u'Source/WebCore/rendering/RenderBox.h', u'Source/WebCore/rendering/RenderBoxModelObject.cpp', u'Source/WebCore/rendering/RenderButton.cpp', u'Source/WebCore/rendering/RenderElement.cpp', u'Source/WebCore/rendering/RenderEmbeddedObject.cpp', u'Source/WebCore/rendering/RenderFileUploadControl.cpp', u'Source/WebCore/rendering/RenderFrameSet.cpp', u'Source/WebCore/rendering/RenderIFrame.h', u'Source/WebCore/rendering/RenderImage.cpp', u'Source/WebCore/rendering/RenderImage.h', u'Source/WebCore/rendering/RenderInline.cpp', u'Source/WebCore/rendering/RenderInline.h', u'Source/WebCore/rendering/RenderLayer.cpp', u'Source/WebCore/rendering/RenderLayer.h', u'Source/WebCore/rendering/RenderLayerBacking.cpp', u'Source/WebCore/rendering/RenderLayerBacking.h', u'Source/WebCore/rendering/RenderLayerCompositor.cpp', u'Source/WebCore/rendering/RenderLayerCompositor.h', u'Source/WebCore/rendering/RenderLayerFilterInfo.h', u'Source/WebCore/rendering/RenderMenuList.cpp', u'Source/WebCore/rendering/RenderMenuList.h', u'Source/WebCore/rendering/RenderObject.cpp', u'Source/WebCore/rendering/RenderObject.h', u'Source/WebCore/rendering/RenderScrollbar.h', u'Source/WebCore/rendering/RenderSearchField.cpp', u'Source/WebCore/rendering/RenderText.cpp', u'Source/WebCore/rendering/RenderText.h', u'Source/WebCore/rendering/RenderTextControl.cpp', u'Source/WebCore/rendering/RenderTextControl.h', u'Source/WebCore/rendering/RenderTextControlMultiLine.cpp', u'Source/WebCore/rendering/RenderTextControlSingleLine.cpp', u'Source/WebCore/rendering/RenderTextLineBoxes.cpp', u'Source/WebCore/rendering/RenderTheme.cpp', u'Source/WebCore/rendering/RenderTheme.h', u'Source/WebCore/rendering/RenderThemeIOS.h', u'Source/WebCore/rendering/RenderThemeIOS.mm', u'Source/WebCore/rendering/RenderThemeMac.h', u'Source/WebCore/rendering/RenderThemeMac.mm', u'Source/WebCore/rendering/RenderVideo.cpp', u'Source/WebCore/rendering/RenderView.cpp', u'Source/WebCore/rendering/RenderView.h', u'Source/WebCore/rendering/RenderWidget.cpp', u'Source/WebCore/rendering/RootInlineBox.cpp', u'Source/WebCore/rendering/break_lines.cpp', '--commit-queue']" exit_code: 1 ERROR: Source/WebCore/rendering/RenderLayer.cpp:185: Wrong number of spaces before statement. (expected: 4) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayer.cpp:2725: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayer.cpp:2725: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderImage.cpp:89: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderImage.cpp:90: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderImage.cpp:90: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] ERROR: Source/WebCore/rendering/RenderImage.cpp:91: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderImage.cpp:92: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderImage.cpp:93: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderImage.cpp:94: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderImage.cpp:94: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] ERROR: Source/WebCore/rendering/RenderImage.cpp:95: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderImage.cpp:95: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] ERROR: Source/WebCore/rendering/RenderImage.cpp:96: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderImage.cpp:96: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] ERROR: Source/WebCore/rendering/RenderView.cpp:374: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderMenuList.cpp:81: Wrong number of spaces before statement. (expected: 4) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderBox.cpp:2937: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderBox.cpp:2937: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] ERROR: Source/WebCore/rendering/RenderBox.cpp:2940: Wrong number of spaces before statement. (expected: 8) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderBox.cpp:2940: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] ERROR: Source/WebCore/rendering/RenderBox.cpp:2942: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderBox.cpp:2942: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] ERROR: Source/WebCore/rendering/RenderBox.cpp:2945: Wrong number of spaces before statement. (expected: 8) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderBox.cpp:2945: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] ERROR: Source/WebCore/rendering/RenderBox.cpp:2947: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:432: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:433: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:435: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:437: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:446: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:447: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderText.cpp:372: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderFileUploadControl.cpp:106: Wrong number of spaces before statement. (expected: 8) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderFileUploadControl.cpp:107: Wrong number of spaces before statement. (expected: 8) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderFileUploadControl.cpp:109: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] Total errors found: 36 in 52 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 218443 [details] Patch Attachment 218443 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/39938327
Comment on attachment 218443 [details] Patch Attachment 218443 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/40238044
Comment on attachment 218443 [details] Patch Attachment 218443 [details] did not pass gtk-ews (gtk): Output: http://webkit-queues.appspot.com/results/39738095
(In reply to comment #2) > ERROR: Source/WebCore/rendering/RenderLayer.cpp:185: Wrong number of spaces before statement. (expected: 4) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderLayer.cpp:2725: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderLayer.cpp:2725: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] See bug #125254. > ERROR: Source/WebCore/rendering/RenderImage.cpp:89: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderImage.cpp:90: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderImage.cpp:90: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] > ERROR: Source/WebCore/rendering/RenderImage.cpp:91: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderImage.cpp:92: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderImage.cpp:93: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderImage.cpp:94: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderImage.cpp:94: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] > ERROR: Source/WebCore/rendering/RenderImage.cpp:95: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderImage.cpp:95: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] > ERROR: Source/WebCore/rendering/RenderImage.cpp:96: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderImage.cpp:96: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] See bug #125246. In particular, the first error (RenderImage.cpp:89) is covered by this bug. I suspect the subsequent warnings are fallout from this bug. > ERROR: Source/WebCore/rendering/RenderView.cpp:374: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] See bug #125246. > ERROR: Source/WebCore/rendering/RenderMenuList.cpp:81: Wrong number of spaces before statement. (expected: 4) [whitespace/indent] [4] See bug #125254. > ERROR: Source/WebCore/rendering/RenderBox.cpp:2937: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderBox.cpp:2937: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] > ERROR: Source/WebCore/rendering/RenderBox.cpp:2940: Wrong number of spaces before statement. (expected: 8) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderBox.cpp:2940: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] > ERROR: Source/WebCore/rendering/RenderBox.cpp:2942: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderBox.cpp:2942: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] > ERROR: Source/WebCore/rendering/RenderBox.cpp:2945: Wrong number of spaces before statement. (expected: 8) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderBox.cpp:2945: Comma should be at the beggining of the line in a member initialization list. [whitespace/init] [4] > ERROR: Source/WebCore/rendering/RenderBox.cpp:2947: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] See bug #125246. > ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:432: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:433: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:435: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:437: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:446: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderLayerCompositor.cpp:447: Wrong number of spaces before statement. (expected: 12) [whitespace/indent] [4] See bug #125254. > ERROR: Source/WebCore/rendering/RenderText.cpp:372: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] See bug #125246. > ERROR: Source/WebCore/rendering/RenderFileUploadControl.cpp:106: Wrong number of spaces before statement. (expected: 8) [whitespace/indent] [4] > ERROR: Source/WebCore/rendering/RenderFileUploadControl.cpp:107: Wrong number of spaces before statement. (expected: 8) [whitespace/indent] [4] See bug #125254. > ERROR: Source/WebCore/rendering/RenderFileUploadControl.cpp:109: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] See bug #125246.
Comment on attachment 218443 [details] Patch Attachment 218443 [details] did not pass win-ews (win): Output: http://webkit-queues.appspot.com/results/39738092
Created attachment 218458 [details] Patch
Attachment 218458 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj', u'Source/WebCore/rendering/InlineBox.cpp', u'Source/WebCore/rendering/InlineBox.h', u'Source/WebCore/rendering/InlineTextBox.cpp', u'Source/WebCore/rendering/RenderBlock.cpp', u'Source/WebCore/rendering/RenderBlock.h', u'Source/WebCore/rendering/RenderBox.cpp', u'Source/WebCore/rendering/RenderBox.h', u'Source/WebCore/rendering/RenderBoxModelObject.cpp', u'Source/WebCore/rendering/RenderButton.cpp', u'Source/WebCore/rendering/RenderElement.cpp', u'Source/WebCore/rendering/RenderEmbeddedObject.cpp', u'Source/WebCore/rendering/RenderFileUploadControl.cpp', u'Source/WebCore/rendering/RenderFrameSet.cpp', u'Source/WebCore/rendering/RenderIFrame.h', u'Source/WebCore/rendering/RenderImage.cpp', u'Source/WebCore/rendering/RenderImage.h', u'Source/WebCore/rendering/RenderInline.cpp', u'Source/WebCore/rendering/RenderInline.h', u'Source/WebCore/rendering/RenderLayer.cpp', u'Source/WebCore/rendering/RenderLayer.h', u'Source/WebCore/rendering/RenderLayerBacking.cpp', u'Source/WebCore/rendering/RenderLayerBacking.h', u'Source/WebCore/rendering/RenderLayerCompositor.cpp', u'Source/WebCore/rendering/RenderLayerCompositor.h', u'Source/WebCore/rendering/RenderLayerFilterInfo.h', u'Source/WebCore/rendering/RenderMenuList.cpp', u'Source/WebCore/rendering/RenderMenuList.h', u'Source/WebCore/rendering/RenderObject.cpp', u'Source/WebCore/rendering/RenderObject.h', u'Source/WebCore/rendering/RenderScrollbar.h', u'Source/WebCore/rendering/RenderSearchField.cpp', u'Source/WebCore/rendering/RenderText.cpp', u'Source/WebCore/rendering/RenderText.h', u'Source/WebCore/rendering/RenderTextControl.cpp', u'Source/WebCore/rendering/RenderTextControl.h', u'Source/WebCore/rendering/RenderTextControlMultiLine.cpp', u'Source/WebCore/rendering/RenderTextControlSingleLine.cpp', u'Source/WebCore/rendering/RenderTextLineBoxes.cpp', u'Source/WebCore/rendering/RenderTheme.cpp', u'Source/WebCore/rendering/RenderTheme.h', u'Source/WebCore/rendering/RenderThemeIOS.h', u'Source/WebCore/rendering/RenderThemeIOS.mm', u'Source/WebCore/rendering/RenderThemeMac.h', u'Source/WebCore/rendering/RenderThemeMac.mm', u'Source/WebCore/rendering/RenderVideo.cpp', u'Source/WebCore/rendering/RenderView.cpp', u'Source/WebCore/rendering/RenderView.h', u'Source/WebCore/rendering/RenderWidget.cpp', u'Source/WebCore/rendering/RootInlineBox.cpp', u'Source/WebCore/rendering/break_lines.cpp', '--commit-queue']" exit_code: 1 ERROR: Source/WebCore/rendering/RenderLayer.cpp:185: Wrong number of spaces before statement. (expected: 4) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderMenuList.cpp:81: Wrong number of spaces before statement. (expected: 4) [whitespace/indent] [4] Total errors found: 2 in 52 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 218458 [details] Patch Attachment 218458 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/42728037
Comment on attachment 218458 [details] Patch Attachment 218458 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/43328073
Created attachment 218603 [details] Patch
Attachment 218603 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj', u'Source/WebCore/rendering/InlineBox.cpp', u'Source/WebCore/rendering/InlineBox.h', u'Source/WebCore/rendering/InlineTextBox.cpp', u'Source/WebCore/rendering/RenderBlock.cpp', u'Source/WebCore/rendering/RenderBlock.h', u'Source/WebCore/rendering/RenderBox.cpp', u'Source/WebCore/rendering/RenderBox.h', u'Source/WebCore/rendering/RenderBoxModelObject.cpp', u'Source/WebCore/rendering/RenderButton.cpp', u'Source/WebCore/rendering/RenderElement.cpp', u'Source/WebCore/rendering/RenderEmbeddedObject.cpp', u'Source/WebCore/rendering/RenderFileUploadControl.cpp', u'Source/WebCore/rendering/RenderFrameSet.cpp', u'Source/WebCore/rendering/RenderIFrame.h', u'Source/WebCore/rendering/RenderImage.cpp', u'Source/WebCore/rendering/RenderImage.h', u'Source/WebCore/rendering/RenderInline.cpp', u'Source/WebCore/rendering/RenderInline.h', u'Source/WebCore/rendering/RenderLayer.cpp', u'Source/WebCore/rendering/RenderLayer.h', u'Source/WebCore/rendering/RenderLayerBacking.cpp', u'Source/WebCore/rendering/RenderLayerBacking.h', u'Source/WebCore/rendering/RenderLayerCompositor.cpp', u'Source/WebCore/rendering/RenderLayerCompositor.h', u'Source/WebCore/rendering/RenderLayerFilterInfo.h', u'Source/WebCore/rendering/RenderMenuList.cpp', u'Source/WebCore/rendering/RenderMenuList.h', u'Source/WebCore/rendering/RenderObject.cpp', u'Source/WebCore/rendering/RenderObject.h', u'Source/WebCore/rendering/RenderScrollbar.h', u'Source/WebCore/rendering/RenderSearchField.cpp', u'Source/WebCore/rendering/RenderText.cpp', u'Source/WebCore/rendering/RenderText.h', u'Source/WebCore/rendering/RenderTextControl.cpp', u'Source/WebCore/rendering/RenderTextControl.h', u'Source/WebCore/rendering/RenderTextControlMultiLine.cpp', u'Source/WebCore/rendering/RenderTextControlSingleLine.cpp', u'Source/WebCore/rendering/RenderTextLineBoxes.cpp', u'Source/WebCore/rendering/RenderTheme.cpp', u'Source/WebCore/rendering/RenderTheme.h', u'Source/WebCore/rendering/RenderThemeIOS.h', u'Source/WebCore/rendering/RenderThemeIOS.mm', u'Source/WebCore/rendering/RenderThemeMac.h', u'Source/WebCore/rendering/RenderThemeMac.mm', u'Source/WebCore/rendering/RenderVideo.cpp', u'Source/WebCore/rendering/RenderView.cpp', u'Source/WebCore/rendering/RenderView.h', u'Source/WebCore/rendering/RenderWidget.cpp', u'Source/WebCore/rendering/RootInlineBox.cpp', u'Source/WebCore/rendering/break_lines.cpp', '--commit-queue']" exit_code: 1 ERROR: Source/WebCore/rendering/RenderLayer.cpp:185: Wrong number of spaces before statement. (expected: 4) [whitespace/indent] [4] ERROR: Source/WebCore/rendering/RenderMenuList.cpp:81: Wrong number of spaces before statement. (expected: 4) [whitespace/indent] [4] Total errors found: 2 in 52 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 218603 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=218603&action=review Pleaes refresh from iOS sources. I think longer term we should replace #if PLATFORM(IOS) with functional macros, like #if USE(TOUCH_OVERFLOW_SCROLLING) or whatever. > Source/WebCore/rendering/InlineTextBox.cpp:1018 > + float documentScale = renderer().frame().documentScale(); This is stale code. > Source/WebCore/rendering/RenderBlock.cpp:3569 > + // On the phone we want to constrain VisiblePositions to the editable region closest to the input position, so "On the phone" -> "On iOS" > Source/WebCore/rendering/RenderLayerCompositor.cpp:2681 > +#if PLATFORM(IOS) > +float RenderLayerCompositor::minimumDocumentScale() const > +{ > + return m_renderView.frameView().frame().minimumDocumentScale(); > +} > + > +bool RenderLayerCompositor::allowCompositingLayerVisualDegradation() const > +{ > + Page* page = m_renderView.frameView().frame().page(); > + return !page || page->settings().allowCompositingLayerVisualDegradation(); > +} > +#endif > + > float RenderLayerCompositor::deviceScaleFactor() const > { > +#if PLATFORM(IOS) > + return m_renderView.frameView().frame().deviceScaleFactor(); > +#else > Page* page = this->page(); > return page ? page->deviceScaleFactor() : 1; > +#endif > } > > float RenderLayerCompositor::pageScaleFactor() const > { > +#if PLATFORM(IOS) > + return m_renderView.frameView().frame().documentScale(); > +#else > Page* page = this->page(); > return page ? page->pageScaleFactor() : 1; > +#endif > } Your patch is stale here. > Source/WebCore/rendering/RenderMenuList.cpp:52 > +#include "LocalizedStrings.h" Is this necessary? > Source/WebCore/rendering/RenderObject.cpp:1156 > +// which are annotated with additional state which helps the iPhone draw selections in its unique way. the iPhone -> iOS
(In reply to comment #14) > (From update of attachment 218603 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=218603&action=review > > Pleaes refresh from iOS sources. > > I think longer term we should replace #if PLATFORM(IOS) with functional macros, like #if USE(TOUCH_OVERFLOW_SCROLLING) or whatever. > I agree. I'll look to fix this up and/or file follow-up bugs as applicable. > > Source/WebCore/rendering/InlineTextBox.cpp:1018 > > + float documentScale = renderer().frame().documentScale(); > > This is stale code. Will fix. > > > Source/WebCore/rendering/RenderBlock.cpp:3569 > > + // On the phone we want to constrain VisiblePositions to the editable region closest to the input position, so > > "On the phone" -> "On iOS" > Will fix. > > Source/WebCore/rendering/RenderLayerCompositor.cpp:2681 > > +#if PLATFORM(IOS) > > +float RenderLayerCompositor::minimumDocumentScale() const > > +{ > > + return m_renderView.frameView().frame().minimumDocumentScale(); > > +} > > + > > +bool RenderLayerCompositor::allowCompositingLayerVisualDegradation() const > > +{ > > + Page* page = m_renderView.frameView().frame().page(); > > + return !page || page->settings().allowCompositingLayerVisualDegradation(); > > +} > > +#endif > > + > > float RenderLayerCompositor::deviceScaleFactor() const > > { > > +#if PLATFORM(IOS) > > + return m_renderView.frameView().frame().deviceScaleFactor(); > > +#else > > Page* page = this->page(); > > return page ? page->deviceScaleFactor() : 1; > > +#endif > > } > > > > float RenderLayerCompositor::pageScaleFactor() const > > { > > +#if PLATFORM(IOS) > > + return m_renderView.frameView().frame().documentScale(); > > +#else > > Page* page = this->page(); > > return page ? page->pageScaleFactor() : 1; > > +#endif > > } > > Your patch is stale here. Will fix. > > > Source/WebCore/rendering/RenderMenuList.cpp:52 > > +#include "LocalizedStrings.h" > > Is this necessary? > Yes, it's necessary as we call htmlSelectMultipleItems() (defined in LocalizedStrings.h) in RenderMenuList::setTextFromOption(). > > Source/WebCore/rendering/RenderObject.cpp:1156 > > +// which are annotated with additional state which helps the iPhone draw selections in its unique way. > > the iPhone -> iOS Will fix.
Committed r160236: <http://trac.webkit.org/changeset/160236>
Committed iOS build fix in <http://trac.webkit.org/changeset/161028>.