Add a preference for disabling vibration in notifications.

chrome/android/javatests/src/org/chromium/chrome/browser/notifications/NotificationPlatformBridgeTest.java

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser.notifications;
 
 import android.annotation.SuppressLint;
 import android.app.Notification;
 import android.content.Context;
 import android.content.res.Resources;
 import android.graphics.Bitmap;
 import android.os.Build;
 import android.test.suitebuilder.annotation.LargeTest;
 import android.test.suitebuilder.annotation.MediumTest;
 
+import org.chromium.base.ThreadUtils;
 import org.chromium.base.annotations.SuppressFBWarnings;
 import org.chromium.base.test.util.Feature;
+import org.chromium.chrome.browser.preferences.PrefServiceBridge;
 import org.chromium.chrome.browser.preferences.website.ContentSetting;
 import org.chromium.chrome.browser.util.UrlUtilities;
 import org.chromium.chrome.browser.widget.RoundedIconGenerator;
 import org.chromium.chrome.test.util.browser.notifications.MockNotificationManagerProxy.NotificationEntry;
 
 import java.util.List;
 
 /**
  * Instrumentation tests for the Notification Bridge.
  *
@@ skipping 45 matching lines @@
         }
 
         assertNotNull(notification.largeIcon);
 
         // Validate the notification's behavior.
         assertEquals(Notification.DEFAULT_ALL, notification.defaults);
         assertEquals(Notification.PRIORITY_DEFAULT, notification.priority);
     }
 
     /**
-     * Verifies that notifications created with the "silent" flag do not inherit system defaults
-     * in regards to their sound, vibration and light indicators.
+     * Verifies that the ONLY_ALERT_ONCE flag is not set when renotify is true.
      */
     @MediumTest
     @Feature({"Browser", "Notifications"})
     public void testNotificationRenotifyProperty() throws Exception {
         loadUrl(getTestServer().getURL(NOTIFICATION_TEST_PAGE));
         setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW);
 
         Notification notification =
                 showAndGetNotification("MyNotification", "{ tag: 'myTag', renotify: true }");
 
-        // Zero indicates that no defaults should be inherited from the system.
         assertEquals(0, notification.flags & Notification.FLAG_ONLY_ALERT_ONCE);
     }
 
     /**
      * Verifies that notifications created with the "silent" flag do not inherit system defaults
      * in regards to their sound, vibration and light indicators.
      */
     @MediumTest
     @Feature({"Browser", "Notifications"})
     public void testNotificationSilentProperty() throws Exception {
         loadUrl(getTestServer().getURL(NOTIFICATION_TEST_PAGE));
         setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW);
 
         Notification notification = showAndGetNotification("MyNotification", "{ silent: true }");
 
         // Zero indicates that no defaults should be inherited from the system.
         assertEquals(0, notification.defaults);
     }
 
     /**
+     * Verifies that when notification vibration is disabled in preferences and no custom pattern is
+     * specified, no vibration is requested from the framework.
+     */
+    @MediumTest
+    @Feature({"Browser", "Notifications"})
+    public void testNotificationVibratePreferenceDisabledDefault() throws Exception {

[gone, 2016/06/07 17:32:54]

This and the below test differ only by one line; does it make sense to combine
them and pass in the expected vibration string?

[Michael van Ouwerkerk, 2016/06/08 13:28:13]

Done.

+        loadUrl(getTestServer().getURL(NOTIFICATION_TEST_PAGE));
+        setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW);
+
+        // Disable notification vibration in preferences.
+        ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+            @Override
+            public void run() {
+                PrefServiceBridge.getInstance().setNotificationsVibrateEnabled(false);
+            }
+        });
+
+        Notification notification = showAndGetNotification("MyNotification", "{}");
+
+        // Vibration should not be in the defaults.
+        assertEquals(
+                Notification.DEFAULT_ALL & ~Notification.DEFAULT_VIBRATE, notification.defaults);
+
+        // There should be a custom no-op vibration pattern.
+        assertEquals(1, notification.vibrate.length);
+        assertEquals(0L, notification.vibrate[0]);
+    }
+
+    /**
+     * Verifies that when notification vibration is disabled in preferences and a custom pattern is
+     * specified, no vibration is requested from the framework.
+     */
+    @MediumTest
+    @Feature({"Browser", "Notifications"})
+    public void testNotificationVibratePreferenceDisabledCustomPattern() throws Exception {
+        loadUrl(getTestServer().getURL(NOTIFICATION_TEST_PAGE));
+        setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW);
+
+        // Disable notification vibration in preferences.
+        ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+            @Override
+            public void run() {
+                PrefServiceBridge.getInstance().setNotificationsVibrateEnabled(false);
+            }
+        });
+
+        Notification notification = showAndGetNotification("MyNotification", "{ vibrate: 42}");
+
+        // Vibration should not be in the defaults.
+        assertEquals(
+                Notification.DEFAULT_ALL & ~Notification.DEFAULT_VIBRATE, notification.defaults);
+
+        // There should be a custom no-op vibration pattern.
+        assertEquals(1, notification.vibrate.length);
+        assertEquals(0L, notification.vibrate[0]);
+    }
+
+    /**
+     * Verifies that by default the notification vibration preference is enabled, and a custom
+     * pattern is passed along.
+     */
+    @MediumTest
+    @Feature({"Browser", "Notifications"})
+    public void testNotificationVibrateCustomPattern() throws Exception {
+        loadUrl(getTestServer().getURL(NOTIFICATION_TEST_PAGE));
+        setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW);
+
+        // By default, vibration is enabled in notifications.
+        ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+            @Override
+            public void run() {
+                assertTrue(PrefServiceBridge.getInstance().isNotificationsVibrateEnabled());
+            }
+        });
+
+        Notification notification = showAndGetNotification("MyNotification", "{ vibrate: 42}");
+
+        // Vibration should not be in the defaults, a custom pattern was provided.
+        assertEquals(
+                Notification.DEFAULT_ALL & ~Notification.DEFAULT_VIBRATE, notification.defaults);
+
+        // The custom pattern should have been passed along.
+        assertEquals(2, notification.vibrate.length);
+        assertEquals(0L, notification.vibrate[0]);
+        assertEquals(42L, notification.vibrate[1]);
+    }
+
+    /**
      * Verifies that notifications which specify an icon will have that icon fetched, converted into
      * a Bitmap and included as the large icon in the notification.
      */
     @MediumTest
     @Feature({"Browser", "Notifications"})
     public void testShowNotificationWithIcon() throws Exception {
         loadUrl(getTestServer().getURL(NOTIFICATION_TEST_PAGE));
         setNotificationContentSettingForCurrentOrigin(ContentSetting.ALLOW);
 
         Notification notification = showAndGetNotification("MyNotification", "{icon: 'icon.png'}");
@@ skipping 195 matching lines @@
         assertEquals(1, notifications.size());
         assertEquals("Two",
                 notifications.get(0).notification.extras.getString(Notification.EXTRA_TITLE));
 
         // Close the last notification and verify that none remain.
         notifications.get(0).notification.contentIntent.send();
         waitForNotificationManagerMutation();
         assertTrue(getNotificationEntries().isEmpty());
     }
 }