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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 1616073005: [CleanUp] Removed unused inclusions of 'base/profiler/scoped_tracker.h'. (Closed)

Created:
4 years, 11 months ago by Raveendra Karu
Modified:
4 years, 7 months ago
CC:
chromium-reviews, tfarina, cbentzel+watch_chromium.org, miu+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[CleanUp] Removed unused inclusions of 'base/profiler/scoped_tracker.h'. Removed unused inclusions of 'base/profiler/scoped_tracker.h' header file from 'chrome/browser'. BUG=580534 Committed: https://crrev.com/859d517422ca9c9e650521b737889bc1b984250f Cr-Commit-Position: refs/heads/master@{#390603}

Patch Set 1 #

Patch Set 2 : Just a rebase. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -7 lines) Patch
M chrome/browser/net/predictor.cc View 1 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/profiles/profile_impl_io_data.cc View 1 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/ui/cocoa/notifications/message_center_tray_bridge.mm View 1 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.cc View 1 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/ui/views/frame/browser_non_client_frame_view_mus.cc View 1 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/ui/views/frame/opaque_browser_frame_view.cc View 1 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/ui/views/tabs/tab.cc View 1 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 20 (10 generated)
Pritam Nikam
Hi Ravi, PTAL!
4 years, 11 months ago (2016-01-22 14:20:05 UTC) #2
mmenke
On 2016/01/22 14:20:05, Pritam Nikam wrote: > Hi Ravi, PTAL! This LGTM. You'll still need ...
4 years, 11 months ago (2016-01-26 16:56:20 UTC) #4
Raveendra Karu
Dear Scott, PTAL at the cleanUp changes.
4 years, 11 months ago (2016-01-27 06:20:20 UTC) #7
Peter Kasting
RS LGTM if this compiles. You may need to update this CL since it's been ...
4 years, 7 months ago (2016-04-29 02:21:19 UTC) #8
Raveendra Karu
On 2016/04/29 02:21:19, Peter Kasting wrote: > RS LGTM if this compiles. You may need ...
4 years, 7 months ago (2016-04-29 07:25:00 UTC) #9
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1616073005/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1616073005/20001
4 years, 7 months ago (2016-04-29 07:25:51 UTC) #11
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 7 months ago (2016-04-29 08:12:56 UTC) #13
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1616073005/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1616073005/20001
4 years, 7 months ago (2016-04-29 08:13:57 UTC) #16
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years, 7 months ago (2016-04-29 08:17:54 UTC) #18
commit-bot: I haz the power
4 years, 7 months ago (2016-04-30 17:24:39 UTC) #19
Message was sent while issue was closed. 
Patchset 2 (id:??) landed as
https://crrev.com/859d517422ca9c9e650521b737889bc1b984250f
Cr-Commit-Position: refs/heads/master@{#390603}

Powered by Google App Engine
This is Rietveld 408576698