Add support for java side bookmark model observers.

Add support for java side bookmark model observers.

Allows java bookmark UIs to update to changes to the underlying model.

BUG=247381
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221869

[Ted C, 2013-09-06 21:43:33]

PTAL

[David Trainor- moved to gerrit, 2013-09-06 21:55:25]

lgtm

[commit-bot: I haz the power, 2013-09-06 22:01:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tedchoc@chromium.org/23849003/5001

[newt (away), 2013-09-06 22:06:32]

lgtm

https://chromiumcodereview.appspot.com/23849003/diff/1/chrome/browser/android...
File chrome/browser/android/bookmarks_bridge.cc (right):

https://chromiumcodereview.appspot.com/23849003/diff/1/chrome/browser/android...
chrome/browser/android/bookmarks_bridge.cc:113: int64 parent_id = -1;
maybe:

    int64 parent_id = parent ? parent->id() : -1;

https://chromiumcodereview.appspot.com/23849003/diff/1/chrome/browser/android...
chrome/browser/android/bookmarks_bridge.cc:207:
Java_BookmarksBridge_bookmarkNodeAdded(
indent

https://chromiumcodereview.appspot.com/23849003/diff/1/chrome/browser/android...
chrome/browser/android/bookmarks_bridge.cc:236:
Java_BookmarksBridge_bookmarkNodeChanged(
indent

https://chromiumcodereview.appspot.com/23849003/diff/1/chrome/browser/android...
chrome/browser/android/bookmarks_bridge.cc:249: env,
indent

[Ted C, 2013-09-06 22:14:27]

https://chromiumcodereview.appspot.com/23849003/diff/1/chrome/browser/android...
File chrome/browser/android/bookmarks_bridge.cc (right):

https://chromiumcodereview.appspot.com/23849003/diff/1/chrome/browser/android...
chrome/browser/android/bookmarks_bridge.cc:113: int64 parent_id = -1;
On 2013/09/06 22:06:33, newt wrote:
> maybe:
> 
>     int64 parent_id = parent ? parent->id() : -1;

Done.

https://chromiumcodereview.appspot.com/23849003/diff/1/chrome/browser/android...
chrome/browser/android/bookmarks_bridge.cc:207:
Java_BookmarksBridge_bookmarkNodeAdded(
On 2013/09/06 22:06:33, newt wrote:
> indent

Done.

https://chromiumcodereview.appspot.com/23849003/diff/1/chrome/browser/android...
chrome/browser/android/bookmarks_bridge.cc:236:
Java_BookmarksBridge_bookmarkNodeChanged(
On 2013/09/06 22:06:33, newt wrote:
> indent

Done.

https://chromiumcodereview.appspot.com/23849003/diff/1/chrome/browser/android...
chrome/browser/android/bookmarks_bridge.cc:249: env,
On 2013/09/06 22:06:33, newt wrote:
> indent

Done.

[commit-bot: I haz the power, 2013-09-06 22:18:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tedchoc@chromium.org/23849003/23001

[commit-bot: I haz the power, 2013-09-07 03:05:58]

Change committed as 221869