Title: [171976] branches/safari-600.1.4-branch/Source/WebCore
- Revision
- 171976
- Author
- [email protected]
- Date
- 2014-08-03 00:37:11 -0700 (Sun, 03 Aug 2014)
Log Message
Merged r171944. <rdar://problem/17879156>
Modified Paths
Diff
Modified: branches/safari-600.1.4-branch/Source/WebCore/ChangeLog (171975 => 171976)
--- branches/safari-600.1.4-branch/Source/WebCore/ChangeLog 2014-08-03 07:34:06 UTC (rev 171975)
+++ branches/safari-600.1.4-branch/Source/WebCore/ChangeLog 2014-08-03 07:37:11 UTC (rev 171976)
@@ -1,3 +1,25 @@
+2014-08-03 Babak Shafiei <[email protected]>
+
+ Merge r171944.
+
+ 2014-07-31 Dean Jackson <[email protected]>
+
+ [Media iOS] Ensure there is a nice default fallback for missing wireless target names
+ https://bugs.webkit.org/show_bug.cgi?id=135488
+ <rdar://problem/17879156>
+
+ Reviewed by Eric Carlson.
+
+ We occasionally run into cases where this.host.externalDeviceDisplayName
+ is empty or null, creating a pretty ugly/confusing string in the
+ wireless playback status screen.
+
+ If this happens, we should default to using "Apple TV".
+
+ * Modules/mediacontrols/mediaControlsiOS.js:
+ (ControllerIOS.prototype.updateWirelessPlaybackStatus): Check if the externalDeviceDisplayName
+ is empty or null, and fall back to "Apple TV" if so.
+
2014-07-31 Matthew Hanson <[email protected]>
Merge r171259. <rdar://problem/17718984>
Modified: branches/safari-600.1.4-branch/Source/WebCore/Modules/mediacontrols/mediaControlsiOS.js (171975 => 171976)
--- branches/safari-600.1.4-branch/Source/WebCore/Modules/mediacontrols/mediaControlsiOS.js 2014-08-03 07:34:06 UTC (rev 171975)
+++ branches/safari-600.1.4-branch/Source/WebCore/Modules/mediacontrols/mediaControlsiOS.js 2014-08-03 07:37:11 UTC (rev 171976)
@@ -110,13 +110,16 @@
var deviceType = "";
var type = this.host.externalDeviceType;
if (type == "airplay") {
+ var externalDeviceDisplayName = this.host.externalDeviceDisplayName;
+ if (!externalDeviceDisplayName || externalDeviceDisplayName == "")
+ externalDeviceDisplayName = "Apple TV";
deviceType = this.UIString('##WIRELESS_PLAYBACK_DEVICE_TYPE##');
- deviceName = this.UIString('##WIRELESS_PLAYBACK_DEVICE_NAME##', '##DEVICE_NAME##', this.host.externalDeviceDisplayName);
+ deviceName = this.UIString('##WIRELESS_PLAYBACK_DEVICE_NAME##', '##DEVICE_NAME##', externalDeviceDisplayName);
} else if (type == "tvout") {
deviceType = this.UIString('##TVOUT_DEVICE_TYPE##');
deviceName = this.UIString('##TVOUT_DEVICE_NAME##');
}
-
+
backgroundImageSVG = backgroundImageSVG.replace('##DEVICE_TYPE##', deviceType);
backgroundImageSVG = backgroundImageSVG.replace('##DEVICE_NAME##', deviceName);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
