Changeset 201585 in webkit

Timestamp:

Jun 1, 2016, 8:19:54 PM (10 years ago)

Author:

Chris Fleizach

Message:

LayoutTest accessibility/ios-simulator/attributed-string-for-range.html failing on ios-simulator debug
​https://bugs.webkit.org/show_bug.cgi?id=158279

Reviewed by Alexey Proskuryakov.

Just use the same description for debug and release. We're not getting any benefit from having two.

• accessibility/ios/WebAccessibilityObjectWrapperIOS.mm:

(-[WebAccessibilityObjectWrapper description]):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• accessibility/ios/WebAccessibilityObjectWrapperIOS.mm (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2016-06-01  Chris Fleizach  <cfleizach@apple.com>
+
+        LayoutTest accessibility/ios-simulator/attributed-string-for-range.html failing on ios-simulator debug
+        https://bugs.webkit.org/show_bug.cgi?id=158279
+
+        Reviewed by Alexey Proskuryakov.
+
+        Just use the same description for debug and release. We're not getting any benefit from having two.
+
+        * accessibility/ios/WebAccessibilityObjectWrapperIOS.mm:
+        (-[WebAccessibilityObjectWrapper description]):
+
 2016-06-01  Chris Dumez  <cdumez@apple.com>
 

trunk/Source/WebCore/accessibility/ios/WebAccessibilityObjectWrapperIOS.mm

@@ -2867 +2867 @@
 - (NSString *)description
 {
-#ifndef NDEBUG
-    CGRect frame = [self accessibilityFrame];
-    return [NSString stringWithFormat:@"Role: (%d) - Text: %@: Value: %@ -- Frame: %f %f %f %f", m_object ? m_object->roleValue() : 0, [self accessibilityLabel], [self accessibilityValue], frame.origin.x, frame.origin.y, frame.size.width, frame.size.height];
-#else
     return [NSString stringWithFormat:@"%@: %@", [self class], [self accessibilityLabel]];
-#endif
 }