Bug 176004

Summary: Fix incorrect enum in atBoundaryOfGranularity call
Product: WebKit Reporter: Megan Gardner <megan_gardner>
Component: New BugsAssignee: Megan Gardner <megan_gardner>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, enrica, thorton, webkit-bug-importer, wenson_hsieh
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch wenson_hsieh: review+

Description Megan Gardner 2017-08-25 17:52:31 PDT
Fix incorrect enum in atBoundaryOfGranularity call
Comment 1 Megan Gardner 2017-08-25 18:03:34 PDT
Created attachment 319123 [details]
Patch
Comment 2 Wenson Hsieh 2017-08-26 00:46:42 PDT
Comment on attachment 319123 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=319123&action=review

Nice find! r=me (though, this might still need a WK2R?)

> Source/WebKit/ChangeLog:6
> +        Enum that was being used was incorrect, as that case in not covered in called function.

The ChangeLog description should go under the "Reviewed by" line. (also, it looks like you meant "is not covered" instead of "in not covered").

> Source/WebKit/ChangeLog:7
> +        Resulted in a no-op, expected behavior will happen now.

I think a sentence or two describing the "expected behavior" here would be helpful. What change in behavior did you observe after making this tweak?
Comment 3 Megan Gardner 2017-08-28 16:56:51 PDT
https://trac.webkit.org/r221281
Comment 4 Radar WebKit Bug Importer 2017-08-28 16:57:46 PDT
<rdar://problem/34122131>