[Webkit-unassigned] [Bug 25971] Some Video Elements Creeping Into Non-Video Builds

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri May 22 17:47:22 PDT 2009


https://bugs.webkit.org/show_bug.cgi?id=25971


aroben at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #30598|review?                     |review+
               Flag|                            |




------- Comment #2 from aroben at apple.com  2009-05-22 17:47 PDT -------
(From update of attachment 30598)
> +++ WebCore/rendering/RenderThemeWin.cpp	(working copy)
> @@ -28,7 +28,9 @@
>  #include "HTMLElement.h"
>  #include "HTMLSelectElement.h"
>  #include "Icon.h"
> +#if ENABLE(VIDEO)
>  #include "RenderMediaControls.h"
> +#endif
>  #include "RenderSlider.h"
>  #include "Settings.h"
>  #include "SoftLinking.h"

Please move conditional includes into their own paragraphs below unconditional
includes before landing.

r=me


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



More information about the webkit-unassigned mailing list