From 37f8cf208fb86f91f6984c0ef0b7c8817729e55c Mon Sep 17 00:00:00 2001
From: Jon Newmuis <newmuis@users.noreply.github.com>
Date: Thu, 20 Dec 2018 17:50:21 -0500
Subject: [PATCH] Hide the share pill in visual tests, and re-enable tests that
 were flaky because of this (#20022)

---
 .../amp-story/amp-story-bookend.html          | 10 ++++++
 .../amp-story/amp-story-bookend.rtl.html      | 10 ++++++
 .../amp-story/amp-story-consent.html          | 10 ++++++
 .../amp-story/amp-story-consent.rtl.html      | 10 ++++++
 .../amp-story/amp-story-cta-layer.html        | 10 ++++++
 .../amp-story-grid-layer-template-fill.html   | 10 ++++++
 ...-story-grid-layer-template-horizontal.html | 10 ++++++
 .../amp-story-grid-layer-template-thirds.html | 10 ++++++
 ...mp-story-grid-layer-template-vertical.html | 10 ++++++
 .../amp-story-pagination-buttons.html         |  9 +++++
 .../amp-story/amp-story-sidebar.html          |  9 +++++
 .../amp-story/amp-story-tooltip.html          | 10 ++++++
 examples/visual-tests/amp-story/basic.html    | 10 ++++++
 .../visual-tests/amp-story/basic.rtl.html     | 10 ++++++
 .../visual-tests/amp-story/embed-mode-1.html  | 10 ++++++
 .../visual-tests/amp-story/embed-mode-2.html  | 10 ++++++
 .../visual-tests/amp-story/info-dialog.html   | 10 ++++++
 .../amp-story/info-dialog.rtl.html            | 10 ++++++
 .../visual-tests/amp-story/share-menu.html    | 10 ++++++
 .../amp-story/share-menu.rtl.html             | 10 ++++++
 test/visual-diff/visual-tests                 | 34 +------------------
 21 files changed, 199 insertions(+), 33 deletions(-)

diff --git a/examples/visual-tests/amp-story/amp-story-bookend.html b/examples/visual-tests/amp-story/amp-story-bookend.html
index f62ae1757156e..ec43d4c7f7900 100644
--- a/examples/visual-tests/amp-story/amp-story-bookend.html
+++ b/examples/visual-tests/amp-story/amp-story-bookend.html
@@ -23,6 +23,16 @@
         display: none;
       }
 
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+      .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
+
       h1.hello-world {
         color: white;
         text-align: center;
diff --git a/examples/visual-tests/amp-story/amp-story-bookend.rtl.html b/examples/visual-tests/amp-story/amp-story-bookend.rtl.html
index 29bbb2060d547..3074aed9c4e59 100644
--- a/examples/visual-tests/amp-story/amp-story-bookend.rtl.html
+++ b/examples/visual-tests/amp-story/amp-story-bookend.rtl.html
@@ -23,6 +23,16 @@
         display: none;
       }
 
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
+
       h1.hello-world {
         color: white;
         text-align: center;
diff --git a/examples/visual-tests/amp-story/amp-story-consent.html b/examples/visual-tests/amp-story/amp-story-consent.html
index 4d887095f54f1..3fa30d3815a9d 100644
--- a/examples/visual-tests/amp-story/amp-story-consent.html
+++ b/examples/visual-tests/amp-story/amp-story-consent.html
@@ -25,6 +25,16 @@
         display: none;
       }
 
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
+
       h1#hello-world {
         color: white;
         text-align: center;
diff --git a/examples/visual-tests/amp-story/amp-story-consent.rtl.html b/examples/visual-tests/amp-story/amp-story-consent.rtl.html
index a5961afdd6f7b..d9b54dcd87d50 100644
--- a/examples/visual-tests/amp-story/amp-story-consent.rtl.html
+++ b/examples/visual-tests/amp-story/amp-story-consent.rtl.html
@@ -25,6 +25,16 @@
         display: none;
       }
 
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
+
       h1#hello-world {
         color: white;
         text-align: center;
diff --git a/examples/visual-tests/amp-story/amp-story-cta-layer.html b/examples/visual-tests/amp-story/amp-story-cta-layer.html
index 9642bb73c25de..e929a9c793087 100644
--- a/examples/visual-tests/amp-story/amp-story-cta-layer.html
+++ b/examples/visual-tests/amp-story/amp-story-cta-layer.html
@@ -24,6 +24,16 @@
         left: 50%;
         transform: translate(-50%, -50%);
       }
+
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
     </style>
     <script>
       // TODO(#17108): Remove this when getMode().test is set.
