Verify MediaKeys parameter values consistently

Source/modules/encryptedmedia/MediaKeys.h

 /*
  * Copyright (C) 2013 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 56 matching lines @@
     void setMediaElement(HTMLMediaElement*);
 
     blink::WebContentDecryptionModule* contentDecryptionModule();
 
     void trace(Visitor*);
 
     // ContextLifecycleObserver
     virtual void contextDestroyed() OVERRIDE;
 
 protected:
+    static bool isSupportedKeySystem(const String& keySystem);
+    static bool isSupportedContentType(const String& keySystem, const String& contentType);

[ddorwin, 2014/02/27 23:28:27]

This one can actually be a member and doesn't need the first parameter.

Except we use it in static isTypeSupported(), so never mind...

+
     MediaKeys(ExecutionContext*, const String& keySystem, PassOwnPtr<ContentDecryptionModule>);
     void initializeNewSessionTimerFired(Timer<MediaKeys>*);
 
     HTMLMediaElement* m_mediaElement;
     const String m_keySystem;
     OwnPtr<ContentDecryptionModule> m_cdm;
 
     // FIXME: Check whether |initData| can be changed by JS. Maybe we should not pass it as a pointer.
     struct InitializeNewSessionData {
         InitializeNewSessionData(PassRefPtrWillBeRawPtr<MediaKeySession> session, const String& contentType, PassRefPtr<Uint8Array> initData)
             : session(session)
             , contentType(contentType)
             , initData(initData) { }
 
         void trace(Visitor*);
 
         RefPtrWillBeMember<MediaKeySession> session;
         String contentType;
         RefPtr<Uint8Array> initData;
     };
     Deque<InitializeNewSessionData> m_pendingInitializeNewSessionData;
     Timer<MediaKeys> m_initializeNewSessionTimer;
 
     WeakPtrFactory<MediaKeys> m_weakFactory;
 };
 
 }
 
 #endif // MediaKeys_h