Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(189)

Issue 1037008: Retry r41799:... (Closed)

Created:
10 years, 9 months ago by Evan Stade
Modified:
9 years, 6 months ago
CC:
chromium-reviews, jam+cc_chromium.org, brettw+cc_chromium.org, ben+cc_chromium.org, John Grabowski, Erik does not do reviews, Aaron Boodman, pam+watch_chromium.org, darin-cc_chromium.org
Visibility:
Public.

Description

Retry r41799: Basic DragImage implementation. Only the chromium part is for review. The webkit part shows how that will look when I create the patch for webkit later. This can be landed without the change to webkit. TODO later: - use the image on windows, mac - implement the other DragImageRef functions TEST=drag an image from the render view in GTK BUG=11457 Review URL: http://codereview.chromium.org/668125 The problem was an incorrect method signature in RenderView. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=41953

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+145 lines, -29 lines) Patch
M chrome/browser/chromeos/app_launcher.h View 2 chunks +5 lines, -1 line 0 comments Download
M chrome/browser/chromeos/app_launcher.cc View 1 chunk +3 lines, -1 line 0 comments Download
M chrome/browser/cocoa/notifications/balloon_view_host_mac.h View 2 chunks +7 lines, -1 line 0 comments Download
M chrome/browser/extensions/extension_host.h View 1 chunk +3 lines, -1 line 0 comments Download
chrome/browser/extensions/extension_host.cc View 1 chunk +3 lines, -1 line 0 comments Download
M chrome/browser/gtk/notifications/balloon_view_host_gtk.h View 1 chunk +3 lines, -1 line 0 comments Download
M chrome/browser/gtk/tab_contents_drag_source.h View 3 chunks +10 lines, -1 line 0 comments Download
M chrome/browser/gtk/tab_contents_drag_source.cc View 3 chunks +13 lines, -1 line 0 comments Download
M chrome/browser/renderer_host/render_view_host.h View 1 chunk +3 lines, -1 line 0 comments Download
M chrome/browser/renderer_host/render_view_host.cc View 1 chunk +4 lines, -2 lines 0 comments Download
M chrome/browser/renderer_host/render_view_host_delegate.h View 2 chunks +4 lines, -1 line 0 comments Download
M chrome/browser/tab_contents/interstitial_page.cc View 2 chunks +6 lines, -2 lines 0 comments Download
M chrome/browser/tab_contents/tab_contents_view_gtk.h View 1 chunk +3 lines, -1 line 0 comments Download
M chrome/browser/tab_contents/tab_contents_view_gtk.cc View 1 chunk +5 lines, -2 lines 0 comments Download
M chrome/browser/tab_contents/tab_contents_view_mac.h View 2 chunks +7 lines, -1 line 0 comments Download
M chrome/browser/tab_contents/tab_contents_view_mac.mm View 2 chunks +6 lines, -2 lines 0 comments Download
M chrome/browser/views/notifications/balloon_view_host.h View 2 chunks +7 lines, -1 line 0 comments Download
M chrome/browser/views/tab_contents/tab_contents_view_gtk.h View 2 chunks +7 lines, -1 line 0 comments Download
M chrome/browser/views/tab_contents/tab_contents_view_gtk.cc View 1 chunk +5 lines, -2 lines 0 comments Download
M chrome/browser/views/tab_contents/tab_contents_view_win.h View 2 chunks +7 lines, -1 line 0 comments Download
chrome/browser/views/tab_contents/tab_contents_view_win.cc View 1 chunk +4 lines, -1 line 0 comments Download
M chrome/common/render_messages_internal.h View 1 chunk +4 lines, -2 lines 0 comments Download
M chrome/renderer/render_view.h View 2 chunks +7 lines, -0 lines 1 comment Download
M chrome/renderer/render_view.cc View 3 chunks +19 lines, -1 line 0 comments Download

Messages

Total messages: 5 (0 generated)
Evan Stade
I was wrong; it did affect linux, it just didn't affect *me* because of some ...
10 years, 9 months ago (2010-03-17 21:22:35 UTC) #1
Evan Stade
trybots with correct base revision: http://build.chromium.org/buildbot/try-server/builders/win/builds/23529 http://build.chromium.org/buildbot/try-server/builders/mac/builds/22505 http://build.chromium.org/buildbot/try-server/builders/linux/builds/22483
10 years, 9 months ago (2010-03-17 21:35:35 UTC) #2
jam
I'm not familiar with this code, can it be sent for review to the original ...
10 years, 9 months ago (2010-03-17 21:57:27 UTC) #3
Evan Stade
+tony http://codereview.chromium.org/1037008/diff/1/25 File chrome/renderer/render_view.h (right): http://codereview.chromium.org/1037008/diff/1/25#newcode252 chrome/renderer/render_view.h:252: virtual void startDragging( this was the method signature ...
10 years, 9 months ago (2010-03-17 22:08:03 UTC) #4
tony
10 years, 9 months ago (2010-03-18 00:10:14 UTC) #5
LG

Powered by Google App Engine
This is Rietveld 408576698