Bug 118463

Summary: [CSS Regions] NamedFlowCollection getters should return undefined when there's no NamedFlow
Product: WebKit Reporter: Radu Stavila <stavila>
Component: CSSAssignee: Radu Stavila <stavila>
Status: RESOLVED INVALID    
Severity: Normal CC: cdumez, commit-queue, esprehn+autocc, haraken, kangil.han, kling, menard, mibalan, WebkitBugTracker
Priority: P2 Keywords: WebExposed
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
URL: http://www.w3.org/TR/2013/WD-css3-regions-20130528/#the-namedflow-interface
Bug Depends on:    
Bug Blocks: 57312    
Attachments:
Description Flags
Patch
cdumez: review+, cdumez: commit-queue-
Updated ChangeLog none

Attachments
Patch (13.70 KB, patch)
2013-07-08 09:41 PDT, Radu Stavila 		cdumez: review+
cdumez: commit-queue-
DetailsFormatted DiffDiff
Updated ChangeLog (13.82 KB, patch)
2013-07-08 10:57 PDT, Radu Stavila 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radu Stavila
Comment 1 2013-07-08 09:41:26 PDT
Created attachment 206252 [details] Patch
Radu Stavila
Comment 2 2013-07-08 09:54:08 PDT
Updated wiki: https://trac.webkit.org/wiki/WebKitIDL#TreatReturnedNullObjectAs
Chris Dumez
Comment 3 2013-07-08 10:52:07 PDT
Comment on attachment 206252 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=206252&action=review Looks good and according to spec. r=me. I'd like someone else to confirm that changing in behavior is OK though before landing as it is web-exposed. > LayoutTests/ChangeLog:7 > + Missing Changelog.
Radu Stavila
Comment 4 2013-07-08 10:57:33 PDT
Created attachment 206259 [details] Updated ChangeLog
Chris Dumez
Comment 5 2013-07-08 11:02:06 PDT
(In reply to comment #3) > (From update of attachment 206252 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=206252&action=review > > Looks good and according to spec. r=me. I'd like someone else to confirm that changing in behavior is OK though before landing as it is web-exposed. By someone else, I mean someone with more CSS knowledge than me as this might break compatibility.
Andreas Kling
Comment 6 2013-07-08 14:02:42 PDT
Comment on attachment 206259 [details] Updated ChangeLog LGTM.
WebKit Commit Bot
Comment 7 2013-07-08 14:25:22 PDT
Comment on attachment 206259 [details] Updated ChangeLog Clearing flags on attachment: 206259 Committed r152466: <http://trac.webkit.org/changeset/152466>
Mihai Balan
Comment 8 2013-08-27 04:54:19 PDT
Manually rolled back in bug 118561. Spec is to be changed.
Note You need to log in before you can comment on or make changes to this bug.