Make browsing mode bottom toolbar have initial visibility of true

The model's visibility property starts off as false even though the
browsing mode bottom toolbar is visible initially. Also looks like we
had a bug in the view binder where visibility did the opposite of
what we wanted it to. Not sure how anything worked without this fix...

Bug: 905551
Change-Id: I6737aa480563f5c19d3938486b575dbd5841a184
Reviewed-on: https://chromium-review.googlesource.com/c/1341143Reviewed-by: Matthew Jones <mdjones@chromium.org>
Commit-Queue: Pedro Amaral <amaralp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#609533}

chrome/android/java/src/org/chromium/chrome/browser/toolbar/bottom/BrowsingModeBottomToolbarModel.java

@@ -53,5 +53,6 @@ public class BrowsingModeBottomToolbarModel extends PropertyModel {
         super(Y_OFFSET, ANDROID_VIEW_VISIBLE, COMPOSITED_VIEW_VISIBLE, LAYOUT_MANAGER,
                 TOOLBAR_SWIPE_LAYOUT, RESOURCE_MANAGER, TOOLBAR_SWIPE_HANDLER, IS_VISIBLE,
                 PRIMARY_COLOR);
+        set(IS_VISIBLE, true);
     }
 }

chrome/android/java/src/org/chromium/chrome/browser/toolbar/bottom/BrowsingModeBottomToolbarViewBinder.java

@@ -84,7 +84,7 @@ public class BrowsingModeBottomToolbarViewBinder
                     .setBackgroundColor(model.get(BrowsingModeBottomToolbarModel.PRIMARY_COLOR));
         } else if (BrowsingModeBottomToolbarModel.IS_VISIBLE == propertyKey) {
             final boolean isVisible = model.get(BrowsingModeBottomToolbarModel.IS_VISIBLE);
-            view.toolbarRoot.setVisibility(isVisible ? View.GONE : View.VISIBLE);
+            view.toolbarRoot.setVisibility(isVisible ? View.VISIBLE : View.GONE);
         } else {
             assert false : "Unhandled property detected in BrowsingModeBottomToolbarViewBinder!";
         }