Commit graph

8139 commits

Author SHA1 Message Date
dgrove@google.com
470ac9ff85 Clean up repo README, add SDK README
Review URL: https://codereview.chromium.org//10977058

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12974 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 17:23:09 +00:00
blois@google.com
9cfdc2dddc Revert "Replacing non-portable CanvasRenderingContext.setFillColor with more portable variants."
This reverts commit 12972 because it was failing in dartium.

TBR=vsm
BUG=

Review URL: https://codereview.chromium.org//10996048

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12973 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 17:13:35 +00:00
blois@google.com
ab684728a8 Replacing non-portable CanvasRenderingContext.setFillColor with more portable variants.
Original CL failed because:
Additional tests were marked as fail because they used the broken behavior. I've fixed these tests and removed their exclusions.

Argument types were mismatched, causing Dartium to fail.

BUG=4592

Review URL: https://codereview.chromium.org//10984055

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12972 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 16:39:36 +00:00
brianwilkerson@google.com
c1be0e7612 Parse and report error for const parameters (issue 5289)
Review URL: https://codereview.chromium.org//10990086

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12971 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 16:34:03 +00:00
brianwilkerson@google.com
3f95f76764 Convert analyzer issue to co19 issue
Review URL: https://codereview.chromium.org//10982069

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12970 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 15:30:44 +00:00
ahe@google.com
dbf1cbd5bd Attempt to fix dartium mac build
Review URL: https://codereview.chromium.org//10982070

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12969 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 15:30:14 +00:00
dgrove@google.com
6f564a3602 Don't copy extraneous files in the html directory.
This addresses the vast majority of issue 4129.
Review URL: https://codereview.chromium.org//10996041

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12968 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 15:05:28 +00:00
ngeoffray@google.com
3d095ba4a6 Update status file to fix the bot.
Review URL: https://codereview.chromium.org//10990082

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12967 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 14:54:51 +00:00
ngeoffray@google.com
f92bab9db9 Fix another assert triggered by host-checked mode: only register fields to the backend.
Review URL: https://codereview.chromium.org//10984070

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12966 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 14:44:14 +00:00
ngeoffray@google.com
2670dba2ec Use getEnclosingClass instead of enclosingElement (which might be a patch thingy). Fixes assertion hit in host-checked mode.
Review URL: https://codereview.chromium.org//10977056

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12965 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 14:33:23 +00:00
ahe@google.com
1c518f6169 Separate compile-time and runtime errors.
Review URL: https://codereview.chromium.org//10977054

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12964 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 14:31:05 +00:00
ager@google.com
fe66a0af1a Revert hiding of VM-only coreimpl list implementation types. While I
investigate performance regression (even after fixing constructors
in the intrinsifier).

R=iposva@google.com,srdjan@google.com,fschneider@google.com
BUG=

Review URL: https://codereview.chromium.org//11000025

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12963 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 14:07:56 +00:00
sgjesse@google.com
7735ef3129 Make the right type annotation fix to r12956
TBR=ngeoffray@google.com

BUG=

Review URL: https://codereview.chromium.org//10977055

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12962 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 13:53:14 +00:00
ager@google.com
9bdfd4fd19 Fix status files
R=lrn@google.com
BUG=

Review URL: https://codereview.chromium.org//10981053

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12961 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 13:50:32 +00:00
ngeoffray@google.com
767f7a4a79 Keep the invariant that intersecting with conflicting returns conflicting.
Review URL: https://codereview.chromium.org//11000023

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12960 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 13:49:57 +00:00
sgjesse@google.com
856a1ef1fd Fixes to last commit
TBR=ngeoffray@google.com

BUG=

Review URL: https://codereview.chromium.org//10979052

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12959 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 13:41:09 +00:00
ager@google.com
2d9645ea5a Intrinsify named constructors on private classes.
This was broken before my GrowableArray change as well for the
_Double.fromInteger constructor.

This is using the fact that named constructors are the only method
names that contain '.'. Therefore, it is enough to check that the
classes match and that the method name starting at the '.' matches.

R=srdjan@google.com,iposva@google.com,fschneider@google.com
BUG=

