[webkit-reviews] review granted: [Bug 97924] [Chromium] WebFrameImpl.cpp could use a style cleanup : [Attachment 166280] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Sep 28 11:03:18 PDT 2012


Dimitri Glazkov (Google) <dglazkov at chromium.org> has granted Adam Barth
<abarth at webkit.org>'s request for review:
Bug 97924: [Chromium] WebFrameImpl.cpp could use a style cleanup
https://bugs.webkit.org/show_bug.cgi?id=97924

Attachment 166280: Patch
https://bugs.webkit.org/attachment.cgi?id=166280&action=review

------- Additional Comments from Dimitri Glazkov (Google)
<dglazkov at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=166280&action=review


> Source/WebKit/chromium/src/WebFrameImpl.cpp:2147
> +    return IntRect(frame()->selection()->bounds(false));

Would ternary if be good here?


More information about the webkit-reviews mailing list