RESOLVED FIXED 161077
[Win] Warning fixes. 
https://bugs.webkit.org/show_bug.cgi?id=161077
Summary [Win] Warning fixes.
Per Arne Vollan
Reported 2016-08-23 03:57:05 PDT
There are still a few warnings left in WebCore.
Attachments
Patch (9.92 KB, patch)
2016-08-23 04:17 PDT, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Patch (10.37 KB, patch)
2016-08-23 06:36 PDT, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Patch (10.29 KB, patch)
2016-08-23 10:00 PDT, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Per Arne Vollan
Comment 1 2016-08-23 04:17:12 PDT
Created attachment 286698 [details] Patch
Per Arne Vollan
Comment 2 2016-08-23 06:36:36 PDT
Created attachment 286708 [details] Patch
Per Arne Vollan
Comment 3 2016-08-23 06:41:20 PDT
Index: Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp =================================================================== --- Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp (revision 204814) +++ Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp (working copy) @@ -392,6 +392,8 @@ void GraphicsLayerCA::initialize(Type la case Type::Shape: platformLayerType = PlatformCALayer::LayerType::LayerTypeShapeLayer; break; + default: + RELEASE_ASSERT_NOT_REACHED(); } m_layer = createPlatformCALayer(platformLayerType, this); noteLayerPropertyChanged(ContentsScaleChanged); Would it be better to just return here?
Per Arne Vollan
Comment 4 2016-08-23 10:00:05 PDT
Created attachment 286723 [details] Patch
Brent Fulgham
Comment 5 2016-08-25 12:34:20 PDT
Comment on attachment 286723 [details] Patch Nice! We're getting so close to zero warnings! :-) r=me.
Per Arne Vollan
Comment 6 2016-08-26 06:08:59 PDT
Thanks for reviewing!
WebKit Commit Bot
Comment 7 2016-08-26 06:31:18 PDT
Comment on attachment 286723 [details] Patch Clearing flags on attachment: 286723 Committed r205011: <http://trac.webkit.org/changeset/205011>
WebKit Commit Bot
Comment 8 2016-08-26 06:31:22 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.