WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED WONTFIX
59007
[chromium] remove temporary API now that Chrome is updated
https://bugs.webkit.org/show_bug.cgi?id=59007
Summary
[chromium] remove temporary API now that Chrome is updated
Evan Martin
Reported
2011-04-20 10:25:41 PDT
[chromium] remove temporary API now that Chrome is updated
Attachments
Patch
(1.59 KB, patch)
2011-04-20 10:26 PDT
,
Evan Martin
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Evan Martin
Comment 1
2011-04-20 10:26:42 PDT
Created
attachment 90358
[details]
Patch
Evan Martin
Comment 2
2011-04-20 10:27:17 PDT
I landed the Chrome-side change now. Not sure if I need to change DEPS here or anything. Feedback welcome.
Tony Chang
Comment 3
2011-04-20 10:34:23 PDT
You need to update Source/WebKit/chromium/DEPS to pick up your change. I normally just make sure it compiles locally and commit it as a TBR and watch the bots.
Darin Fisher (:fishd, Google)
Comment 4
2011-04-20 12:58:03 PDT
Comment on
attachment 90358
[details]
Patch webkit depends back on some chromium files to support DRT. but i dont think that includes a dependency on a WebFrameClient impl. so maybe you dont need to roll deps?
Stephen Chenney
Comment 5
2013-04-11 13:38:50 PDT
Assuming fixed as this is an old issue. Reopen in crbug if that's not true.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug