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

Issue 319283003: Refactor dart:_blink into classes to improve startup time (Closed)

Created:
6 years, 6 months ago by Leaf
Modified:
6 years, 6 months ago
Reviewers:
vsm, Jacob, kasperl
CC:
reviews_dartlang.org, ricow1, siva
Visibility:
Public.

Description

Refactor dart:_blink into classes to improve startup time. This CL reduces the snapshot size by about 20% (580kb). BUG= R=jacobr@google.com Committed: https://code.google.com/p/dart/source/detail?r=37189

Patch Set 1 #

Patch Set 2 : Rebase #

Patch Set 3 : Rebase #

Patch Set 4 : Rebase #

Patch Set 5 : Rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8728 lines, -7522 lines) Patch
M sdk/lib/_blink/dartium/_blink_dartium.dart View 1 2 3 4 2 chunks +5083 lines, -3905 lines 0 comments Download
M sdk/lib/html/dartium/html_dartium.dart View 468 chunks +2388 lines, -2388 lines 0 comments Download
M sdk/lib/indexed_db/dartium/indexed_db_dartium.dart View 1 2 3 4 10 chunks +72 lines, -72 lines 0 comments Download
M sdk/lib/svg/dartium/svg_dartium.dart View 1 2 3 4 113 chunks +712 lines, -712 lines 0 comments Download
M sdk/lib/web_audio/dartium/web_audio_dartium.dart View 1 2 3 4 23 chunks +148 lines, -148 lines 0 comments Download
M sdk/lib/web_gl/dartium/web_gl_dartium.dart View 1 2 3 4 10 chunks +182 lines, -182 lines 0 comments Download
M sdk/lib/web_sql/dartium/web_sql_dartium.dart View 1 2 3 4 7 chunks +14 lines, -14 lines 0 comments Download
M tools/dom/scripts/htmldartgenerator.py View 1 2 3 4 4 chunks +7 lines, -9 lines 0 comments Download
M tools/dom/scripts/systemnative.py View 1 2 3 4 16 chunks +46 lines, -26 lines 0 comments Download
M tools/dom/src/blink_native_DOMImplementation.dart View 1 2 3 4 1 chunk +38 lines, -28 lines 0 comments Download
M tools/dom/src/html_native_DOMImplementation.dart View 1 2 3 4 6 chunks +24 lines, -24 lines 0 comments Download
M tools/dom/templates/html/impl/impl_Element.darttemplate View 1 2 3 4 1 chunk +14 lines, -14 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Leaf
6 years, 6 months ago (2014-06-09 21:53:10 UTC) #1
Jacob
lgtm
6 years, 6 months ago (2014-06-09 22:13:13 UTC) #2
Leaf
Committed patchset #5 manually as r37189 (presubmit successful).
6 years, 6 months ago (2014-06-10 17:38:37 UTC) #3
kasperl
6 years, 6 months ago (2014-06-10 18:06:05 UTC) #4
Message was sent while issue was closed.
Is this a merge-to-trunk candidate?

Powered by Google App Engine
This is Rietveld 408576698