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

Issue 552553002: Fix the uses of T* conversion operator from scoped_refptr<T> which is now removed. (Closed)

Created:
6 years, 3 months ago by lcwu1
Modified:
6 years, 3 months ago
Reviewers:
spang
CC:
chromium-reviews, rjkroege, kalyank, tdresser+watch_chromium.org, piman+watch_chromium.org, ozone-reviews_chromium.org, damienv1, gunsch
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Fix the uses of T* conversion operator from scoped_refptr<T> which is now removed. Please see https://codereview.chromium.org/510323002 for reference. NOTRY=true Committed: https://crrev.com/8a59821dfe4696acf6269891b0ef0bfb7ad33a3f Cr-Commit-Position: refs/heads/master@{#293720}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -13 lines) Patch
M ui/base/cursor/ozone/bitmap_cursor_factory_ozone.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M ui/events/ozone/evdev/event_factory_evdev.cc View 1 chunk +1 line, -1 line 0 comments Download
M ui/ozone/gpu/gpu_memory_buffer_factory_ozone_native_buffer.cc View 1 chunk +1 line, -1 line 0 comments Download
M ui/ozone/platform/dri/dri_cursor.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M ui/ozone/platform/dri/dri_surface.cc View 1 chunk +1 line, -1 line 0 comments Download
M ui/ozone/platform/dri/hardware_display_controller.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M ui/ozone/platform/dri/screen_manager.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 10 (2 generated)
lcwu1
6 years, 3 months ago (2014-09-07 00:03:26 UTC) #2
spang
lgtm
6 years, 3 months ago (2014-09-08 14:23:23 UTC) #3
spang
thanks! landing this on your behalf to fix our build.
6 years, 3 months ago (2014-09-08 14:25:09 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/lcwu@chromium.org/552553002/1
6 years, 3 months ago (2014-09-08 14:27:00 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1) as 072a664cb5cdcc9ba27db927fdd3a88ea6c0dce6
6 years, 3 months ago (2014-09-08 14:28:36 UTC) #7
spang
FYI You can land urgent build fixes like this one TBR (to be reviewed).
6 years, 3 months ago (2014-09-08 14:31:27 UTC) #8
lcwu1
On 2014/09/08 14:31:27, spang wrote: > FYI > > You can land urgent build fixes ...
6 years, 3 months ago (2014-09-08 15:10:54 UTC) #9
commit-bot: I haz the power
6 years, 3 months ago (2014-09-10 03:46:08 UTC) #10
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/8a59821dfe4696acf6269891b0ef0bfb7ad33a3f
Cr-Commit-Position: refs/heads/master@{#293720}

Powered by Google App Engine
This is Rietveld 408576698