Should provide checks while loading bookmarks
Should provide checks while loading bookmarks

Instead of providing a separate check to validate bookmarks while loading, we could assume invalid bookmarks are invalid, hence we don't break the loading sequence. We keep looping since another bookmark might be valid.

If the JSON syntax is broken, it will exit gracefully because the parser can't parse the file. This patch is only good for people who are editing their bookmarks directly and missing a item in the bookmark node.

BUG=10709 (http://crbug.com/10709)

TEST=Deleted any line from the Bookmarks file within the bookmark children node. Loaded up Chromium and it loaded the error free bookmarks.

Review URL: http://codereview.chromium.org/118351

git-svn-id: de016e52bd170d2d4f2344f9bf92d50478b649e0@17960 0039d316-1c4b-4281-b951-d872f2087c98

Fireblend/chromium-crosswalk,fujunwei/chromium-crosswalk,jaruba/chromium.src,Pluto-tv/chromium-crosswalk,TheTypoMaster/chromium-crosswalk,zcbenz/cefode-chromium,Chilledheart/chromium,dednal/chromium.src,markYoungH/chromium.src,fujunwei/chromium-crosswalk,ChromiumWebApps/chromium,TheTypoMaster/chromium-crosswalk,ondra-novak/chromium.src,junmin-zhu/chromium-rivertrail,mohamed--abdel-maksoud/chromium.src,Fireblend/chromium-crosswalk,timopulkkinen/BubbleFish,timopulkkinen/BubbleFish,anirudhSK/chromium,nacl-webkit/chrome_deps,markYoungH/chromium.src,robclark/chromium,nacl-webkit/chrome_deps,Fireblend/chromium-crosswalk,M4sse/chromium.src,PeterWangIntel/chromium-crosswalk,krieger-od/nwjs_chromium.src,anirudhSK/chromium,keishi/chromium,dednal/chromium.src,rogerwang/chromium,hujiajie/pa-chromium,Chilledheart/chromium,hujiajie/pa-chromium,Just-D/chromium-1,hujiajie/pa-chromium,zcbenz/cefode-chromium,ondra-novak/chromium.src,hgl888/chromium-crosswalk,patrickm/chromium.src,PeterWangIntel/chromium-crosswalk,anirudhSK/chromium,Chilledheart/chromium,ChromiumWebApps/chromium,Just-D/chromium-1,Jonekee/chromium.src,Jonekee/chromium.src,hgl888/chromium-crosswalk,krieger-od/nwjs_chromium.src,ChromiumWebApps/chromium,markYoungH/chromium.src,pozdnyakov/chromium-crosswalk,anirudhSK/chromium,mogoweb/chromium-crosswalk,PeterWangIntel/chromium-crosswalk,hujiajie/pa-chromium,anirudhSK/chromium,ltilve/chromium,ltilve/chromium,ltilve/chromium,robclark/chromium,Pluto-tv/chromium-crosswalk,keishi/chromium,krieger-od/nwjs_chromium.src,rogerwang/chromium,patrickm/chromium.src,crosswalk-project/chromium-crosswalk-efl,Pluto-tv/chromium-crosswalk,ChromiumWebApps/chromium,anirudhSK/chromium,jaruba/chromium.src,mohamed--abdel-maksoud/chromium.src,TheTypoMaster/chromium-crosswalk,Fireblend/chromium-crosswalk,rogerwang/chromium,Pluto-tv/chromium-crosswalk,dushu1203/chromium.src,fujunwei/chromium-crosswalk,anirudhSK/chromium,nacl-webkit/chrome_deps,zcbenz/cefode-chromium,ChromiumWebApps/chromium,timopulkkinen/BubbleFish,mogoweb/chromium-crosswalk,M4sse/chromium.src,robclark/chromium,mogoweb/chromium-crosswalk,rogerwang/chromium,chuan9/chromium-crosswalk,patrickm/chromium.src,mohamed--abdel-maksoud/chromium.src,junmin-zhu/chromium-rivertrail,crosswalk-project/chromium-crosswalk-efl,mogoweb/chromium-crosswalk,ltilve/chromium,dednal/chromium.src,patrickm/chromium.src,junmin-zhu/chromium-rivertrail,hgl888/chromium-crosswalk,robclark/chromium,hgl888/chromium-crosswalk-efl,dushu1203/chromium.src,Jonekee/chromium.src,ChromiumWebApps/chromium,axinging/chromium-crosswalk,dednal/chromium.src,dednal/chromium.src,littlstar/chromium.src,jaruba/chromium.src,timopulkkinen/BubbleFish,fujunwei/chromium-crosswalk,dushu1203/chromium.src,rogerwang/chromium,Jonekee/chromium.src,ondra-novak/chromium.src,Pluto-tv/chromium-crosswalk,Jonekee/chromium.src,crosswalk-project/chromium-crosswalk-efl,hgl888/chromium-crosswalk,fujunwei/chromium-crosswalk,keishi/chromium,ondra-novak/chromium.src,junmin-zhu/chromium-rivertrail,markYoungH/chromium.src,ltilve/chromium,hujiajie/pa-chromium,PeterWangIntel/chromium-crosswalk,rogerwang/chromium,keishi/chromium,junmin-zhu/chromium-rivertrail,krieger-od/nwjs_chromium.src,PeterWangIntel/chromium-crosswalk,dednal/chromium.src,patrickm/chromium.src,jaruba/chromium.src,krieger-od/nwjs_chromium.src,zcbenz/cefode-chromium,axinging/chromium-crosswalk,rogerwang/chromium,zcbenz/cefode-chromium,hgl888/chromium-crosswalk,pozdnyakov/chromium-crosswalk,bright-sparks/chromium-spacewalk,nacl-webkit/chrome_deps,pozdnyakov/chromium-crosswalk,dednal/chromium.src,bright-sparks/chromium-spacewalk,TheTypoMaster/chromium-crosswalk,ChromiumWebApps/chromium,mohamed--abdel-maksoud/chromium.src,keishi/chromium,Chilledheart/chromium,pozdnyakov/chromium-crosswalk,jaruba/chromium.src,littlstar/chromium.src,nacl-webkit/chrome_deps,krieger-od/nwjs_chromium.src,keishi/chromium,krieger-od/nwjs_chromium.src,PeterWangIntel/chromium-crosswalk,axinging/chromium-crosswalk,keishi/chromium,littlstar/chromium.src,M4sse/chromium.src,Chilledheart/chromium,Pluto-tv/chromium-crosswalk,dushu1203/chromium.src,ondra-novak/chromium.src,Just-D/chromium-1,krieger-od/nwjs_chromium.src,Pluto-tv/chromium-crosswalk,Just-D/chromium-1,mohamed--abdel-maksoud/chromium.src,chuan9/chromium-crosswalk,fujunwei/chromium-crosswalk,timopulkkinen/BubbleFish,bright-sparks/chromium-spacewalk,ChromiumWebApps/chromium,hgl888/chromium-crosswalk-efl,Pluto-tv/chromium-crosswalk,markYoungH/chromium.src,Just-D/chromium-1,fujunwei/chromium-crosswalk,dednal/chromium.src,ltilve/chromium,ondra-novak/chromium.src,Jonekee/chromium.src,mogoweb/chromium-crosswalk,bright-sparks/chromium-spacewalk,robclark/chromium,hgl888/chromium-crosswalk,bright-sparks/chromium-spacewalk,axinging/chromium-crosswalk,dushu1203/chromium.src,pozdnyakov/chromium-crosswalk,zcbenz/cefode-chromium,markYoungH/chromium.src,crosswalk-project/chromium-crosswalk-efl,jaruba/chromium.src,hgl888/chromium-crosswalk,chuan9/chromium-crosswalk,ondra-novak/chromium.src,mogoweb/chromium-crosswalk,fujunwei/chromium-crosswalk,hujiajie/pa-chromium,TheTypoMaster/chromium-crosswalk,dednal/chromium.src,chuan9/chromium-crosswalk,mogoweb/chromium-crosswalk,nacl-webkit/chrome_deps,pozdnyakov/chromium-crosswalk,anirudhSK/chromium,Jonekee/chromium.src,nacl-webkit/chrome_deps,M4sse/chromium.src,Fireblend/chromium-crosswalk,axinging/chromium-crosswalk,nacl-webkit/chrome_deps,jaruba/chromium.src,dushu1203/chromium.src,Jonekee/chromium.src,robclark/chromium,TheTypoMaster/chromium-crosswalk,junmin-zhu/chromium-rivertrail,jaruba/chromium.src,dednal/chromium.src,Just-D/chromium-1,hgl888/chromium-crosswalk,dushu1203/chromium.src,Jonekee/chromium.src,dushu1203/chromium.src,Chilledheart/chromium,axinging/chromium-crosswalk,M4sse/chromium.src,chuan9/chromium-crosswalk,Just-D/chromium-1,M4sse/chromium.src,nacl-webkit/chrome_deps,pozdnyakov/chromium-crosswalk,hgl888/chromium-crosswalk,crosswalk-project/chromium-crosswalk-efl,chuan9/chromium-crosswalk,hgl888/chromium-crosswalk-efl,hujiajie/pa-chromium,patrickm/chromium.src,zcbenz/cefode-chromium,keishi/chromium,dushu1203/chromium.src,bright-sparks/chromium-spacewalk,crosswalk-project/chromium-crosswalk-efl,hujiajie/pa-chromium,timopulkkinen/BubbleFish,nacl-webkit/chrome_deps,PeterWangIntel/chromium-crosswalk,hgl888/chromium-crosswalk-efl,M4sse/chromium.src,Jonekee/chromium.src,Just-D/chromium-1,PeterWangIntel/chromium-crosswalk,ltilve/chromium,ChromiumWebApps/chromium,krieger-od/nwjs_chromium.src,robclark/chromium,Chilledheart/chromium,krieger-od/nwjs_chromium.src,hgl888/chromium-crosswalk-efl,TheTypoMaster/chromium-crosswalk,junmin-zhu/chromium-rivertrail,Pluto-tv/chromium-crosswalk,dednal/chromium.src,junmin-zhu/chromium-rivertrail,fujunwei/chromium-crosswalk,axinging/chromium-crosswalk,Fireblend/chromium-crosswalk,markYoungH/chromium.src,zcbenz/cefode-chromium,littlstar/chromium.src,mogoweb/chromium-crosswalk,Just-D/chromium-1,timopulkkinen/BubbleFish,rogerwang/chromium,keishi/chromium,anirudhSK/chromium,ltilve/chromium,keishi/chromium,pozdnyakov/chromium-crosswalk,Fireblend/chromium-crosswalk,markYoungH/chromium.src,bright-sparks/chromium-spacewalk,hujiajie/pa-chromium,littlstar/chromium.src,Fireblend/chromium-crosswalk,Chilledheart/chromium,timopulkkinen/BubbleFish,Fireblend/chromium-crosswalk,PeterWangIntel/chromium-crosswalk,rogerwang/chromium,bright-sparks/chromium-spacewalk,bright-sparks/chromium-spacewalk,chuan9/chromium-crosswalk,hgl888/chromium-crosswalk-efl,M4sse/chromium.src,patrickm/chromium.src,robclark/chromium,hgl888/chromium-crosswalk-efl,axinging/chromium-crosswalk,M4sse/chromium.src,ChromiumWebApps/chromium,zcbenz/cefode-chromium,patrickm/chromium.src,anirudhSK/chromium,markYoungH/chromium.src,mohamed--abdel-maksoud/chromium.src,dushu1203/chromium.src,ondra-novak/chromium.src,littlstar/chromium.src,TheTypoMaster/chromium-crosswalk,axinging/chromium-crosswalk,pozdnyakov/chromium-crosswalk,robclark/chromium,nacl-webkit/chrome_deps,zcbenz/cefode-chromium,crosswalk-project/chromium-crosswalk-efl,anirudhSK/chromium,zcbenz/cefode-chromium,chuan9/chromium-crosswalk,ltilve/chromium,crosswalk-project/chromium-crosswalk-efl,ChromiumWebApps/chromium,timopulkkinen/BubbleFish,timopulkkinen/BubbleFish,markYoungH/chromium.src,mohamed--abdel-maksoud/chromium.src,jaruba/chromium.src,robclark/chromium,anirudhSK/chromium,M4sse/chromium.src,junmin-zhu/chromium-rivertrail,TheTypoMaster/chromium-crosswalk,chuan9/chromium-crosswalk,rogerwang/chromium,mohamed--abdel-maksoud/chromium.src,junmin-zhu/chromium-rivertrail,ondra-novak/chromium.src,mogoweb/chromium-crosswalk,Chilledheart/chromium,crosswalk-project/chromium-crosswalk-efl,Jonekee/chromium.src,krieger-od/nwjs_chromium.src,keishi/chromium,markYoungH/chromium.src,M4sse/chromium.src,mohamed--abdel-maksoud/chromium.src,pozdnyakov/chromium-crosswalk,pozdnyakov/chromium-crosswalk,dushu1203/chromium.src,mogoweb/chromium-crosswalk,jaruba/chromium.src,hujiajie/pa-chromium,mohamed--abdel-maksoud/chromium.src,hgl888/chromium-crosswalk-efl,ChromiumWebApps/chromium,patrickm/chromium.src,littlstar/chromium.src,mohamed--abdel-maksoud/chromium.src,jaruba/chromium.src,hgl888/chromium-crosswalk-efl,axinging/chromium-crosswalk,timopulkkinen/BubbleFish,hujiajie/pa-chromium,junmin-zhu/chromium-rivertrail,hgl888/chromium-crosswalk-efl,axinging/chromium-crosswalk,littlstar/chromium.src