[java] Introduce common interface for all web storage types (#10528)

valfirst · diemol · web-flow · commit a22e15736dd4 · 2022-04-10T01:51:06.000+02:00
Co-authored-by: Diego Molina &lt;diemol@users.noreply.github.com&gt;
[skip ci]
diff --git a/java/src/org/openqa/selenium/html5/LocalStorage.java b/java/src/org/openqa/selenium/html5/LocalStorage.java
@@ -17,23 +17,9 @@
 
 package org.openqa.selenium.html5;
 
-import java.util.Set;
-
 /**
  * Represents the local storage for the site currently opened in the browser. Each site has its own
  * separate storage area.
  */
-public interface LocalStorage {
-
-  String getItem(String key);
-
-  Set<String> keySet();
-
-  void setItem(String key, String value);
-
-  String removeItem(String key);
-
-  void clear();
-
-  int size();
+public interface LocalStorage extends Storage {
 }
diff --git a/java/src/org/openqa/selenium/html5/SessionStorage.java b/java/src/org/openqa/selenium/html5/SessionStorage.java
@@ -17,25 +17,11 @@
 
 package org.openqa.selenium.html5;
 
-import java.util.Set;
-
 /**
  * Represents the session storage in the browser for the site currently opened in the browser. The
  * session storage areas is specific to the current top level browsing context. Each context has a
  * unique set of session storage, one for each origin. Sites can add data to the session storage and
  * it will be accessible to any page from the same site opened in that window.
  */
-public interface SessionStorage {
-
-  String getItem(String key);
-
-  Set<String> keySet();
-
-  void setItem(String key, String value);
-
-  String removeItem(String key);
-
-  void clear();
-
-  int size();
+public interface SessionStorage extends Storage {
 }
diff --git a/java/src/org/openqa/selenium/html5/Storage.java b/java/src/org/openqa/selenium/html5/Storage.java
@@ -0,0 +1,38 @@
+// Licensed to the Software Freedom Conservancy (SFC) under one
+// or more contributor license agreements.  See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership.  The SFC licenses this file
+// to you 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.
+
+package org.openqa.selenium.html5;
+
+import java.util.Set;
+
+/**
+ * Represents common operations available for all web storage types (session or local).
+ */
+public interface Storage {
+
+  String getItem(String key);
+
+  Set<String> keySet();
+
+  void setItem(String key, String value);
+
+  String removeItem(String key);
+
+  void clear();
+
+  int size();
+}
