WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
140862
Long spins under Frame::rangeForPoint when doing Lookup on feedly.com
https://bugs.webkit.org/show_bug.cgi?id=140862
Summary
Long spins under Frame::rangeForPoint when doing Lookup on feedly.com
Tim Horton
Reported
2015-01-25 00:09:34 PST
Long spins under Frame::rangeForPoint when doing Lookup on feedly.com
Attachments
Patch
(1.79 KB, patch)
2015-01-25 00:10 PST
,
Tim Horton
mitz: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Tim Horton
Comment 1
2015-01-25 00:10:03 PST
Created
attachment 245296
[details]
Patch
mitz
Comment 2
2015-01-25 00:12:59 PST
Comment on
attachment 245296
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=245296&action=review
> Source/WebCore/page/Frame.cpp:856 > + if (!containerText) > + return nullptr; > + if (!containerText->renderer() || containerText->renderer()->style().userSelect() == SELECT_NONE)
I’d combine these into one condition.
Tim Horton
Comment 3
2015-01-25 00:18:06 PST
http://trac.webkit.org/changeset/179086
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug