[Webkit-unassigned] [Bug 34915] Chromium: Need to be able to get the bounds of selection rectangle(s)
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Feb 22 17:31:40 PST 2010
https://bugs.webkit.org/show_bug.cgi?id=34915
David Levin <levin at chromium.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #49185|review?, commit-queue? |review-, commit-queue-
Flag| |
--- Comment #13 from David Levin <levin at chromium.org> 2010-02-22 17:31:40 PST ---
(From update of attachment 49185)
Some trivial nits to fix and then I'd be willing to r+, cq+ (based on this plus
past reviews).
> diff --git a/WebKit/chromium/ChangeLog b/WebKit/chromium/ChangeLog
> @@ -1,3 +1,21 @@
> +2010-02-21 Dmitriy Belenko <dbelenko at google.com>
> +
> + Reviewed by NOBODY (OOPS!).
> +
Typically one does:
Bug title
Bug link
Description.
Like this:
> + Chromium: Need to be able to get the bounds of selection rectangle(s)
> + https://bugs.webkit.org/show_bug.cgi?id=34915
> +
> + This change will enable about 30 test cases to pass in Chromium.
> + All of these test cases are related to selection rect boundaries.
> + This change will enable the test cases to retrieve the selection
> + rect boundary rectangle for the current selection.
> +
> +
> + * public/WebFrame.h:
> + * src/WebFrameImpl.cpp:
> + (WebKit::WebFrameImpl::selectionBoundsRect):
> + * src/WebFrameImpl.h:
> +
> diff --git a/WebKit/chromium/public/WebFrame.h b/WebKit/chromium/public/WebFrame.h
> + // Returns the bounds rect for current selection. If selection is performed
> + // on transformed text, the rect will still bound the selection, but will
no comma before the but (since the phrase after but isn't a complete sentence.
> + // not be transformed itself. If no selection is present the rect will be
Add a comma after present.
> + // empty ((0,0), (0,0))
Add a period to the end of the sentence.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list