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

Issue 10824359: Remove ImageSkia::empty and ImageSkia::extractSubset (Closed)

Created:
8 years, 4 months ago by pkotwicz
Modified:
8 years, 4 months ago
Reviewers:
sky
CC:
chromium-reviews, mihaip-chromium-reviews_chromium.org, sadrul, nkostylev+watch_chromium.org, ben+watch_chromium.org, tfarina, dcheng, Aaron Boodman, rsesek+watch_chromium.org, oshima+watch_chromium.org, stevenjb+watch_chromium.org, davemoore+watch_chromium.org
Visibility:
Public.

Description

The way that ImageSkia::empty has been implemented, it is equivalent to ImageSkia::isNull. Replaced all instances of ImageSkia::empty with ImageSkia::isNull Removed ImageSkia::extractSubset, and replaced calls to use ImageSkiaOperations::ExtractSubset instead BUG=None Test=Compiles, try bots pass. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=152731

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Patch Set 4 : #

Patch Set 5 : #

Patch Set 6 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+51 lines, -68 lines) Patch
M ash/desktop_background/desktop_background_view.cc View 1 chunk +1 line, -1 line 0 comments Download
M ash/launcher/launcher_button.cc View 2 1 chunk +1 line, -1 line 0 comments Download
M ash/launcher/tabbed_launcher_button.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M ash/system/power/tray_power.cc View 1 3 chunks +3 lines, -4 lines 0 comments Download
M chrome/browser/chromeos/login/user_image.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chromeos/login/user_manager_impl.cc View 1 3 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/chromeos/status/network_menu.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chromeos/status/network_menu_icon.cc View 1 2 3 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/extensions/image_loading_tracker.cc View 1 2 3 4 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/notifications/desktop_notification_service.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/ash/launcher/browser_launcher_item_controller.cc View 1 2 3 1 chunk +6 lines, -4 lines 0 comments Download
M chrome/browser/ui/views/ash/balloon_view_ash.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/views/web_intent_picker_views.cc View 1 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/ui/webui/chromeos/login/network_dropdown.cc View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/options/chromeos/change_picture_options_handler.cc View 1 3 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/ui/webui/options/chromeos/wallpaper_source.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/extension_action.cc View 1 chunk +1 line, -1 line 0 comments Download
M ui/app_list/app_list_item_view.cc View 1 chunk +1 line, -1 line 0 comments Download
M ui/app_list/search_result_view.cc View 1 chunk +1 line, -1 line 0 comments Download
M ui/base/resource/resource_bundle.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M ui/gfx/image/image_skia.h View 1 2 3 2 chunks +0 lines, -8 lines 0 comments Download
M ui/gfx/image/image_skia.cc View 1 2 3 2 chunks +0 lines, -10 lines 0 comments Download
M ui/gfx/image/image_skia_util_mac.mm View 1 2 3 4 5 3 chunks +3 lines, -3 lines 0 comments Download
M ui/views/controls/button/image_button.cc View 1 chunk +1 line, -1 line 0 comments Download
M ui/views/controls/button/image_button_unittest.cc View 1 4 chunks +5 lines, -5 lines 0 comments Download
M ui/views/controls/button/text_button.h View 1 chunk +1 line, -1 line 0 comments Download
M ui/views/controls/image_view.cc View 1 chunk +1 line, -1 line 0 comments Download
M ui/views/controls/menu/menu_item_view.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M ui/views/controls/menu/menu_win.cc View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 14 (0 generated)
pkotwicz
The way that ImageSkia::empty has been implemented, it is equivalent to ImageSkia::isNull. Replaced all instances ...
8 years, 4 months ago (2012-08-20 15:42:58 UTC) #1
sky
Peter mentioned this is a temporary step. We're going to rename isNull to empty next. ...
8 years, 4 months ago (2012-08-20 16:57:16 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/10824359/6004
8 years, 4 months ago (2012-08-20 21:00:58 UTC) #3
commit-bot: I haz the power
Try job failure for 10824359-6004 (retry) on win for step "compile" (clobber build). It's a ...
8 years, 4 months ago (2012-08-20 22:15:04 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/10824359/6004
8 years, 4 months ago (2012-08-21 17:40:18 UTC) #5
commit-bot: I haz the power
Failed to apply patch for chrome/browser/ui/ash/launcher/browser_launcher_item_controller.cc: While running patch -p1 --forward --force; patching file chrome/browser/ui/ash/launcher/browser_launcher_item_controller.cc ...
8 years, 4 months ago (2012-08-21 17:51:57 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/10824359/8005
8 years, 4 months ago (2012-08-21 17:58:38 UTC) #7
commit-bot: I haz the power
Failed to apply patch for chrome/browser/ui/ash/launcher/browser_launcher_item_controller.cc: While running patch -p1 --forward --force; patching file chrome/browser/ui/ash/launcher/browser_launcher_item_controller.cc ...
8 years, 4 months ago (2012-08-21 17:58:50 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/10824359/54
8 years, 4 months ago (2012-08-21 18:58:40 UTC) #9
commit-bot: I haz the power
Try job failure for 10824359-54 (retry) on mac for step "compile" (clobber build). It's a ...
8 years, 4 months ago (2012-08-21 19:23:45 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/10824359/9014
8 years, 4 months ago (2012-08-21 20:42:02 UTC) #11
commit-bot: I haz the power
Try job failure for 10824359-9014 (retry) on mac for step "compile" (clobber build). It's a ...
8 years, 4 months ago (2012-08-21 20:56:57 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/10824359/19001
8 years, 4 months ago (2012-08-21 23:03:05 UTC) #13
commit-bot: I haz the power
8 years, 4 months ago (2012-08-22 04:11:25 UTC) #14
Change committed as 152731

Powered by Google App Engine
This is Rietveld 408576698