Review URL: https://codereview.chromium.org//11000021

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12958 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 13:35:19 +00:00
ngeoffray@google.com
fa578567e3 Add a value range analysis phase to remove bounds checks.
Review URL: https://codereview.chromium.org//10968060

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12957 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 13:23:20 +00:00
sgjesse@google.com
5a1a2831e8 Change the type inference for fields in dart2js
The type inference for fields now uses the same mechanism as is
used for type inference for arguments and return types. That
means that compilation is optimistically taking the current
inferred type for a field. When an inferred type for a field
changes all functions previously compiled under the previous type
assumption are scheduled for recompilation.

There is special handling of constructors where fields which are
always set in a constructor is identified. This is done to avoid
having to combine with the null type which can be set by the
initializer list.

This change removes the two phase compilation previously used for
inferring field types.

R=kasperl@google.com, ngeoffray@google.com

BUG=

Review URL: https://codereview.chromium.org//10964016

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12956 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 13:21:05 +00:00
lrn@google.com
37197c2553 Stop using the Hashable interface.
All objects have a hashCode method, so there is no need for a Hashable
interface.
The interface itself is retained for at least two weeks, to give time for
code to remove uses of it.

Review URL: https://codereview.chromium.org//10993059

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12955 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 13:07:54 +00:00
ngeoffray@google.com
6b3b096ad4 Fix dart2dart bot
Review URL: https://codereview.chromium.org//10984068

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12954 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 13:01:15 +00:00
scheglov@google.com
344cc4b415 Quick fixes 'Create class' and 'Create constructor'
http://code.google.com/p/dart/issues/detail?id=5325

R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//10996036

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12953 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 12:46:07 +00:00
ager@google.com
8e6a2c8345 Hide VM-only coreimpl List implementation types. These should not be
exposed as visible parts of coreimpl.

Review URL: https://codereview.chromium.org//10990055

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12952 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 12:37:30 +00:00
ager@google.com
1032612e3e Correctly mark scalarlists as abstract classes with factory
constructors.

The marking as abstract avoids warnings about non-implemented
inherited methods for the scalarlists on construction.

Once the classes are abstract the constructors have to be
factory constructors which they are anyway.

R=sgjesse@google.com
BUG=dartbug.com/5469

Review URL: https://codereview.chromium.org//11000018

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12951 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 12:35:11 +00:00
ngeoffray@google.com
04637e33be Fix dartium bots.
Review URL: https://codereview.chromium.org//10982063

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12949 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 11:53:05 +00:00
lrn@google.com
ae0d6fca62 Ensure that hashCode and runtimeType work on null.
BUG=5510

Review URL: https://codereview.chromium.org//10979050

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12948 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 11:38:16 +00:00
ngeoffray@google.com
dc6d97e2d4 Update test to new API and have the _test suffix.
Review URL: https://codereview.chromium.org//10996043

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12947 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 11:36:18 +00:00
podivilov@google.com
b001a58bca Get rid of System classes for backend generators.
They introduce unnecessary level of indirection with no added value.

R=antonm@google.com,vsm@google.com

Review URL: https://codereview.chromium.org//10986048

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12946 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 10:25:08 +00:00
johnniwinther@google.com
d2585eb746 Removed use of GT_GT_GT_TOKEN/GT_GT_GT_EQ_TOKEN in dartdoc.
Review URL: https://codereview.chromium.org//10986063

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12945 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 10:19:57 +00:00
ahe@google.com
8ca223d2c4 Update co19 status
Review URL: https://codereview.chromium.org//10981050

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12944 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 08:14:30 +00:00
ahe@google.com
684c93a740 Rename a variable to fix build breakage
Review URL: https://codereview.chromium.org//10990080

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12943 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 07:44:19 +00:00
ahe@google.com
e13c0b1c7d Always fail if a process failed to launch.
This reveals a problem with the implementation of extra commands, which is also fixed.

Review URL: https://codereview.chromium.org//10993048

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12942 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 07:39:40 +00:00
ager@google.com
81e81825c9 Make data setter private on GrowableObjectArray.
There is no data setter on Lists. Additionally, this allows to
create buffer overflows.

R=vegorov@google.com,iposva@google.com
BUG=

Review URL: https://codereview.chromium.org//10982045

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12941 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 06:21:07 +00:00
scheglov@google.com
c67487ccd6 Fix for NPE with StringLiteral in toString()
R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//10990074

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12940 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-27 04:06:14 +00:00
efortuna@google.com
e73afc3647 Reapply FooList -> List<Foo> with Gamepad fix for dartium.
Review URL: https://codereview.chromium.org//10982053

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12939 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 23:23:54 +00:00
vsm@google.com
700da179e7 Roll IDL to multivm@910, Take #2
I had to suppress the Window.indexedDB and Element.remove methods in favor of the existing custom code.