diff --git a/examples/visual-tests/amp-story/amp-story-grid-layer-template-fill.html b/examples/visual-tests/amp-story/amp-story-grid-layer-template-fill.html
index 7a913646dd149..5a4cd4964d645 100644
--- a/examples/visual-tests/amp-story/amp-story-grid-layer-template-fill.html
+++ b/examples/visual-tests/amp-story/amp-story-grid-layer-template-fill.html
@@ -14,6 +14,16 @@
       amp-story-page {
         background: yellow;
       }
+
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
     </style>
     <script>
       // TODO(#17108): Remove this when getMode().test is set.
diff --git a/examples/visual-tests/amp-story/amp-story-grid-layer-template-horizontal.html b/examples/visual-tests/amp-story/amp-story-grid-layer-template-horizontal.html
index e657524d90f93..5099202480079 100644
--- a/examples/visual-tests/amp-story/amp-story-grid-layer-template-horizontal.html
+++ b/examples/visual-tests/amp-story/amp-story-grid-layer-template-horizontal.html
@@ -15,6 +15,16 @@
         background: white;
         font-family: sans-serif;
       }
+
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
     </style>
     <script>
       // TODO(#17108): Remove this when getMode().test is set.
diff --git a/examples/visual-tests/amp-story/amp-story-grid-layer-template-thirds.html b/examples/visual-tests/amp-story/amp-story-grid-layer-template-thirds.html
index abbf14cd4b5a1..e505cd2253788 100644
--- a/examples/visual-tests/amp-story/amp-story-grid-layer-template-thirds.html
+++ b/examples/visual-tests/amp-story/amp-story-grid-layer-template-thirds.html
@@ -15,6 +15,16 @@
         background: white;
         font-family: sans-serif;
       }
+
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
     </style>
     <script>
       // TODO(#17108): Remove this when getMode().test is set.
diff --git a/examples/visual-tests/amp-story/amp-story-grid-layer-template-vertical.html b/examples/visual-tests/amp-story/amp-story-grid-layer-template-vertical.html
index 146bff7d6a27a..593f913016bf2 100644
--- a/examples/visual-tests/amp-story/amp-story-grid-layer-template-vertical.html
+++ b/examples/visual-tests/amp-story/amp-story-grid-layer-template-vertical.html
@@ -15,6 +15,16 @@
         background: white;
         font-family: sans-serif;
       }
+
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
     </style>
     <script>
       // TODO(#17108): Remove this when getMode().test is set.
diff --git a/examples/visual-tests/amp-story/amp-story-pagination-buttons.html b/examples/visual-tests/amp-story/amp-story-pagination-buttons.html
index b78c25cf5e832..cf4ab1848e28c 100644
--- a/examples/visual-tests/amp-story/amp-story-pagination-buttons.html
+++ b/examples/visual-tests/amp-story/amp-story-pagination-buttons.html
@@ -22,6 +22,15 @@
       amp-story-page {
         background-color: white;
       }
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
     </style>
     <script type="application/ld+json">
       {
diff --git a/examples/visual-tests/amp-story/amp-story-sidebar.html b/examples/visual-tests/amp-story/amp-story-sidebar.html
index 39b7f9a55b9db..f9303a3e2430c 100644
--- a/examples/visual-tests/amp-story/amp-story-sidebar.html
+++ b/examples/visual-tests/amp-story/amp-story-sidebar.html
@@ -21,6 +21,15 @@
         /* Hide the spinner for loading pages, since this causes flakes */
         display: none;
       }
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
       h1.hello-world {
         color: white;
         text-shadow: 3px 3px rgba(0, 0, 0, 0.5);
diff --git a/examples/visual-tests/amp-story/amp-story-tooltip.html b/examples/visual-tests/amp-story/amp-story-tooltip.html
index 1b4dd93563b5f..1c8c9908e43fb 100644
--- a/examples/visual-tests/amp-story/amp-story-tooltip.html
+++ b/examples/visual-tests/amp-story/amp-story-tooltip.html
@@ -21,6 +21,16 @@
         display: none;
       }
 
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
+
       h1.hello-world {
         color: white;
         text-shadow: 3px 3px rgba(0, 0, 0, 0.5);
diff --git a/examples/visual-tests/amp-story/basic.html b/examples/visual-tests/amp-story/basic.html
index 779abd21343c9..38f058563e861 100644
--- a/examples/visual-tests/amp-story/basic.html
+++ b/examples/visual-tests/amp-story/basic.html
@@ -24,6 +24,16 @@
         display: none;
       }
 
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
+
       h1.hello-world {
         color: white;
         text-shadow: 3px 3px rgba(0, 0, 0, 0.5);
diff --git a/examples/visual-tests/amp-story/basic.rtl.html b/examples/visual-tests/amp-story/basic.rtl.html
index c3680fa93b9a7..5e0ae2a0c837a 100644
--- a/examples/visual-tests/amp-story/basic.rtl.html
+++ b/examples/visual-tests/amp-story/basic.rtl.html
@@ -24,6 +24,16 @@
         display: none;
       }
 
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
+
       h1.hello-world {
         color: white;
         text-shadow: 3px 3px rgba(0, 0, 0, 0.5);
diff --git a/examples/visual-tests/amp-story/embed-mode-1.html b/examples/visual-tests/amp-story/embed-mode-1.html
index 957d31a093cdf..d3f5702c370ac 100644
--- a/examples/visual-tests/amp-story/embed-mode-1.html
+++ b/examples/visual-tests/amp-story/embed-mode-1.html
@@ -23,6 +23,16 @@
         display: none;
       }
 
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
+
       h1#hello-world {
         color: white;
         text-align: center;
