EFL and Qt use m_viewSize as a contents size while Apple use it as a viewport size, so EFL and Qt introduced m_viewportSize to make WebPage know a viewport size. EFL and Qt use m_viewSize as a contents size because the size of non compositing layer is contents size, and EFL and Qt mark whole non compositing layer as dirty using m_viewSize. Bug 110299 (http://webkit.org/b/110299) changes that it is not necessary to know m_viewSize when marking whole non compositing layer as dirty. So we can make EFL and Qt use m_viewSize as a viewport size also. This patch removes m_viewportSize and related methods to increase readability.
Created attachment 189256 [details] Patch
Comment on attachment 189256 [details] Patch I'm glad that m_viewSize now means viewport size everywhere, r=me.
Comment on attachment 189256 [details] Patch Clearing flags on attachment: 189256 Committed r143474: <http://trac.webkit.org/changeset/143474>
All reviewed patches have been landed. Closing bug.
(In reply to comment #2) > (From update of attachment 189256 [details]) > I'm glad that m_viewSize now means viewport size everywhere, r=me. Thank you for review. I'm glad too :)