Cleanup webaudio related header inclusions in HTMLMediaElement

Source/core/html/HTMLMediaElement.h

 /*
  * Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012, 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 14 matching lines @@
 
 #ifndef HTMLMediaElement_h
 #define HTMLMediaElement_h
 
 #include "core/CoreExport.h"
 #include "core/dom/ActiveDOMObject.h"
 #include "core/events/GenericEventQueue.h"
 #include "core/html/HTMLElement.h"
 #include "core/html/track/TextTrack.h"
 #include "platform/Supplementable.h"
+#include "public/platform/WebMediaPlayerClient.h"
+#include "public/platform/WebMimeRegistry.h"
+
+#if ENABLE(WEB_AUDIO)
 #include "platform/audio/AudioSourceProvider.h"
 #include "public/platform/WebAudioSourceProviderClient.h"
-#include "public/platform/WebMediaPlayerClient.h"
-#include "public/platform/WebMimeRegistry.h"
+#endif
 
 namespace blink {
 class WebInbandTextTrack;
 class WebLayer;
 }
 
 namespace blink {
 
 #if ENABLE(WEB_AUDIO)
-class AudioSourceProvider;
 class AudioSourceProviderClient;
+class WebAudioSourceProvider;

[philipj_slow, 2015/07/31 07:33:40]

This wasn't here before, why is it needed now?

[Srirama, 2015/07/31 07:38:49]

It is indirectly included previously(WebMediaPlayerClient->WebMediaPlayer).
I just remember one of the review comment from David in my previous patch that,
IWYU(Include What You Use). So i thought it is better to include directly.

[philipj_slow, 2015/07/31 08:46:31]

This isn't an include, but yeah, it used in this header without an include, so
good!

 #endif
 class AudioTrackList;
 class ContentType;
 class CueTimeline;
 class Event;
 class ExceptionState;
 class HTMLSourceElement;
 class HTMLTrackElement;
 class KURL;
 class MediaController;
@@ skipping 565 matching lines @@
 inline bool isHTMLMediaElement(const HTMLElement& element)
 {
     return isHTMLAudioElement(element) || isHTMLVideoElement(element);
 }
 
 DEFINE_HTMLELEMENT_TYPE_CASTS_WITH_FUNCTION(HTMLMediaElement);
 
 } // namespace blink
 
 #endif // HTMLMediaElement_h