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

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.