Bug 91067 - [Chromium] Remove unrequired API in WebSurroundingText
Summary: [Chromium] Remove unrequired API in WebSurroundingText
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit API (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Leandro Graciá Gil
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-12 03:15 PDT by Leandro Graciá Gil
Modified: 2012-07-12 08:31 PDT (History)
6 users (show)

See Also:

Attachments
Patch (2.78 KB, patch)
2012-07-12 04:34 PDT, Leandro Graciá Gil 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Leandro Graciá Gil 2012-07-12 03:15:54 PDT 
Currently WebSurroundingText provides two different but very similar initialize methods. The first was used by Chromium, which recently moved to the second one (changeset 146154). Consequently, both layout tests and Chromium now use the same API (2nd) and the old one (1st) can be safely removed.
Comment 1 Leandro Graciá Gil 2012-07-12 04:34:26 PDT 
Created attachment 151909 [details]
Patch
Comment 2 WebKit Review Bot 2012-07-12 04:36:34 PDT 
Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.
Comment 3 WebKit Review Bot 2012-07-12 08:31:12 PDT 
Comment on attachment 151909 [details]
Patch

Clearing flags on attachment: 151909

Committed r122461: <http://trac.webkit.org/changeset/122461>
Comment 4 WebKit Review Bot 2012-07-12 08:31:22 PDT 
All reviewed patches have been landed.  Closing bug.