Bug 91067

Summary: [Chromium] Remove unrequired API in WebSurroundingText
Product: WebKit Reporter: Leandro Graciá Gil <leandrogracia>
Component: WebKit APIAssignee: Leandro Graciá Gil <leandrogracia>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, dglazkov, fishd, jamesr, tkent+wkapi, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Leandro Graciá Gil
Reported 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.
Attachments
Patch (2.78 KB, patch)
2012-07-12 04:34 PDT, Leandro Graciá Gil 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Leandro Graciá Gil
Comment 1 2012-07-12 04:34:26 PDT
Created attachment 151909 [details] Patch
WebKit Review Bot
Comment 2 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.
WebKit Review Bot
Comment 3 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>
WebKit Review Bot
Comment 4 2012-07-12 08:31:22 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.