Bug 91067 - [Chromium] Remove unrequired API in WebSurroundingText
Summary: [Chromium] Remove unrequired API in WebSurroundingText
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
Depends on:
Reported: 2012-07-12 03:15 PDT by Leandro Graciá Gil
Modified: 2012-07-12 08:31 PDT (History)
6 users (show)

See Also:

Patch (2.78 KB, patch)
2012-07-12 04:34 PDT, Leandro Graciá Gil
no flags Details | Formatted Diff | Diff

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]
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]

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.