diff --git a/examples/visual-tests/amp-story/embed-mode-2.html b/examples/visual-tests/amp-story/embed-mode-2.html
index 01a9e5428f16e..8c8deb1e9d5ee 100644
--- a/examples/visual-tests/amp-story/embed-mode-2.html
+++ b/examples/visual-tests/amp-story/embed-mode-2.html
@@ -23,6 +23,16 @@
         display: none;
       }
 
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
+
       h1#hello-world {
         color: white;
         text-align: center;
diff --git a/examples/visual-tests/amp-story/info-dialog.html b/examples/visual-tests/amp-story/info-dialog.html
index bc28c6656f6fd..d00d132831132 100644
--- a/examples/visual-tests/amp-story/info-dialog.html
+++ b/examples/visual-tests/amp-story/info-dialog.html
@@ -23,6 +23,16 @@
         display: none;
       }
 
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
+
       h1#hello-world {
         color: white;
         text-align: center;
diff --git a/examples/visual-tests/amp-story/info-dialog.rtl.html b/examples/visual-tests/amp-story/info-dialog.rtl.html
index 04fea7a061f26..d59f210509985 100644
--- a/examples/visual-tests/amp-story/info-dialog.rtl.html
+++ b/examples/visual-tests/amp-story/info-dialog.rtl.html
@@ -23,6 +23,16 @@
         display: none;
       }
 
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
+
       h1#hello-world {
         color: white;
         text-align: center;
diff --git a/examples/visual-tests/amp-story/share-menu.html b/examples/visual-tests/amp-story/share-menu.html
index 738f328ae26ba..4fe8e9e6c80f0 100644
--- a/examples/visual-tests/amp-story/share-menu.html
+++ b/examples/visual-tests/amp-story/share-menu.html
@@ -23,6 +23,16 @@
         display: none;
       }
 
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
+
       h1#hello-world {
         color: white;
         text-align: center;
diff --git a/examples/visual-tests/amp-story/share-menu.rtl.html b/examples/visual-tests/amp-story/share-menu.rtl.html
index 0eb2a00b21977..961df425044dc 100644
--- a/examples/visual-tests/amp-story/share-menu.rtl.html
+++ b/examples/visual-tests/amp-story/share-menu.rtl.html
@@ -23,6 +23,16 @@
         display: none;
       }
 
+      /*
+       * The share pill text does not show in the same place consistently, and
+       * as such causes the visual tests to be flaky.  This hides the share pill
+       * with visibility: hidden, so as to keep other elements in the same place
+       * that they would otherwise be, while deflaking the tests.  See #19890.
+       */
+       .i-amphtml-story-share-pill {
+        visibility: hidden;
+      }
+
       h1#hello-world {
         color: white;
         text-align: center;
