Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(127)

Issue 153233002: *** DO NOT LAND *** Remove regions support, keeping a bare minimum to support "region-based"... (Closed)

Created:
6 years, 10 months ago by mstensho (USE GERRIT)
Modified:
6 years, 9 months ago
Reviewers:
CC:
blink-reviews, eae+blinkwatch, fs, eric.carlson_apple.com, apavlov+blink_chromium.org, adamk+blink_chromium.org, pdr, aandrey+blink_chromium.org, rwlbuis, Nils Barth (inactive), blink-layers+watch_chromium.org, caseq+blink_chromium.org, Nate Chapin, arv+blink, chromiumbugtracker_adobe.com, bemjb+rendering_chromium.org, dsinclair, yurys+blink_chromium.org, abarth-chromium, marja+watch_chromium.org, dglazkov+blink, jchaffraix+rendering, devtools-reviews_chromium.org, philipj_slow, loislo+blink_chromium.org, zoltan1, sof, lushnikov+blink_chromium.org, eustas+blink_chromium.org, paulirish+reviews_chromium.org, gyuyoung.kim_webkit.org, darktears, haraken, krit, kojih, jsbell+bindings_chromium.org, feature-media-reviews_chromium.org, alph+blink_chromium.org, leviw+renderwatch, vsevik+blink_chromium.org, pfeldman+blink_chromium.org, ed+blinkwatch_opera.com, f(malita), Inactive, Stephen Chennney, watchdog-blink-watchlist_google.com
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Visibility:
Public.

Description

