[sync] Open tabs from other devices are shown after disabling "Open Tabs"

chrome/android/java/src/org/chromium/chrome/browser/ntp/ForeignSessionHelper.java

Index: chrome/android/java/src/org/chromium/chrome/browser/ntp/ForeignSessionHelper.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/ForeignSessionHelper.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/ForeignSessionHelper.java
index 9e687fbd665d9dc8c211b571d1df8dcdae20a374..6bb413a5aeed4e4800b4d98ab02ea746b9c4d44a 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/ForeignSessionHelper.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/ForeignSessionHelper.java
@@ -160,6 +160,9 @@ class ForeignSessionHelper {
      *         reason.
      */
     List<ForeignSession> getForeignSessions() {
+        if (!isTabSyncEnabled()) {

[Ted C, 2016/10/07 17:39:47]

in java if the statement and condition fit one line, then you can do that

here it would be easily readable as well, so I would opt for that.

with that said, you're being consistent with the lines a few down, so that is ok
as well

[Patrick Noland, 2016/10/07 18:41:15]

This is the result of "git cl format" and it's consistent with the below block,
so I'm going to leave it as is.

+            return null;
+        }
         List<ForeignSession> result = new ArrayList<ForeignSession>();
         boolean received = nativeGetForeignSessions(mNativeForeignSessionHelper, result);
         if (!received) {