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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: runtime/observatory/lib/src/elements/allocation_profile.dart

Issue 2991203002: Fix Observatory virtual-collection layout issues (Closed)
Patch Set: Fix persistent handles page Created 3 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | runtime/observatory/lib/src/elements/containers/virtual_collection.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/observatory/lib/src/elements/allocation_profile.dart
diff --git a/runtime/observatory/lib/src/elements/allocation_profile.dart b/runtime/observatory/lib/src/elements/allocation_profile.dart
index 441091dc151cd468dad39acd83b84d3d7b3a39df..cbd1ff8c4f1f8ff3d9ae8dce2824ce6b23096830 100644
--- a/runtime/observatory/lib/src/elements/allocation_profile.dart
+++ b/runtime/observatory/lib/src/elements/allocation_profile.dart
@@ -356,67 +356,75 @@ class AllocationProfileElement extends HtmlElement implements Renderable {
new SpanElement()..classes = ['name']
];
- Element _createCollectionHeader() => new DivElement()
- ..children = [
- new DivElement()
- ..classes = ['collection-item']
- ..children = [
- new SpanElement()
- ..classes = ['group']
- ..text = 'Accumulated',
- new SpanElement()
- ..classes = ['group']
- ..text = 'Current',
- new SpanElement()
- ..classes = ['group']
- ..text = '(NEW) Accumulated',
- new SpanElement()
- ..classes = ['group']
- ..text = '(NEW) Current',
- new SpanElement()
- ..classes = ['group']
- ..text = '(OLD) Accumulated',
- new SpanElement()
- ..classes = ['group']
- ..text = '(OLD) Current',
- ],
- new DivElement()
- ..classes = ['collection-item']
- ..children = [
- _createHeaderButton(const ['bytes'], 'Size',
- _SortingField.accumulatedSize, _SortingDirection.descending),
- _createHeaderButton(const ['instances'], 'Instances',
- _SortingField.accumulatedInstances, _SortingDirection.descending),
- _createHeaderButton(const ['bytes'], 'Size',
- _SortingField.currentSize, _SortingDirection.descending),
- _createHeaderButton(const ['instances'], 'Instances',
- _SortingField.currentInstances, _SortingDirection.descending),
- _createHeaderButton(const ['bytes'], 'Size',
- _SortingField.newAccumulatedSize, _SortingDirection.descending),
- _createHeaderButton(
- const ['instances'],
- 'Instances',
- _SortingField.newAccumulatedInstances,
- _SortingDirection.descending),
- _createHeaderButton(const ['bytes'], 'Size',
- _SortingField.newCurrentSize, _SortingDirection.descending),
- _createHeaderButton(const ['instances'], 'Instances',
- _SortingField.newCurrentInstances, _SortingDirection.descending),
- _createHeaderButton(const ['bytes'], 'Size',
- _SortingField.oldAccumulatedSize, _SortingDirection.descending),
- _createHeaderButton(
- const ['instances'],
- 'Instances',
- _SortingField.oldAccumulatedInstances,
- _SortingDirection.descending),
- _createHeaderButton(const ['bytes'], 'Size',
- _SortingField.oldCurrentSize, _SortingDirection.descending),
- _createHeaderButton(const ['instances'], 'Instances',
- _SortingField.oldCurrentInstances, _SortingDirection.descending),
- _createHeaderButton(const ['name'], 'Class', _SortingField.className,
- _SortingDirection.ascending)
- ],
- ];
+ List<HtmlElement> _createCollectionHeader() => [
+ new DivElement()
+ ..classes = ['collection-item']
+ ..children = [
+ new SpanElement()
+ ..classes = ['group']
+ ..text = 'Accumulated',
+ new SpanElement()
+ ..classes = ['group']
+ ..text = 'Current',
+ new SpanElement()
+ ..classes = ['group']
+ ..text = '(NEW) Accumulated',
+ new SpanElement()
+ ..classes = ['group']
+ ..text = '(NEW) Current',
+ new SpanElement()
+ ..classes = ['group']
+ ..text = '(OLD) Accumulated',
+ new SpanElement()
+ ..classes = ['group']
+ ..text = '(OLD) Current',
+ ],
+ new DivElement()
+ ..classes = ['collection-item']
+ ..children = [
+ _createHeaderButton(const ['bytes'], 'Size',
+ _SortingField.accumulatedSize, _SortingDirection.descending),
+ _createHeaderButton(
+ const ['instances'],
+ 'Instances',
+ _SortingField.accumulatedInstances,
+ _SortingDirection.descending),
+ _createHeaderButton(const ['bytes'], 'Size',
+ _SortingField.currentSize, _SortingDirection.descending),
+ _createHeaderButton(const ['instances'], 'Instances',
+ _SortingField.currentInstances, _SortingDirection.descending),
+ _createHeaderButton(const ['bytes'], 'Size',
+ _SortingField.newAccumulatedSize, _SortingDirection.descending),
+ _createHeaderButton(
+ const ['instances'],
+ 'Instances',
+ _SortingField.newAccumulatedInstances,
+ _SortingDirection.descending),
+ _createHeaderButton(const ['bytes'], 'Size',
+ _SortingField.newCurrentSize, _SortingDirection.descending),
+ _createHeaderButton(
+ const ['instances'],
+ 'Instances',
+ _SortingField.newCurrentInstances,
+ _SortingDirection.descending),
+ _createHeaderButton(const ['bytes'], 'Size',
+ _SortingField.oldAccumulatedSize, _SortingDirection.descending),
+ _createHeaderButton(
+ const ['instances'],
+ 'Instances',
+ _SortingField.oldAccumulatedInstances,
+ _SortingDirection.descending),
+ _createHeaderButton(const ['bytes'], 'Size',
+ _SortingField.oldCurrentSize, _SortingDirection.descending),
+ _createHeaderButton(
+ const ['instances'],
+ 'Instances',
+ _SortingField.oldCurrentInstances,
+ _SortingDirection.descending),
+ _createHeaderButton(const ['name'], 'Class',
+ _SortingField.className, _SortingDirection.ascending)
+ ],
+ ];
ButtonElement _createHeaderButton(List<String> classes, String text,
_SortingField field, _SortingDirection direction) =>
« no previous file with comments | « no previous file | runtime/observatory/lib/src/elements/containers/virtual_collection.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698