33281 2010-01-06 17:18:46 -0800 We should reuse MediaControlTimeDisplayElement::formatTime function 2023-10-13 08:22:22 -0700 1 1 1 Unclassified WebKit Media 528+ (Nightly build) All All RESOLVED CONFIGURATION CHANGED P2 Normal --- 1 cmarrin cmarrin ahmad.saleem792 eric.carlson jer.noble oldest_to_newest 177884 0 cmarrin 2010-01-06 17:18:46 -0800 This function is currently duplicated in FullscreenVideoController on Windows. We should make the function a static method so it can be reused. 1984890 1 ahmad.saleem792 2023-10-13 08:22:22 -0700 This patch merged 'MediaControlTimeDisplayElement' into 'MediaControlElementTypes.cpp'. https://github.com/WebKit/WebKit/commit/f9e9a9581511ef57234ffbd25cd6a7532123b54e and this commit also deleted above file: https://github.com/WebKit/WebKit/commit/e3e80694948e94e81d10faf2c0da5fb213ece5e8 Plus no function with '::formatTime' exist as per quick search. I don't think this 2010 old bug would add much value and if needed any refactoring, it can be done in new bug. I am marking this as 'RESOLVED CONFIGURATION CHANGED' but will CC - Eric and Jer for any input.