Bug 176004 - Fix incorrect enum in atBoundaryOfGranularity call
Summary: Fix incorrect enum in atBoundaryOfGranularity call
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Megan Gardner
Keywords: InRadar
Depends on:
Reported: 2017-08-25 17:52 PDT by Megan Gardner
Modified: 2017-08-28 16:57 PDT (History)
5 users (show)

See Also:

Patch (2.02 KB, patch)
2017-08-25 18:03 PDT, Megan Gardner
wenson_hsieh: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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]
Comment 2 Wenson Hsieh 2017-08-26 00:46:42 PDT
Comment on attachment 319123 [details]

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
Comment 4 Radar WebKit Bug Importer 2017-08-28 16:57:46 PDT