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

Issue 8634002: Revert 111080 - Remove Task inheritance in clipboard code. (Closed)

Created:
9 years, 1 month ago by dcheng
Modified:
9 years, 1 month ago
Reviewers:
dcheng
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, dcheng, dpranke-watch+content_chromium.org
Visibility:
Public.

Description

Revert 111080 - Remove Task inheritance in clipboard code. In one instance, we can simply use currying to remove the need to inherit from Task; in the other, we retain the wrapper since it wraps an Objective-C call. Also remove an overload of ui::Clipboard::WriteObjects that's never used. BUG=none TEST=trybots Review URL: http://codereview.chromium.org/8606002 TBR=dcheng@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=111083

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+35 lines, -23 lines) Patch
M content/browser/renderer_host/clipboard_message_filter.cc View 4 chunks +20 lines, -9 lines 0 comments Download
M content/browser/renderer_host/clipboard_message_filter_mac.mm View 3 chunks +3 lines, -14 lines 0 comments Download
M ui/base/clipboard/clipboard.h View 1 chunk +6 lines, -0 lines 0 comments Download
M ui/base/clipboard/clipboard_mac.mm View 1 chunk +1 line, -0 lines 0 comments Download
M ui/base/clipboard/clipboard_win.cc View 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
dcheng
9 years, 1 month ago (2011-11-22 02:36:31 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698