Revert of Update BookmarkBarController to use non-deprecated dragging APIs.

Revert of Update BookmarkBarController to use non-deprecated dragging APIs. (patchset #7 id:120001 of https://codereview.chromium.org/1815053002/ )

Reason for revert:
Prevents drag and drop from bookmark bar to tabstrip:
https://bugs.chromium.org/p/chromium/issues/detail?id=599522

Original issue's description:
> Update BookmarkBarController to use non-deprecated dragging APIs.
> 
> This CL theoretically should not produce any behavior change.
> 
> This CL replaces the method -[NSView dragImage:...] with -[NSView
> beginDraggingSessionWithItems:...]. There are three major differences.
> 
> 1. The new API is asynchronous, whereas the old one ran a nested run loop. This
> CL runs a nested run loop to maintain the previous behavior.
> 2. The new API makes use of some new terminology and protocols
> (NSDraggingSession, NSDraggingSource, etc.) but the underlying logic is still
> the same. This CL renames some methods and variables.
> 3. The new API requires that custom PasteBoard types use reverse DNS notation.
> 
> BUG=592663
> 
> Committed: https://crrev.com/f2200abc5d620cc7495635767bf833aacd7af48f
> Cr-Commit-Position: refs/heads/master@{#382722}

TBR=avi@chromium.org,mark@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=592663

[erikchen, 2016-04-01 14:55:19]

Created Revert of Update BookmarkBarController to use non-deprecated dragging
APIs.

[commit-bot: I haz the power, 2016-04-01 14:55:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1853503004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1853503004/1

[erikchen, 2016-04-01 14:57:46]

The CQ bit was unchecked by erikchen@chromium.org

[erikchen, 2016-04-03 15:24:52]

Description was changed from

==========
Revert of Update BookmarkBarController to use non-deprecated dragging APIs.
(patchset #7 id:120001 of https://codereview.chromium.org/1815053002/ )

Reason for revert:
Prevents drag and drop from bookmark bar to tabstrip:
https://bugs.chromium.org/p/chromium/issues/detail?id=599522

Original issue's description:
> Update BookmarkBarController to use non-deprecated dragging APIs.
> 
> This CL theoretically should not produce any behavior change.
> 
> This CL replaces the method -[NSView dragImage:...] with -[NSView
> beginDraggingSessionWithItems:...]. There are three major differences.
> 
> 1. The new API is asynchronous, whereas the old one ran a nested run loop.
This
> CL runs a nested run loop to maintain the previous behavior.
> 2. The new API makes use of some new terminology and protocols
> (NSDraggingSession, NSDraggingSource, etc.) but the underlying logic is still
> the same. This CL renames some methods and variables.
> 3. The new API requires that custom PasteBoard types use reverse DNS notation.
> 
> BUG=592663
> 
> Committed: https://crrev.com/f2200abc5d620cc7495635767bf833aacd7af48f
> Cr-Commit-Position: refs/heads/master@{#382722}

TBR=avi@chromium.org,mark@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=592663
==========

to

==========
Revert of Update BookmarkBarController to use non-deprecated dragging APIs.
(patchset #7 id:120001 of https://codereview.chromium.org/1815053002/ )

Reason for revert:
Prevents drag and drop from bookmark bar to tabstrip:
https://bugs.chromium.org/p/chromium/issues/detail?id=599522

Original issue's description:
> Update BookmarkBarController to use non-deprecated dragging APIs.
> 
> This CL theoretically should not produce any behavior change.
> 
> This CL replaces the method -[NSView dragImage:...] with -[NSView
> beginDraggingSessionWithItems:...]. There are three major differences.
> 
> 1. The new API is asynchronous, whereas the old one ran a nested run loop.
This
> CL runs a nested run loop to maintain the previous behavior.
> 2. The new API makes use of some new terminology and protocols
> (NSDraggingSession, NSDraggingSource, etc.) but the underlying logic is still
> the same. This CL renames some methods and variables.
> 3. The new API requires that custom PasteBoard types use reverse DNS notation.
> 
> BUG=592663
> 
> Committed: https://crrev.com/f2200abc5d620cc7495635767bf833aacd7af48f
> Cr-Commit-Position: refs/heads/master@{#382722}

TBR=avi@chromium.org,mark@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=592663
==========