From 5415a3354f0d0df7c48b6a973af96ede43bb29d4 Mon Sep 17 00:00:00 2001
From: Dima Voytenko <dvoytenko@google.com>
Date: Thu, 14 Jan 2016 13:14:19 -0800
Subject: [PATCH] Standard action to hide elements

---
 build-system/tasks/presubmit-checks.js   |  2 +
 examples/article-access.amp.html         |  3 +-
 src/amp-core-service.js                  |  2 +
 src/service/standard-actions-impl.js     | 62 ++++++++++++++++++++++++
 test/functional/test-standard-actions.js | 45 +++++++++++++++++
 5 files changed, 113 insertions(+), 1 deletion(-)
 create mode 100644 src/service/standard-actions-impl.js
 create mode 100644 test/functional/test-standard-actions.js

diff --git a/build-system/tasks/presubmit-checks.js b/build-system/tasks/presubmit-checks.js
index 2ab7c26b8225..520c9f26bb6c 100644
--- a/build-system/tasks/presubmit-checks.js
+++ b/build-system/tasks/presubmit-checks.js
@@ -53,6 +53,7 @@ var forbiddenTerms = {
     message: privateServiceFactory,
     whitelist: [
       'src/service/action-impl.js',
+      'src/service/standard-actions-impl.js',
       'src/amp-core-service.js',
     ],
   },
@@ -104,6 +105,7 @@ var forbiddenTerms = {
     whitelist: [
       'src/amp-core-service.js',
       'src/service/resources-impl.js',
+      'src/service/standard-actions-impl.js',
     ],
   },
   // Privacy sensitive
diff --git a/examples/article-access.amp.html b/examples/article-access.amp.html
index 9de38689b1e8..65beb37517a3 100644
--- a/examples/article-access.amp.html
+++ b/examples/article-access.amp.html
@@ -189,10 +189,11 @@ <h1 itemprop="headline">Lorem Ipsum</h1>
           </div>
         </div>
 
-        <section amp-access="views > 0" amp-access-hide class="meter-section">
+        <section id="meter-notif" amp-access="views > 0" amp-access-hide class="meter-section">
           <template amp-access-template type="amp-mustache">
             You are viewing article {{views}} of {{maxViews}}!
           </template>
+          <a on="tap:meter-notif.hide">[Close]</a>
         </section>
 
         <section amp-access="NOT access" amp-access-hide class="login-section">
diff --git a/src/amp-core-service.js b/src/amp-core-service.js
index 905b7dfeddc0..aabde59978a3 100644
--- a/src/amp-core-service.js
+++ b/src/amp-core-service.js
@@ -17,6 +17,7 @@
 import {installActionService} from './service/action-impl';
 import {installHistoryService} from './service/history-impl';
 import {installResourcesService} from './service/resources-impl';
+import {installStandardActions} from './service/standard-actions-impl';
 import {installViewerService} from './service/viewer-impl';
 import {installViewportService} from './service/viewport-impl';
 import {installVsyncService} from './service/vsync-impl';
@@ -34,4 +35,5 @@ export function installCoreServices(window) {
   installVsyncService(window);
   installActionService(window);
   installResourcesService(window);
+  installStandardActions(window);
 }
diff --git a/src/service/standard-actions-impl.js b/src/service/standard-actions-impl.js
new file mode 100644
index 000000000000..035a4491745f
--- /dev/null
+++ b/src/service/standard-actions-impl.js
@@ -0,0 +1,62 @@
+/**
+ * Copyright 2016 The AMP HTML Authors. All Rights Reserved.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS-IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+import {getService} from '../service';
+import {installActionService} from './action-impl';
+import {installResourcesService} from './resources-impl';
+
+
+/**
+ * This service contains implementations of some of the most typical actions,
+ * such as hiding DOM elements.
+ * @private Visible for testing.
+ */
+export class StandardActions {
+  /**
+   * @param {!Window} win
+   */
+  constructor(win) {
+    /** @const @private {!ActionService} */
+    this.actions_ = installActionService(win);
+
+    /** @const @private {!Resources} */
+    this.resources_ = installResourcesService(win);
+
+    this.actions_.addGlobalMethodHandler('hide', this.handleHide.bind(this));
+  }
+
+  /**
+   * Handles "hide" action. This is a very simple action where "display: none"
+   * is applied to the target element.
+   * @param {!ActionInvocation} invocation
+   */
+  handleHide(invocation) {
+    this.resources_.mutateElement(invocation.target, () => {
+      invocation.target.style.display = 'none';
+    });
+  }
+}
+
+
+/**
+ * @param {!Window} win
+ * @return {!ActionService}
+ */
+export function installStandardActions(win) {
+  return getService(win, 'standard-actions', () => {
+    return new StandardActions(win);
+  });
+};
diff --git a/test/functional/test-standard-actions.js b/test/functional/test-standard-actions.js
new file mode 100644
index 000000000000..4d78d6aae415
--- /dev/null
+++ b/test/functional/test-standard-actions.js
@@ -0,0 +1,45 @@
+/**
+ * Copyright 2016 The AMP HTML Authors. All Rights Reserved.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS-IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+import {StandardActions} from '../../src/service/standard-actions-impl';
+import * as sinon from 'sinon';
+
+describe('StandardActions', () => {
+
+  let sandbox;
+  let actions;
+  let mutateElementStub;
+
+  beforeEach(() => {
+    sandbox = sinon.sandbox.create();
+    actions = new StandardActions(window);
+    mutateElementStub = sandbox.stub(actions.resources_, 'mutateElement',
+        (unusedElement, mutator) => mutator());
+  });
+
+  afterEach(() => {
+    sandbox.restore();
+    sandbox = null;
+  });
+
+  it('should handle "hide" action', () => {
+    const element = document.createElement('div');
+    actions.handleHide({target: element});
+    expect(mutateElementStub.callCount).to.equal(1);
+    expect(mutateElementStub.firstCall.args[0]).to.equal(element);
+    expect(element.style.display).to.equal('none');
+  });
+});