Make item getter and nameditem getter return nullable in Plugin and PluginArray

third_party/WebKit/Source/modules/plugins/PluginArray.idl

Index: third_party/WebKit/Source/modules/plugins/PluginArray.idl
diff --git a/third_party/WebKit/Source/modules/plugins/PluginArray.idl b/third_party/WebKit/Source/modules/plugins/PluginArray.idl
index e107399b767fa2bb5125dd943e3d1d024315d5c2..5bcbbcfad0306f9ed29a37c77d44be5e6c5d1413 100644
--- a/third_party/WebKit/Source/modules/plugins/PluginArray.idl
+++ b/third_party/WebKit/Source/modules/plugins/PluginArray.idl
@@ -18,12 +18,14 @@
     Boston, MA 02110-1301, USA.
 */
 
+// https://html.spec.whatwg.org/multipage/webappapis.html#plugins-2
+
 [
     ImplementedAs=DOMPluginArray
 ] interface PluginArray {
     readonly attribute unsigned long length;
-    getter Plugin item([Default=Undefined] optional unsigned long index);
-    Plugin namedItem([Default=Undefined] optional DOMString name);
+    getter Plugin? item([Default=Undefined] optional unsigned long index);
+    Plugin? namedItem([Default=Undefined] optional DOMString name);
     [NotEnumerable, ImplementedAs=namedItem] getter Plugin ([Default=Undefined] optional DOMString name);

[foolip, 2016/12/14 19:19:44]

Oops, both there and in the other file we missed the third instance that should
be made nullable even if not in spec. Can you follow up and also add TODOs
saying something like "the namedItem method and anonymous named getter should be
merged"?

     void refresh([Default=Undefined] optional boolean reload);
 };