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

Issue 2569333003: Rename BookmarkIndex to TitledUrlIndex and BookmarkMatch to TitledUrlMatch (Closed)

Created:
4 years ago by mattreynolds
Modified:
4 years ago
Reviewers:
Mark P, sky
CC:
chromium-reviews, tfarina
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Rename BookmarkIndex to TitledUrlIndex and BookmarkMatch to TitledUrlMatch These classes were refactored to decouple them from BookmarkModel. Now that the refactoring is complete, rename the classes and methods to remove references to bookmarks. BUG=630769 Committed: https://crrev.com/25e9a31458ee9aef59ac3baa4c884c638dcbbd0a Cr-Commit-Position: refs/heads/master@{#438640}

Patch Set 1 #

Total comments: 2

Patch Set 2 : don't update copyrights #

Patch Set 3 : fix profile_writer_unittest.cc #

Patch Set 4 : fix bookmark_bridge.cc #

Unified diffs Side-by-side diffs Delta from patch set Stats (+108 lines, -568 lines) Patch
M chrome/browser/android/bookmarks/bookmark_bridge.cc View 1 2 3 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/importer/profile_writer_unittest.cc View 1 2 2 chunks +3 lines, -3 lines 0 comments Download
M components/bookmarks/browser/BUILD.gn View 2 chunks +4 lines, -4 lines 0 comments Download
D components/bookmarks/browser/bookmark_index.h View 1 chunk +0 lines, -94 lines 0 comments Download
D components/bookmarks/browser/bookmark_index.cc View 1 chunk +0 lines, -260 lines 0 comments Download
M components/bookmarks/browser/bookmark_index_unittest.cc View 11 chunks +14 lines, -16 lines 0 comments Download
D components/bookmarks/browser/bookmark_match.h View 1 chunk +0 lines, -51 lines 0 comments Download
D components/bookmarks/browser/bookmark_match.cc View 1 chunk +0 lines, -50 lines 0 comments Download
M components/bookmarks/browser/bookmark_model.h View 3 chunks +5 lines, -5 lines 0 comments Download
M components/bookmarks/browser/bookmark_model.cc View 4 chunks +5 lines, -5 lines 0 comments Download
M components/bookmarks/browser/bookmark_storage.h View 4 chunks +5 lines, -5 lines 0 comments Download
M components/bookmarks/browser/bookmark_storage.cc View 2 chunks +2 lines, -2 lines 0 comments Download
A + components/bookmarks/browser/titled_url_index.h View 1 5 chunks +15 lines, -18 lines 0 comments Download
A + components/bookmarks/browser/titled_url_index.cc View 1 11 chunks +18 lines, -18 lines 0 comments Download
A + components/bookmarks/browser/titled_url_match.h View 1 3 chunks +7 lines, -7 lines 0 comments Download
A + components/bookmarks/browser/titled_url_match.cc View 1 2 chunks +6 lines, -6 lines 0 comments Download
M components/omnibox/browser/bookmark_provider.h View 2 chunks +3 lines, -3 lines 0 comments Download
M components/omnibox/browser/bookmark_provider.cc View 9 chunks +14 lines, -14 lines 0 comments Download
M components/omnibox/browser/bookmark_provider_unittest.cc View 3 chunks +4 lines, -4 lines 0 comments Download

Messages

Total messages: 23 (12 generated)
mattreynolds
mpearson@chromium.org: Please review changes in //components/omnibox sky@chromium.org: Please review changes in //components/bookmarks This CL should ...
4 years ago (2016-12-13 23:31:02 UTC) #2
Mark P
On 2016/12/13 23:31:02, mattreynolds wrote: > mailto:mpearson@chromium.org: Please review changes in //components/omnibox components/omnibox/... lgtm > ...
4 years ago (2016-12-13 23:37:42 UTC) #3
sky
LGTM https://codereview.chromium.org/2569333003/diff/1/components/bookmarks/browser/titled_url_index.h File components/bookmarks/browser/titled_url_index.h (right): https://codereview.chromium.org/2569333003/diff/1/components/bookmarks/browser/titled_url_index.h#newcode1 components/bookmarks/browser/titled_url_index.h:1: // Copyright 2016 The Chromium Authors. All rights ...
4 years ago (2016-12-13 23:57:26 UTC) #4
mattreynolds
Thanks Mark and Scott! https://codereview.chromium.org/2569333003/diff/1/components/bookmarks/browser/titled_url_index.h File components/bookmarks/browser/titled_url_index.h (right): https://codereview.chromium.org/2569333003/diff/1/components/bookmarks/browser/titled_url_index.h#newcode1 components/bookmarks/browser/titled_url_index.h:1: // Copyright 2016 The Chromium ...
4 years ago (2016-12-14 01:14:21 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2569333003/20001
4 years ago (2016-12-14 01:15:14 UTC) #8
commit-bot: I haz the power
Try jobs failed on following builders: linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_asan_rel_ng/builds/279143)
4 years ago (2016-12-14 01:37:12 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2569333003/40001
4 years ago (2016-12-14 02:01:28 UTC) #13
commit-bot: I haz the power
Try jobs failed on following builders: android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_compile_dbg/builds/180776)
4 years ago (2016-12-14 02:41:01 UTC) #15
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2569333003/60001
4 years ago (2016-12-14 20:39:27 UTC) #18
commit-bot: I haz the power
Committed patchset #4 (id:60001)
4 years ago (2016-12-14 21:52:49 UTC) #21
commit-bot: I haz the power
4 years ago (2016-12-14 21:56:06 UTC) #23
Message was sent while issue was closed.
Patchset 4 (id:??) landed as
https://crrev.com/25e9a31458ee9aef59ac3baa4c884c638dcbbd0a
Cr-Commit-Position: refs/heads/master@{#438640}

Powered by Google App Engine
This is Rietveld 408576698