Comment #9 on issue 16068 by [email protected]: Persist bookmark IDs  
across browser restarts
http://code.google.com/p/chromium/issues/detail?id=16068

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=20565

------------------------------------------------------------------------
r20565 | [email protected] | 2009-07-13 16:27:13 -0700 (Mon, 13 Jul 2009)  
| 10 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/bookmarks/bookmark_codec.cc?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/bookmarks/bookmark_codec.h?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/bookmarks/bookmark_codec_unittest.cc?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/bookmarks/bookmark_drag_data.cc?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/bookmarks/bookmark_drag_data.h?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/bookmarks/bookmark_model.cc?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/bookmarks/bookmark_model.h?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/bookmarks/bookmark_model_unittest.cc?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/bookmarks/bookmark_storage.cc?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/bookmarks/bookmark_storage.h?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_bookmarks_module.cc?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_bookmarks_module.h?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_bookmarks_module_constants.cc?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_bookmarks_module_constants.h?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/bookmark_editor_gtk.cc?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/bookmark_manager_gtk.cc?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/bookmark_tree_model.cc?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/bookmark_tree_model.h?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/bookmark_editor_view.cc?r1=20565&r2=20564
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/bookmark_editor_view.h?r1=20565&r2=20564

Try the original CL "Always persist bookmark IDs" again with the fix to
Valgrind issue. The fix is in bookmark_storage.h - initialized the newly
added member ids_reassigned_ of LoadDetails class.

See http://codereview.chromium.org/149310 for the original CL.

TEST=NONE
BUG=16068

Review URL: http://codereview.chromium.org/155456
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to