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

Issue 10827047: Removing instances of profile_->GetBookmarkModel() as part of converting BookmarkModel to a PKS. (Closed)

Created:
8 years, 4 months ago by rpetterson
Modified:
8 years, 4 months ago
Reviewers:
CC:
chromium-reviews, tfarina
Visibility:
Public.

Description

Removing instances of profile_->GetBookmarkModel() as part of converting BookmarkModel to a PKS. BUG=112525 TEST=unittests, manual TBR=sky@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148744

Patch Set 1 #

Patch Set 2 : #

Messages

Total messages: 5 (0 generated)
rpetterson
8 years, 4 months ago (2012-07-26 18:02:21 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rlp@chromium.org/10827047/8002
8 years, 4 months ago (2012-07-27 08:58:39 UTC) #2
commit-bot: I haz the power
Change committed as 148744
8 years, 4 months ago (2012-07-27 11:39:03 UTC) #3
tfarina
On Fri, Jul 27, 2012 at 8:39 AM, <commit-bot@chromium.org> wrote: > Change committed as 148744 ...
8 years, 4 months ago (2012-07-27 12:35:30 UTC) #4
rpetterson
8 years, 4 months ago (2012-07-27 17:35:22 UTC) #5
Thanks!

On Fri, Jul 27, 2012 at 5:35 AM, Thiago Farina <tfarina@chromium.org> wrote:

> On Fri, Jul 27, 2012 at 8:39 AM,  <commit-bot@chromium.org> wrote:
> > Change committed as 148744
> >
> Even though you putted TBR=sky@chromium.org, Scott was not copied to
> this CL nor he was added to the reviewers list.
>
> Copying him now...
>
> > https://chromiumcodereview.appspot.com/10827047/
>
>
>
> --
> Thiago
>

Powered by Google App Engine
This is Rietveld 408576698