Review URL: https://codereview.chromium.org//10993053

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12937 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 23:05:10 +00:00
blois@google.com
adcd66064b Revert "Replacing non-portable CanvasRenderingContext.setFillColor with more portable variants."
This reverts commit 10979029.

This broke dartium (not sure why) and canvas_test, which actually relied on the broken behavior.

TBR=vsm
BUG=

Review URL: https://codereview.chromium.org//10983052

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12936 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 22:59:49 +00:00
blois@google.com
539fb987b1 Replacing non-portable CanvasRenderingContext.setFillColor with more portable variants.
BUG=4594

Review URL: https://codereview.chromium.org//10979029

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12935 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 22:37:35 +00:00
hausner@google.com
5411a33a48 Address Siva's review comment
Review URL: https://codereview.chromium.org//10985054

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12932 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 21:26:44 +00:00
vsm@google.com
7488109fc1 Revert "Roll IDL to multivm@910"
This change is breaking Safari and Chrome bots.  Investigating.

Review URL: https://codereview.chromium.org//11000007

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12931 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 21:05:04 +00:00
hausner@google.com
f452955cd7 Fix co19 status
Review URL: https://codereview.chromium.org//10982055

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12930 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 21:04:23 +00:00
srdjan@google.com
0c9dd53739 More compact code for strict compare with 0.
Noticed while tracking mysterious Bubblesort performance degradation.
Review URL: https://codereview.chromium.org//10984051

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12929 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 21:00:51 +00:00
hausner@google.com
a241a7bae6 Make throw an expression
Throw e is now an expression rather than a statement. The language
spec says that throw without an expression (i.e. rethrowing the
current exception) is an expression as well. I have not implemented
this. It is rather difficult to determine at expression level whether
the keyword "throw" is followed by an expression or not. Thus,
throw without an expression is only allowed at statement level.

I think we should change the language spec to that effect.
Review URL: https://codereview.chromium.org//10982051

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12928 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 20:53:36 +00:00
vsm@google.com
c9d56a6967 Roll IDL to multivm@910
TBR=sra

Review URL: https://codereview.chromium.org//10983049

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12927 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 20:47:57 +00:00
efortuna@google.com
192ca09005 webkitRequestAimationFrame->requestAnimationFrame with hand edits to
html_dartium.
Review URL: https://codereview.chromium.org//10990061

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12926 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 18:39:53 +00:00
blois@google.com
eada8aa043 Fixing up WheelEvent to follow W3C standards and work on all platforms.
There were a couple of problems here:
- The name of the event is different between platforms, logic to handle this has been added.
- The event object was not getting wrapped on Firefox.
- The wheel event parameters were different across all platforms.

The event name ('mouseWheel' vs 'DOMMouseScroll' vs 'wheel') has been fixed on ElementEvents, but not BodyElementEvents or DocumentElementEvents. I've opened bug #5414 to track fixing those up.

BUG=5186

Review URL: https://codereview.chromium.org//10969073

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12921 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 17:54:39 +00:00
iposva@google.com
b1f0b23fc7 - Remove obsolete class.
Review URL: https://codereview.chromium.org//10979035

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12920 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 17:25:39 +00:00
devoncarew@google.com
74f89e5f98 Update the meta package to use the new pub format.
Review URL: https://codereview.chromium.org//10979033

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12919 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 16:47:10 +00:00
jmesserly@google.com
98f7601e38 Add Element.xtag to public interface
This lets us refer to it easily, instead of via (element as Dynamic).xtag, which in turn allows users to think about it as the Element being the source of truth, and storing a reference to the Element instead of the WebComponent object.

I opened up the doc comment in Firefox to double check that it renders correctly in API doc.

Review URL: https://codereview.chromium.org//10987045

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12917 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 16:38:01 +00:00
ngeoffray@google.com
ed445aae7a Fix braino: iteration is through a loop, so you continue it with continue.
Review URL: https://codereview.chromium.org//10989034

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@12915 260f80e4-7a28-3924-810f-c04153c831b5
2012-09-26 16:11:23 +00:00