Add MenuItemDecorator class

Add possibility to set 'destructive' action style.

Change-Id: I181d889a042196e507e145c48f5aabf2304daa0c

com.eclipsesource.tabris.test/src/com/eclipsesource/tabris/internal/DataWhitelistTest.java

@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2013, 2017 EclipseSource and others.
+ * Copyright (c) 2013, 2018 EclipseSource and others.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
@@ -135,6 +135,11 @@ public void testHasLocalClipboard() {
     assertContains( "localClipboard" );
   }
 
+  @Test
+  public void testHasActionStyle() {
+    assertContains( "actionStyle" );
+  }
+
   private void assertContains( String actualKey ) {
     WhiteListEntry[] keys = DataWhitelist.WhiteListEntry.values();
     boolean foundKey = false;

com.eclipsesource.tabris.test/src/com/eclipsesource/tabris/widgets/enhancement/MenuItemDecoratorTest.java

@@ -0,0 +1,47 @@
+/*******************************************************************************
+ * Copyright (c) 2018 EclipseSource and others.
+ * All rights reserved. This program and the accompanying materials
+ * are made available under the terms of the Eclipse Public License v1.0
+ * which accompanies this distribution, and is available at
+ * http://www.eclipse.org/legal/epl-v10.html
+ *
+ * Contributors:
+ *    EclipseSource - initial API and implementation
+ ******************************************************************************/
+package com.eclipsesource.tabris.widgets.enhancement;
+
+import static com.eclipsesource.tabris.internal.DataWhitelist.WhiteListEntry.ACTION_STYLE;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.verify;
+
+import org.eclipse.swt.widgets.MenuItem;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.runners.MockitoJUnitRunner;
+
+import com.eclipsesource.tabris.test.util.TabrisEnvironment;
+
+@RunWith( MockitoJUnitRunner.class )
+public class MenuItemDecoratorTest {
+
+  @Rule
+  public TabrisEnvironment environment = new TabrisEnvironment();
+  private MenuItemDecorator decorator;
+  private MenuItem menuItem;
+
+  @Before
+  public void setUp() {
+    menuItem = mock( MenuItem.class );
+    decorator = new MenuItemDecorator( menuItem );
+  }
+
+  @Test
+  public void testDestructiveStyle() {
+    decorator.useDestructiveStyle();
+
+    verify( menuItem ).setData( ACTION_STYLE.getKey(), "destructive" );
+  }
+
+}

com.eclipsesource.tabris.test/src/com/eclipsesource/tabris/widgets/enhancement/WidgetsTest.java

@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2012, 2017 EclipseSource and others.
+ * Copyright (c) 2012, 2018 EclipseSource and others.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
@@ -15,6 +15,7 @@
 import static org.mockito.Mockito.mock;
 
 import org.eclipse.swt.widgets.Composite;
+import org.eclipse.swt.widgets.MenuItem;
 import org.eclipse.swt.widgets.ProgressBar;
 import org.eclipse.swt.widgets.Shell;
 import org.eclipse.swt.widgets.TabFolder;
@@ -209,4 +210,24 @@ public void testOnProgressBarDoesNotCache() {
     assertNotSame( decorator1, decorator2 );
   }
 
+  @Test
+  public void testOnMenuItem() {
+    assertNotNull( Widgets.onMenuItem( mock( MenuItem.class ) ) );
+  }
+
+  @Test( expected = IllegalArgumentException.class )
+  public void testOnMenuItemWithNull() {
+    Widgets.onMenuItem( null );
+  }
+
+  @Test
+  public void testOnMenuItemDoesNotCache() {
+    MenuItem widget = mock( MenuItem.class );
+
+    MenuItemDecorator decorator1 = Widgets.onMenuItem( widget );
+    MenuItemDecorator decorator2 = Widgets.onMenuItem( widget );
+
+    assertNotSame( decorator1, decorator2 );
+  }
+
 }

com.eclipsesource.tabris/src/com/eclipsesource/tabris/internal/DataWhitelist.java

@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2013, 2017 EclipseSource and others.
+ * Copyright (c) 2013, 2018 EclipseSource and others.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
@@ -38,7 +38,8 @@ public enum WhiteListEntry {
     AUTO_CORRECT( "autoCorrect" ),
     REFRESH_HANDLER( "refreshHandler" ),
     REFRESH_COMPOSITE("refreshComposite"),
-    LOCAL_CLIPBOARD( "localClipboard" );
+    LOCAL_CLIPBOARD( "localClipboard" ),
+    ACTION_STYLE( "actionStyle" );
 
     private final String key;
 

com.eclipsesource.tabris/src/com/eclipsesource/tabris/widgets/enhancement/MenuItemDecorator.java

@@ -0,0 +1,42 @@
+/*******************************************************************************
+ * Copyright (c) 2018 EclipseSource and others.
+ * All rights reserved. This program and the accompanying materials
+ * are made available under the terms of the Eclipse Public License v1.0
+ * which accompanies this distribution, and is available at
+ * http://www.eclipse.org/legal/epl-v10.html
+ *
+ * Contributors:
+ *    EclipseSource - initial API and implementation
+ ******************************************************************************/
+package com.eclipsesource.tabris.widgets.enhancement;
+
+import static com.eclipsesource.tabris.internal.DataWhitelist.WhiteListEntry.ACTION_STYLE;
+
+import org.eclipse.swt.widgets.MenuItem;
+
+import com.eclipsesource.tabris.internal.WidgetsUtil;
+
+/**
+ * @since 3.5
+ */
+public class MenuItemDecorator {
+
+  private final MenuItem item;
+
+  MenuItemDecorator( MenuItem item ) {
+    this.item = item;
+  }
+
+  /**
+   * Enables the 'destructive' action style for this menu item (iOs only).
+   * A destructive action is shown in red.
+   *
+   * @since 3.5
+   */
+  public MenuItemDecorator useDestructiveStyle() {
+    WidgetsUtil.setData( item, ACTION_STYLE, "destructive" );
+    return this;
+  }
+
+}
+

com.eclipsesource.tabris/src/com/eclipsesource/tabris/widgets/enhancement/Widgets.java

@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2012, 2017 EclipseSource and others.
+ * Copyright (c) 2012, 2018 EclipseSource and others.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
@@ -13,6 +13,7 @@
 import static com.eclipsesource.tabris.internal.WidgetsUtil.checkComponent;
 
 import org.eclipse.swt.widgets.Composite;
+import org.eclipse.swt.widgets.MenuItem;
 import org.eclipse.swt.widgets.ProgressBar;
 import org.eclipse.swt.widgets.Shell;
 import org.eclipse.swt.widgets.TabFolder;
@@ -93,6 +94,14 @@ public static ProgressBarDecorator onProgressBar( ProgressBar progressBar ) {
     return new ProgressBarDecorator( progressBar );
   }
 
+  /**
+   * @since 3.5
+   */
+  public static MenuItemDecorator onMenuItem( MenuItem menuItem ) {
+    checkComponent( menuItem );
+    return new MenuItemDecorator( menuItem );
+  }
+
   private Widgets() {
     // prevent instantiation
   }