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

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

 /*
     Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
     Copyright (C) 2008 Apple Inc. All rights reserved.
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
     License as published by the Free Software Foundation; either
     version 2 of the License, or (at your option) any later version.
 
     This library is distributed in the hope that it will be useful,
     but WITHOUT ANY WARRANTY; without even the implied warranty of
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
     Library General Public License for more details.
 
     You should have received a copy of the GNU Library General Public License
     along with this library; see the file COPYING.LIB.  If not, write to
     the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
     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);
 };