diff --git a/test/visual-diff/visual-tests b/test/visual-diff/visual-tests
index 739a6779965f5..d36862c6f1c1e 100644
--- a/test/visual-diff/visual-tests
+++ b/test/visual-diff/visual-tests
@@ -338,8 +338,6 @@
       ]
     },
     {
-      "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/946709
       "url": "examples/visual-tests/amp-story/basic.html",
       "name": "amp-story: basic (desktop)",
       "viewport": {"width": 1440, "height": 900},
@@ -352,8 +350,6 @@
       "loading_complete_delay_ms": 500
     },
     {
-      "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/946709
       "url": "examples/visual-tests/amp-story/amp-story-grid-layer-template-fill.html",
       "name": "amp-story: Grid layer (fill) (desktop)",
       "viewport": {"width": 1440, "height": 900},
@@ -364,8 +360,6 @@
       ]
     },
     {
-      "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/946709
       "url": "examples/visual-tests/amp-story/amp-story-grid-layer-template-vertical.html",
       "name": "amp-story: Grid layer (vertical) (desktop)",
       "viewport": {"width": 1440, "height": 900},
@@ -376,8 +370,6 @@
       ]
     },
     {
-      "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/946709
       "url": "examples/visual-tests/amp-story/amp-story-grid-layer-template-horizontal.html",
       "name": "amp-story: Grid layer (horizontal) (desktop)",
       "viewport": {"width": 1440, "height": 900},
@@ -388,8 +380,6 @@
       ]
     },
     {
-      "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/946709
       "url": "examples/visual-tests/amp-story/amp-story-grid-layer-template-thirds.html",
       "name": "amp-story: Grid layer (thirds) (desktop)",
       "viewport": {"width": 1440, "height": 900},
@@ -403,8 +393,6 @@
       ]
     },
     {
-      "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/946709
       "url": "examples/visual-tests/amp-story/amp-story-cta-layer.html",
       "name": "amp-story: CTA layer (desktop)",
       "viewport": {"width": 1440, "height": 900},
@@ -417,8 +405,6 @@
       "loading_complete_delay_ms": 500
     },
     {
-      "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/946709
       "url": "examples/visual-tests/amp-story/embed-mode-1.html",
       "name": "amp-story: embed mode 1 (desktop)",
       "viewport": {"width": 1440, "height": 900},
@@ -427,8 +413,6 @@
       ]
     },
     {
-      "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/946709
       "url": "examples/visual-tests/amp-story/embed-mode-2.html",
       "name": "amp-story: embed mode 2 (desktop)",
       "viewport": {"width": 1440, "height": 900},
@@ -438,8 +422,6 @@
       ]
     },
     {
-      "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/946709
       "url": "examples/visual-tests/amp-story/amp-story-bookend.html",
       "name": "amp-story: bookend (desktop)",
       "viewport": {"width": 1440, "height": 900},
@@ -450,8 +432,6 @@
       ]
     },
     {
-      "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/946709
       "url": "examples/visual-tests/amp-story/amp-story-consent.html",
       "name": "amp-story: consent (desktop)",
       "viewport": {"width": 1440, "height": 900},
@@ -509,8 +489,6 @@
       ]
     },
     {
-      "flaky": true,
-      // See https://travis-ci.org/ampproject/amphtml/jobs/456056607#L640
       "url": "examples/visual-tests/amp-story/amp-story-consent.rtl.html",
       "name": "amp-story: consent (rtl)",
       "viewport": {"width": 320, "height": 480},
@@ -520,8 +498,6 @@
       ]
     },
     {
-      "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/946709
       "url": "examples/visual-tests/amp-story/basic.rtl.html",
       "name": "amp-story: basic (desktop) (rtl)",
       "viewport": {"width": 1440, "height": 900},
@@ -534,8 +510,6 @@
       "loading_complete_delay_ms": 500
     },
     {
-      "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/946709
       "url": "examples/visual-tests/amp-story/amp-story-bookend.rtl.html",
       "name": "amp-story: bookend (desktop) (rtl)",
       "viewport": {"width": 1440, "height": 900},
@@ -547,7 +521,7 @@
     },
     {
       "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/946709
+      // See https://travis-ci.org/ampproject/amphtml/jobs/456056607#L640
       "url": "examples/visual-tests/amp-story/amp-story-consent.rtl.html",
       "name": "amp-story: consent (desktop) (rtl)",
       "viewport": {"width": 1440, "height": 900},
@@ -585,8 +559,6 @@
       "interactive_tests": "examples/visual-tests/amp-story/amp-story-tooltip.js"
     },
     {
-      "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/1179875
       "url": "examples/visual-tests/amp-story/amp-story-tooltip.html",
       "name": "amp-story: tooltip desktop",
       "viewport": {"width": 1440, "height": 900},
@@ -596,8 +568,6 @@
       "interactive_tests": "examples/visual-tests/amp-story/amp-story-tooltip-desktop.js"
     },
     {
-      "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/1260087
       "url": "examples/visual-tests/amp-story/amp-story-pagination-buttons.html",
       "name": "amp-story: pagination-buttons desktop",
       "viewport": {"width": 1440, "height": 900},
@@ -617,8 +587,6 @@
     "interactive_tests": "examples/visual-tests/amp-story/amp-story-sidebar.js"
     },
     {
-      "flaky": true,
-      // See https://percy.io/ampproject/amphtml/builds/1260087
       "url": "examples/visual-tests/amp-story/amp-story-pagination-buttons.html",
       "name": "amp-story: pagination-buttons desktop",
       "viewport": {"width": 1440, "height": 900},