Fix Bookmark manager DnD: WebDragDestinationDelegate was getting recreated with

Fix Bookmark manager DnD: WebDragDestinationDelegate was getting recreated with
every call to GetDragDestDelegate() destroying local state.

BUG=123359
TEST=dnd works on bookmark manager


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=134782

[sky, 2012-05-01 17:33:51]

https://chromiumcodereview.appspot.com/10273030/diff/1/content/browser/web_co...
File content/browser/web_contents/web_contents_view_aura.cc (right):

https://chromiumcodereview.appspot.com/10273030/diff/1/content/browser/web_co...
content/browser/web_contents/web_contents_view_aura.cc:217: if (delegate_.get())
Add a comment as to why you're doing this. Also, you're not initializing it and
it doesn't seem that you ever reset it (I assume you need an else case that sets
it to NULL).

[varunjain, 2012-05-01 17:56:35]

https://chromiumcodereview.appspot.com/10273030/diff/1/content/browser/web_co...
File content/browser/web_contents/web_contents_view_aura.cc (right):

https://chromiumcodereview.appspot.com/10273030/diff/1/content/browser/web_co...
content/browser/web_contents/web_contents_view_aura.cc:217: if (delegate_.get())
On 2012/05/01 17:33:51, sky wrote:
> Add a comment as to why you're doing this. Also, you're not initializing it
and
> it doesn't seem that you ever reset it (I assume you need an else case that
sets
> it to NULL).

added comment and initialized to NULL

[sky, 2012-05-01 18:19:59]

LGTM

[commit-bot: I haz the power, 2012-05-01 19:09:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/varunjain@chromium.org/10273030/7001

[commit-bot: I haz the power, 2012-05-01 21:43:19]

Change committed as 134782