Touching HTMLMediaElement.h or MediaPlayer.h causes a world rebuild.
Created attachment 253410 [details] Patch
Attachment 253410 [details] did not pass style-queue: ERROR: Source/WebCore/html/HTMLMediaElementEnums.h:47: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] ERROR: Source/WebCore/html/HTMLMediaElementEnums.h:48: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] Total errors found: 2 in 31 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 253410 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=253410&action=review > Source/WebCore/html/HTMLMediaElementEnums.h:45 > + enum DelayedActionType { > + LoadMediaResource = 1 << 0, > + ConfigureTextTracks = 1 << 1, > + TextTrackChangesNotification = 1 << 2, > + ConfigureTextTrackDisplay = 1 << 3, > + CheckPlaybackTargetCompatablity = 1 << 4, > + > + EveryDelayedAction = LoadMediaResource | ConfigureTextTracks | TextTrackChangesNotification | ConfigureTextTrackDisplay | CheckPlaybackTargetCompatablity, > + }; Does this need to be visible outside of HTMLMediaElement?
(In reply to comment #3) > Comment on attachment 253410 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=253410&action=review > > > Source/WebCore/html/HTMLMediaElementEnums.h:45 > > + enum DelayedActionType { > > + LoadMediaResource = 1 << 0, > > + ConfigureTextTracks = 1 << 1, > > + TextTrackChangesNotification = 1 << 2, > > + ConfigureTextTrackDisplay = 1 << 3, > > + CheckPlaybackTargetCompatablity = 1 << 4, > > + > > + EveryDelayedAction = LoadMediaResource | ConfigureTextTracks | TextTrackChangesNotification | ConfigureTextTrackDisplay | CheckPlaybackTargetCompatablity, > > + }; > > Does this need to be visible outside of HTMLMediaElement? It does to HTMLAudioElement, but it should go inside protected:, along with all the methods which take or return it.
Created attachment 253456 [details] Patch for landing
Attachment 253456 [details] did not pass style-queue: ERROR: Source/WebCore/html/HTMLMediaElementEnums.h:47: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] ERROR: Source/WebCore/html/HTMLMediaElementEnums.h:48: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] Total errors found: 2 in 31 files If any of these errors are false positives, please file a bug against check-webkit-style.
Committed r184670: <http://trac.webkit.org/changeset/184670>