Bug 40869 - Change some WebPluginDabase ivars to use WebCore::String instead of NSString.
Summary: Change some WebPluginDabase ivars to use WebCore::String instead of NSString.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Anders Carlsson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-18 17:28 PDT by Anders Carlsson
Modified: 2010-06-20 13:28 PDT (History)
3 users (show)

See Also:


Attachments
Patch (10.24 KB, patch)
2010-06-18 17:29 PDT, Anders Carlsson
no flags Details | Formatted Diff | Diff
Patch (12.70 KB, patch)
2010-06-20 13:18 PDT, Anders Carlsson
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Anders Carlsson 2010-06-18 17:28:25 PDT
Change some WebPluginDabase ivars to use WebCore::String instead of NSString.
Comment 1 Anders Carlsson 2010-06-18 17:29:31 PDT
Created attachment 59171 [details]
Patch
Comment 2 Sam Weinig 2010-06-18 17:32:07 PDT
Comment on attachment 59171 [details]
Patch

r=me
Comment 3 Anders Carlsson 2010-06-18 17:35:22 PDT
Committed r61459: <http://trac.webkit.org/changeset/61459>
Comment 4 Adam Barth 2010-06-18 20:19:45 PDT
Might this have caused these tests to start timing out on Tiger?

dom/html/level2/html/AppletsCollection.html
plugins/qt-qwidget-plugin.html
Comment 5 Adam Barth 2010-06-18 20:26:04 PDT
Hum...  It's 8:30 on a Friday and I don't see you on IRC.  I'm going to try rolling this out to see if it fixes the problem.
Comment 6 Adam Barth 2010-06-18 20:36:08 PDT
Rolled out in http://trac.webkit.org/changeset/61464
Comment 7 Adam Barth 2010-06-18 20:36:32 PDT
Comment on attachment 59171 [details]
Patch

Clearing Sam's r+
Comment 8 WebKit Review Bot 2010-06-18 21:12:25 PDT
http://trac.webkit.org/changeset/61464 might have broken SnowLeopard Intel Release (Tests)
Comment 9 WebKit Review Bot 2010-06-18 22:27:31 PDT
http://trac.webkit.org/changeset/61459 might have broken Leopard Intel Debug (Tests)
The following changes are on the blame list:
http://trac.webkit.org/changeset/61458
http://trac.webkit.org/changeset/61459
Comment 10 Anders Carlsson 2010-06-20 13:18:03 PDT
Created attachment 59212 [details]
Patch
Comment 11 Anders Carlsson 2010-06-20 13:28:10 PDT
Committed r61509: <http://trac.webkit.org/changeset/61509>