[Webkit-unassigned] [Bug 44992] [chromium] Prepare Clipboard/DragData for transition to new drag-and-drop interface.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=44992


Tony Chang <tony at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #70318|review?, commit-queue?      |review+, commit-queue-
               Flag|                            |




--- Comment #25 from Tony Chang <tony at chromium.org>  2010-10-11 11:07:24 PST ---
(From update of attachment 70318)
View in context: https://bugs.webkit.org/attachment.cgi?id=70318&action=review

> WebCore/platform/chromium/ClipboardChromium.cpp:93
> +        policy == ClipboardWritable
> +        ? ChromiumDataObject::createWritable(clipboardType)
> +        : ChromiumDataObject::createReadable(clipboardType);

In RDO.cpp, you put the ternary operator at the end of lines.  Can you be consistent?  I'm not sure there's an actual style rule. In practice, most people just put the whole expression on a single line (which also seems fine to me).

> WebCore/platform/chromium/ReadableDataObject.cpp:82
> +                Pasteboard::generalPasteboard()->isSelectionMode() ?
> +                PasteboardPrivate::SelectionBuffer :
> +                PasteboardPrivate::StandardBuffer;

This is where you use the opposite style.

> WebKit/chromium/public/WebView.h:228
> +    virtual WebDragOperation dragTargetDragEnter2(
> +        int identity, const WebPoint& clientPoint, const WebPoint& screenPoint,
> +        WebDragOperationsMask operationsAllowed) = 0;

I have a slight preference to using dragTargetDragEnterNew rather than 2, but either is ok.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.