BMM: Split initialization into two phases

getStrings is async and when it is done it sets the loadTimeData.
However, we depend on the loadTimeData to be present before the
call to getStrings returns. Have getStrings callback continue the
initialization.

BUG=312356
R=dbeam@chromium.org

Review URL: https://codereview.chromium.org/52803009

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@233077 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/resources/bookmark_manager/js/main.js

@@ -1217,7 +1217,11 @@ function initializeBookmarkManager() {
   if (!chrome.bookmarks)
     console.error('Bookmarks extension API is not available');
 
-  chrome.bookmarkManagerPrivate.getStrings(loadLocalizedStrings);
+  chrome.bookmarkManagerPrivate.getStrings(continueInitializeBookmarkManager);
+}
+
+function continueInitializeBookmarkManager(localizedStrings) {
+  loadLocalizedStrings(localizedStrings);
 
   bmm.treeLookup[searchTreeItem.bookmarkId] = searchTreeItem;