WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
160967
Move an NPAPI-only function from WebCore to WebKit
https://bugs.webkit.org/show_bug.cgi?id=160967
Summary
Move an NPAPI-only function from WebCore to WebKit
Anders Carlsson
Reported
2016-08-18 12:01:33 PDT
Move an NPAPI-only function from WebCore to WebKit
Attachments
Patch
(12.89 KB, patch)
2016-08-18 12:02 PDT
,
Anders Carlsson
no flags
Details
Formatted Diff
Diff
Patch
(14.92 KB, patch)
2016-08-18 12:15 PDT
,
Anders Carlsson
no flags
Details
Formatted Diff
Diff
Patch
(16.06 KB, patch)
2016-08-18 12:17 PDT
,
Anders Carlsson
no flags
Details
Formatted Diff
Diff
Patch
(16.06 KB, patch)
2016-08-18 12:17 PDT
,
Anders Carlsson
no flags
Details
Formatted Diff
Diff
Patch
(16.97 KB, patch)
2016-08-18 13:04 PDT
,
Anders Carlsson
no flags
Details
Formatted Diff
Diff
Patch
(16.97 KB, patch)
2016-08-18 14:01 PDT
,
Anders Carlsson
sam
: review+
Details
Formatted Diff
Diff
Show Obsolete
(5)
View All
Add attachment
proposed patch, testcase, etc.
Anders Carlsson
Comment 1
2016-08-18 12:02:11 PDT
Created
attachment 286382
[details]
Patch
Anders Carlsson
Comment 2
2016-08-18 12:15:59 PDT
Created
attachment 286386
[details]
Patch
Anders Carlsson
Comment 3
2016-08-18 12:17:02 PDT
Created
attachment 286387
[details]
Patch
Anders Carlsson
Comment 4
2016-08-18 12:17:50 PDT
Created
attachment 286388
[details]
Patch
Anders Carlsson
Comment 5
2016-08-18 13:04:00 PDT
Created
attachment 286391
[details]
Patch
Anders Carlsson
Comment 6
2016-08-18 14:01:04 PDT
Created
attachment 286392
[details]
Patch
Anders Carlsson
Comment 7
2016-08-18 17:14:06 PDT
Committed
r204617
: <
http://trac.webkit.org/changeset/204617
>
Darin Adler
Comment 8
2016-08-19 13:15:53 PDT
Comment on
attachment 286392
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=286392&action=review
> Source/WebKit/win/Plugins/PluginView.cpp:756 > + , m_elementNPObject(nullptr)
Why not initialize this in the class definition instead?
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