WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
115979
[Mac] Give every Clipboard an underlying Pasteboard
https://bugs.webkit.org/show_bug.cgi?id=115979
Summary
[Mac] Give every Clipboard an underlying Pasteboard
Darin Adler
Reported
2013-05-11 22:45:44 PDT
[Mac] Give every Clipboard an underlying Pasteboard
Attachments
Patch
(6.53 KB, patch)
2013-05-11 22:49 PDT
,
Darin Adler
kling
: review+
darin
: commit-queue-
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Darin Adler
Comment 1
2013-05-11 22:49:19 PDT
Created
attachment 201481
[details]
Patch
Andreas Kling
Comment 2
2013-05-12 15:45:46 PDT
Comment on
attachment 201481
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=201481&action=review
r=me, glad to see this is moving :)
> Source/WebCore/dom/Clipboard.h:37 > +#define USE_LEGACY_STYLE_ABSTRACT_CLIPBOARD_CLASS !PLATFORM(MAC)
I would have named this WTF_USE_LEGACY_... and used #if USE(LEGACY_...)
> Source/WebCore/platform/mac/ClipboardMac.mm:60 > - : Clipboard(policy, clipboardType) > + : Clipboard(policy, clipboardType, Pasteboard::create(m_pasteboardName)) > , m_pasteboardName(pasteboardName)
Is m_pasteboardName guaranteed to be initialized before you pass it to the base class ctor here?
Darin Adler
Comment 3
2013-05-12 18:20:15 PDT
(In reply to
comment #2
)
> > Source/WebCore/dom/Clipboard.h:37 > > +#define USE_LEGACY_STYLE_ABSTRACT_CLIPBOARD_CLASS !PLATFORM(MAC) > > I would have named this WTF_USE_LEGACY_... and used #if USE(LEGACY_...)
OK. I can change that.
> > Source/WebCore/platform/mac/ClipboardMac.mm:60 > > - : Clipboard(policy, clipboardType) > > + : Clipboard(policy, clipboardType, Pasteboard::create(m_pasteboardName)) > > , m_pasteboardName(pasteboardName) > > Is m_pasteboardName guaranteed to be initialized before you pass it to the base class ctor here?
It’s not. I thought I had fixed that! I’ll land with it fixed.
Darin Adler
Comment 4
2013-05-12 18:25:05 PDT
Committed
r149972
: <
http://trac.webkit.org/changeset/149972
>
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