[Webkit-unassigned] [Bug 195360] [iOS] Hit test for content overlapping fast-scrollable overflow

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=195360

Sam Weinig <sam at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |sam at webkit.org

--- Comment #2 from Sam Weinig <sam at webkit.org> ---
Comment on attachment 363739
  --> https://bugs.webkit.org/attachment.cgi?id=363739
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=363739&action=review

> Source/WebKit/UIProcess/RemoteLayerTree/ios/RemoteLayerTreeViews.mm:77
> +    for (auto i = viewsAtPoint.size(); i--;) {

I know this doesn't really have to do with this bug, this makes me sad I still haven't added the reversed adaptor :(.  I think all it would end up being is something like:

template<typename Container>
IteratorRange<typename Container::reverse_iterator> makeReversedIteratorRange(Container& container)
{
    return makeIteratorRange(std::rbegin(container), std::rend(container));
}

template<typename Container>
IteratorRange<typename Container::const_reverse_iterator> makeReversedIteratorRange(const Container& container)
{
    return makeIteratorRange(std::crbegin(container), std::crend(container));
}


and then this loop would become:

for (auto *view : makeReversedIteratorRange(viewsAtPoint)) {

Do you think that's worth adding?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20190306/d15b4c68/attachment-0001.html>