|
Avoid unnacessarily conversion from index to BookmarkNode pointer.
BookmarkModel::Remove() should take BookmarkNode pointer.
This will few save conversion from BookmarkNode pointer to index for calling Remove and converting index to BookmarkNode pointer to call RemoveAndDeleteNode().
This way we will able to save call's like parent->GetIndexOf(node),and again conversion to Bookmark Pointer.
BUG= 330842
Committed: https://crrev.com/39312670405cf55ce9b4790817e11f3492ae3c05
Cr-Commit-Position: refs/heads/master@{#328135}
Total comments: 1
Total comments: 8
Total comments: 2
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+80 lines, -92 lines) |
Patch |
|
M |
chrome/browser/android/bookmarks/bookmarks_bridge.cc
|
View
|
1
|
1 chunk |
+3 lines, -5 lines |
0 comments
|
Download
|
|
M |
chrome/browser/android/provider/chrome_browser_provider.cc
|
View
|
1
|
1 chunk |
+2 lines, -4 lines |
0 comments
|
Download
|
|
M |
chrome/browser/extensions/api/bookmarks/bookmark_api_helpers.cc
|
View
|
1
|
1 chunk |
+1 line, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/history/android/bookmark_model_sql_handler.cc
|
View
|
1
|
1 chunk |
+1 line, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/sync/glue/bookmark_change_processor.cc
|
View
|
1
2
|
2 chunks |
+2 lines, -3 lines |
0 comments
|
Download
|
|
M |
chrome/browser/sync/glue/bookmark_model_associator.cc
|
View
|
1
2
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/sync/profile_sync_service_bookmark_unittest.cc
|
View
|
1
|
6 chunks |
+12 lines, -13 lines |
0 comments
|
Download
|
|
M |
chrome/browser/sync/test/integration/bookmarks_helper.cc
|
View
|
1
|
2 chunks |
+3 lines, -3 lines |
0 comments
|
Download
|
|
M |
chrome/browser/ui/bookmarks/bookmark_context_menu_controller.cc
|
View
|
1
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
chrome/browser/ui/bookmarks/bookmark_ui_utils_unittest.cc
|
View
|
1
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/ui/bookmarks/recently_used_folders_combo_model_unittest.cc
|
View
|
1
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
chrome/browser/ui/cocoa/applescript/bookmark_folder_applescript.mm
|
View
|
1
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller.mm
|
View
|
1
|
1 chunk |
+2 lines, -5 lines |
0 comments
|
Download
|
|
M |
chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller_unittest.mm
|
View
|
1
|
6 chunks |
+6 lines, -7 lines |
0 comments
|
Download
|
|
M |
chrome/browser/ui/cocoa/bookmarks/bookmark_bar_folder_controller_unittest.mm
|
View
|
1
|
4 chunks |
+4 lines, -5 lines |
0 comments
|
Download
|
|
M |
chrome/browser/ui/cocoa/bookmarks/bookmark_editor_base_controller_unittest.mm
|
View
|
1
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/ui/cocoa/bookmarks/bookmark_editor_controller_unittest.mm
|
View
|
1
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
chrome/browser/ui/cocoa/bookmarks/bookmark_menu_bridge_unittest.mm
|
View
|
1
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/ui/cocoa/bookmarks/bookmark_model_observer_for_cocoa_unittest.mm
|
View
|
1
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
chrome/browser/ui/views/bookmarks/bookmark_bar_view_unittest.cc
|
View
|
1
|
1 chunk |
+4 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc
|
View
|
1
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
components/bookmarks/browser/bookmark_expanded_state_tracker_unittest.cc
|
View
|
1
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
components/bookmarks/browser/bookmark_index_unittest.cc
|
View
|
1
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
components/bookmarks/browser/bookmark_model.h
|
View
|
1
2
|
1 chunk |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
components/bookmarks/browser/bookmark_model.cc
|
View
|
1
2
3
|
2 chunks |
+5 lines, -3 lines |
0 comments
|
Download
|
|
M |
components/bookmarks/browser/bookmark_model_unittest.cc
|
View
|
1
|
4 chunks |
+4 lines, -4 lines |
0 comments
|
Download
|
|
M |
components/bookmarks/browser/bookmark_utils.cc
|
View
|
1
2
|
3 chunks |
+3 lines, -4 lines |
0 comments
|
Download
|
|
M |
components/bookmarks/managed/managed_bookmarks_tracker.cc
|
View
|
1
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
components/enhanced_bookmarks/enhanced_bookmark_model_unittest.cc
|
View
|
1
|
3 chunks |
+4 lines, -6 lines |
0 comments
|
Download
|
|
M |
components/undo/bookmark_undo_service.cc
|
View
|
1
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
components/undo/bookmark_undo_service_test.cc
|
View
|
1
|
3 chunks |
+3 lines, -3 lines |
0 comments
|
Download
|
Total messages: 17 (3 generated)
|