RESOLVED FIXED 146872
[iOS WK2] Scrolling issues on horizontally scrollable RTL pages 
https://bugs.webkit.org/show_bug.cgi?id=146872
Summary [iOS WK2] Scrolling issues on horizontally scrollable RTL pages
Simon Fraser (smfr)
Reported 2015-07-10 22:14:26 PDT
[iOS WK2] Scrolling issues on horizontally scrollable RTL pages
Attachments
Patch (22.65 KB, patch)
2015-07-10 22:24 PDT, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
Patch (26.97 KB, patch)
2015-07-10 23:41 PDT, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
Patch (27.02 KB, patch)
2015-07-11 10:51 PDT, Simon Fraser (smfr) 		sam: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1 2015-07-10 22:24:57 PDT
Created attachment 256643 [details] Patch
Simon Fraser (smfr)
Comment 2 2015-07-10 23:41:45 PDT
Created attachment 256648 [details] Patch
Simon Fraser (smfr)
Comment 3 2015-07-11 10:51:12 PDT
Created attachment 256657 [details] Patch
Sam Weinig
Comment 4 2015-07-11 10:58:11 PDT
Comment on attachment 256657 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=256657&action=review > Source/WebKit2/Shared/mac/RemoteLayerTreeTransaction.mm:1250 > + dumpProperty(ts, "transactionID", m_transactionID); If we ever use this for tests, will putting the transactionID in the dump make it inconsistent. Or will we use a different dumping mechanism for testing.
Simon Fraser (smfr)
Comment 5 2015-07-11 11:21:43 PDT
https://trac.webkit.org/r186712 rdar://problem/7569416
Tim Horton
Comment 6 2015-07-11 15:11:39 PDT
(In reply to comment #4) > Comment on attachment 256657 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=256657&action=review > > > Source/WebKit2/Shared/mac/RemoteLayerTreeTransaction.mm:1250 > > + dumpProperty(ts, "transactionID", m_transactionID); > > If we ever use this for tests, will putting the transactionID in the dump > make it inconsistent. Or will we use a different dumping mechanism for > testing. I can't imagine that being any worse than the layer IDs, which are already in there.
Note You need to log in before you can comment on or make changes to this bug.