Bug 113490 - Use String.trim() in editing/selection/move-vertically-with-paddings-borders.html test
Summary: Use String.trim() in editing/selection/move-vertically-with-paddings-borders....
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 110487
  Show dependency treegraph
 
Reported: 2013-03-28 01:30 PDT by Claudio Saavedra
Modified: 2013-03-28 12:51 PDT (History)
2 users (show)

See Also:


Attachments
Patch (1.63 KB, patch)
2013-03-28 01:31 PDT, Claudio Saavedra
no flags Details | Formatted Diff | Diff
Patch for landing (1.62 KB, patch)
2013-03-28 12:14 PDT, Claudio Saavedra
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Claudio Saavedra 2013-03-28 01:30:13 PDT
Use String.trim() in editing/selection/move-vertically-with-paddings-borders.html test
Comment 1 Claudio Saavedra 2013-03-28 01:31:21 PDT
Created attachment 195499 [details]
Patch
Comment 2 Ryosuke Niwa 2013-03-28 11:25:32 PDT
Comment on attachment 195499 [details]
Patch

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

> LayoutTests/editing/selection/move-vertically-with-paddings-borders.html:39
> -    return window.getSelection().toString();
> +    return window.getSelection().toString().trim();

window. is not needed here.
Comment 3 Claudio Saavedra 2013-03-28 12:14:53 PDT
Created attachment 195615 [details]
Patch for landing
Comment 4 WebKit Review Bot 2013-03-28 12:51:01 PDT
Comment on attachment 195615 [details]
Patch for landing

Clearing flags on attachment: 195615

Committed r147143: <http://trac.webkit.org/changeset/147143>
Comment 5 WebKit Review Bot 2013-03-28 12:51:04 PDT
All reviewed patches have been landed.  Closing bug.