Remove regions support, keeping a bare minimum to support "region-based" multicol. Remove everything strictly specific to CSS regions, but keep the parts that "region-based" multicol needs. So the render flow thread framework is still around, albeit heavily trimmed down. RenderRegion has also been kept, but is now obviously a misnomer (to be fixed later). It serves as a base class for what css3-break refers to as a "fragmentainer" (e.g. column, page or region). Not sure if it should really be a renderer type, seeing that these need to be created during layout (which should be avoided for renderers). A fragmentainer is more akin to what a line box represents than to what a renderer does. Additional removal and cleanup is possible, but let's start with this. Removed everything region-specific from LayoutTests. BUG=none

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3343 lines, -6911 lines) Patch
M Source/bindings/v8/custom/V8CSSRuleCustom.cpp View 2 chunks +0 lines, -3 lines 0 comments Download
M Source/core/core.gypi View 6 chunks +0 lines, -15 lines 0 comments Download
M Source/core/css/CSSComputedStyleDeclaration.cpp View 4 chunks +0 lines, -24 lines 0 comments Download
M Source/core/css/CSSGrammar.y View 8 chunks +0 lines, -62 lines 0 comments Download
M Source/core/css/CSSPrimitiveValueMappings.h View 1 chunk +0 lines, -30 lines 0 comments Download
M Source/core/css/CSSProperties.in View 2 chunks +0 lines, -6 lines 0 comments Download
M Source/core/css/CSSProperty.cpp View 1 chunk +0 lines, -6 lines 0 comments Download
M Source/core/css/CSSPropertyNames.in View 1 chunk +0 lines, -6 lines 0 comments Download
M Source/core/css/CSSPropertySourceData.h View 1 chunk +0 lines, -1 line 0 comments Download
D Source/core/css/CSSRegionRule.h View 1 chunk +0 lines, -55 lines 0 comments Download
D Source/core/css/CSSRegionRule.cpp View 1 chunk +0 lines, -61 lines 0 comments Download
M Source/core/css/CSSRule.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/CSSRule.cpp View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/css/CSSRule.idl View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/CSSTokenizer-in.cpp View 1 chunk +0 lines, -4 lines 0 comments Download
M Source/core/css/CSSValueKeywords.in View 1 chunk +0 lines, -4 lines 0 comments Download
M Source/core/css/ElementRuleCollector.h View 4 chunks +0 lines, -4 lines 0 comments Download
M Source/core/css/ElementRuleCollector.cpp View 3 chunks +0 lines, -18 lines 0 comments Download
M Source/core/css/RuleSet.h View 3 chunks +0 lines, -6 lines 0 comments Download
M Source/core/css/RuleSet.cpp View 3 chunks +0 lines, -29 lines 0 comments Download
M Source/core/css/RuntimeCSSEnabled.cpp View 1 chunk +0 lines, -9 lines 0 comments Download
M Source/core/css/StyleInvalidationAnalysis.cpp View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/StylePropertySet.cpp View 1 chunk +0 lines, -3 lines 0 comments Download
M Source/core/css/StyleRule.h View 4 chunks +0 lines, -21 lines 0 comments Download
M Source/core/css/StyleRule.cpp View 5 chunks +0 lines, -24 lines 0 comments Download
M Source/core/css/StyleSheetContents.cpp View 2 chunks +1 line, -5 lines 0 comments Download
D Source/core/css/WebKitCSSRegionRule.idl View 1 chunk +0 lines, -36 lines 0 comments Download
M Source/core/css/parser/BisonCSSParser.h View 4 chunks +0 lines, -8 lines 0 comments Download
M Source/core/css/parser/BisonCSSParser-in.cpp View 6 chunks +0 lines, -127 lines 0 comments Download
M Source/core/css/resolver/ScopedStyleResolver.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/resolver/ScopedStyleResolver.cpp View 2 chunks +0 lines, -14 lines 0 comments Download
M Source/core/css/resolver/StyleAdjuster.cpp View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/resolver/StyleBuilderCustom.cpp View 2 chunks +0 lines, -6 lines 0 comments Download
M Source/core/css/resolver/StyleResolver.h View 4 chunks +1 line, -19 lines 0 comments Download
M Source/core/css/resolver/StyleResolver.cpp View 8 chunks +2 lines, -33 lines 0 comments Download
M Source/core/css/resolver/StyleResolverState.h View 3 chunks +1 line, -9 lines 0 comments Download
M Source/core/css/resolver/StyleResolverState.cpp View 1 chunk +1 line, -2 lines 0 comments Download
D Source/core/dom/DOMNamedFlowCollection.h View 1 chunk +0 lines, -68 lines 0 comments Download
D Source/core/dom/DOMNamedFlowCollection.cpp View 1 chunk +0 lines, -90 lines 0 comments Download
M Source/core/dom/Document.h View 4 chunks +0 lines, -8 lines 0 comments Download
M Source/core/dom/Document.cpp View 3 chunks +0 lines, -20 lines 0 comments Download
M Source/core/dom/Document.idl View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/dom/Element.h View 5 chunks +0 lines, -14 lines 0 comments Download
M Source/core/dom/Element.cpp View 5 chunks +0 lines, -93 lines 0 comments Download
M Source/core/dom/Element.idl View 1 chunk +0 lines, -4 lines 0 comments Download
M Source/core/dom/ElementRareData.h View 3 chunks +0 lines, -11 lines 0 comments Download
M Source/core/dom/ElementRareData.cpp View 2 chunks +0 lines, -2 lines 0 comments Download
D Source/core/dom/NamedFlow.h View 1 chunk +0 lines, -93 lines 0 comments Download
D Source/core/dom/NamedFlow.cpp View 1 chunk +0 lines, -249 lines 0 comments Download
D Source/core/dom/NamedFlowCollection.h View 1 chunk +0 lines, -75 lines 0 comments Download
D Source/core/dom/NamedFlowCollection.cpp View 1 chunk +0 lines, -134 lines 0 comments Download
M Source/core/dom/Node.h View 3 chunks +11 lines, -18 lines 0 comments Download
M Source/core/dom/Node.cpp View 1 chunk +0 lines, -5 lines 0 comments Download
M Source/core/dom/NodeRenderingTraversal.cpp View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/dom/PseudoElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/dom/PseudoElement.cpp View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/dom/RenderTreeBuilder.h View 3 chunks +0 lines, -7 lines 0 comments Download
M Source/core/dom/RenderTreeBuilder.cpp View 5 chunks +1 line, -81 lines 0 comments Download
D Source/core/dom/WebKitNamedFlow.idl View 1 chunk +0 lines, -42 lines 0 comments Download
D Source/core/dom/WebKitNamedFlowCollection.idl View 1 chunk +0 lines, -39 lines 0 comments Download
M Source/core/events/EventTarget.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/events/EventTargetFactory.in View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/events/EventTypeNames.in View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/frame/UseCounter.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/frame/UseCounter.cpp View 1 chunk +0 lines, -6 lines 0 comments Download
M Source/core/inspector/InspectorCSSAgent.h View 8 chunks +0 lines, -20 lines 0 comments Download
M Source/core/inspector/InspectorCSSAgent.cpp View 9 chunks +0 lines, -264 lines 0 comments Download
M Source/core/inspector/InspectorInstrumentation.idl View 1 chunk +0 lines, -12 lines 0 comments Download
M Source/core/page/EventHandler.cpp View 2 chunks +0 lines, -11 lines 0 comments Download
M Source/core/rendering/CompositedLayerMapping.cpp View 3 chunks +5 lines, -5 lines 0 comments Download
M Source/core/rendering/FixedTableLayout.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/rendering/FlowThreadController.h View 1 chunk +4 lines, -47 lines 0 comments Download
M Source/core/rendering/FlowThreadController.cpp View 1 chunk +4 lines, -206 lines 0 comments Download
M Source/core/rendering/HitTestLocation.h View 4 chunks +1 line, -7 lines 0 comments Download
M Source/core/rendering/HitTestLocation.cpp View 7 chunks +2 lines, -10 lines 0 comments Download
M Source/core/rendering/HitTestResult.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/rendering/LayerPaintingInfo.h View 2 chunks +1 line, -3 lines 0 comments Download
M Source/core/rendering/PaintInfo.h View 3 chunks +1 line, -4 lines 0 comments Download
D Source/core/rendering/RegionOversetState.h View 1 chunk +0 lines, -33 lines 0 comments Download
M Source/core/rendering/RenderBlock.h View 5 chunks +2 lines, -44 lines 0 comments Download
M Source/core/rendering/RenderBlock.cpp View 12 chunks +10 lines, -140 lines 0 comments Download
M Source/core/rendering/RenderBlockFlow.h View 8 chunks +4 lines, -44 lines 0 comments Download
M Source/core/rendering/RenderBlockFlow.cpp View 18 chunks +20 lines, -111 lines 0 comments Download
M Source/core/rendering/RenderBlockLineLayout.cpp View 6 chunks +1 line, -16 lines 0 comments Download
M Source/core/rendering/RenderBox.h View 9 chunks +9 lines, -17 lines 0 comments Download
M Source/core/rendering/RenderBox.cpp View 40 chunks +43 lines, -285 lines 0 comments Download
M Source/core/rendering/RenderBoxModelObject.h View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/rendering/RenderBoxModelObject.cpp View 6 chunks +6 lines, -13 lines 0 comments Download
D Source/core/rendering/RenderBoxRegionInfo.h View 1 chunk +0 lines, -59 lines 0 comments Download
M Source/core/rendering/RenderDeprecatedFlexibleBox.cpp View 3 chunks +2 lines, -5 lines 0 comments Download
M Source/core/rendering/RenderFlexibleBox.cpp View 4 chunks +3 lines, -8 lines 0 comments Download
M Source/core/rendering/RenderFlowThread.h View 8 chunks +2 lines, -71 lines 0 comments Download
M Source/core/rendering/RenderFlowThread.cpp View 24 chunks +7 lines, -512 lines 0 comments Download
M Source/core/rendering/RenderGrid.cpp View 1 chunk +0 lines, -3 lines 0 comments Download
M Source/core/rendering/RenderImage.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/rendering/RenderInline.cpp View 1 chunk +1 line, -3 lines 0 comments Download
M Source/core/rendering/RenderLayer.h View 2 chunks +2 lines, -3 lines 0 comments Download
M Source/core/rendering/RenderLayer.cpp View 21 chunks +24 lines, -25 lines 0 comments Download
M Source/core/rendering/RenderLayerClipper.h View 1 chunk +1 line, -4 lines 0 comments Download
M Source/core/rendering/RenderLayerClipper.cpp View 8 chunks +8 lines, -11 lines 0 comments Download
M Source/core/rendering/RenderLayerCompositor.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/core/rendering/RenderLayerStackingNode.cpp View 1 chunk +1 line, -2 lines 0 comments Download
M Source/core/rendering/RenderMedia.cpp View 2 chunks +0 lines, -9 lines 0 comments Download
M Source/core/rendering/RenderMultiColumnSet.h View 1 chunk +0 lines, -4 lines 0 comments Download
D Source/core/rendering/RenderNamedFlowFragment.h View 1 chunk +0 lines, -74 lines 0 comments Download
D Source/core/rendering/RenderNamedFlowFragment.cpp View 1 chunk +0 lines, -108 lines 0 comments Download
D Source/core/rendering/RenderNamedFlowThread.h View 1 chunk +0 lines, -144 lines 0 comments Download
D Source/core/rendering/RenderNamedFlowThread.cpp View 1 chunk +0 lines, -730 lines 0 comments Download
M Source/core/rendering/RenderObject.h View 5 chunks +1 line, -7 lines 0 comments Download
M Source/core/rendering/RenderObject.cpp View 10 chunks +1 line, -70 lines 0 comments Download
M Source/core/rendering/RenderRegion.h View 5 chunks +0 lines, -106 lines 0 comments Download
M Source/core/rendering/RenderRegion.cpp View 12 chunks +3 lines, -463 lines 0 comments Download
M Source/core/rendering/RenderRegionSet.h View 2 chunks +0 lines, -8 lines 0 comments Download
M Source/core/rendering/RenderRegionSet.cpp View 1 chunk +0 lines, -6 lines 0 comments Download
M Source/core/rendering/RenderReplica.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/rendering/RenderTable.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/rendering/RenderTable.cpp View 4 chunks +6 lines, -8 lines 0 comments Download
M Source/core/rendering/RenderTableSection.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/rendering/RenderTreeAsText.cpp View 4 chunks +2 lines, -95 lines 0 comments Download
M Source/core/rendering/RenderView.h View 2 chunks +0 lines, -5 lines 0 comments Download
M Source/core/rendering/RenderView.cpp View 4 chunks +2 lines, -69 lines 0 comments Download
M Source/core/rendering/RootInlineBox.h View 3 chunks +1 line, -9 lines 0 comments Download
M Source/core/rendering/RootInlineBox.cpp View 2 chunks +1 line, -24 lines 0 comments Download
M Source/core/rendering/line/BreakingContextInlineHeaders.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/rendering/shapes/ShapeInfo.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/rendering/style/RenderStyle.h View 6 chunks +0 lines, -25 lines 0 comments Download
M Source/core/rendering/style/RenderStyle.cpp View 2 chunks +1 line, -4 lines 0 comments Download
M Source/core/rendering/style/RenderStyleConstants.h View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/rendering/style/StyleRareNonInheritedData.h View 1 chunk +0 lines, -8 lines 0 comments Download
M Source/core/rendering/style/StyleRareNonInheritedData.cpp View 3 chunks +0 lines, -18 lines 0 comments Download
M Source/core/rendering/svg/RenderSVGRoot.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/devtools/Inspector-1.1.json View 3 chunks +0 lines, -67 lines 0 comments Download
M Source/devtools/devtools.gypi View 4 chunks +0 lines, -7 lines 0 comments Download
D Source/devtools/front_end/CSSNamedFlowCollectionsView.js View 1 chunk +0 lines, -433 lines 0 comments Download
D Source/devtools/front_end/CSSNamedFlowView.js View 1 chunk +0 lines, -256 lines 0 comments Download
M Source/devtools/front_end/CSSStyleModel.js View 6 chunks +0 lines, -232 lines 0 comments Download
M Source/devtools/front_end/ElementsPanel.js View 2 chunks +0 lines, -14 lines 0 comments Download
D Source/devtools/front_end/Images/namedFlowOverflow.png View Binary file 0 comments Download
D Source/devtools/front_end/Images/regionEmpty.png View Binary file 0 comments Download
D Source/devtools/front_end/Images/regionFit.png View Binary file 0 comments Download
D Source/devtools/front_end/Images/regionOverset.png View Binary file 0 comments Download
D Source/devtools/front_end/cssNamedFlows.css View 1 chunk +0 lines, -99 lines 0 comments Download
A Source/devtools/front_end/protocol_externs.js View 1 chunk +3132 lines, -0 lines 0 comments Download
M Source/devtools/protocol.json View 3 chunks +0 lines, -67 lines 0 comments Download
M Source/platform/RuntimeEnabledFeatures.in View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
mstensho (USE GERRIT)
6 years, 10 months ago (2014-02-11 16:46:42 UTC) #1
Message was sent while issue was closed.
Decided to close this one after all. To be continued in:
https://codereview.chromium.org/159863002/

Powered by Google App Engine
This is Rietveld 408576698