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

Unified Diff: sdk/lib/web_sql/dart2js/web_sql_dart2js.dart

Issue 27033009: Making all base class DOM types throw as well (to prevent a default constructor for DOM types that … (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 2 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 | « sdk/lib/web_gl/dartium/web_gl_dartium.dart ('k') | sdk/lib/web_sql/dartium/web_sql_dartium.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sdk/lib/web_sql/dart2js/web_sql_dart2js.dart
diff --git a/sdk/lib/web_sql/dart2js/web_sql_dart2js.dart b/sdk/lib/web_sql/dart2js/web_sql_dart2js.dart
index 17851e2c6bfb6d044e7a718a7bb677b3c56b0006..f43139ebcaf32ed24cc545fd73fa942e8de03591 100644
--- a/sdk/lib/web_sql/dart2js/web_sql_dart2js.dart
+++ b/sdk/lib/web_sql/dart2js/web_sql_dart2js.dart
@@ -82,6 +82,8 @@ typedef void SqlTransactionErrorCallback(SqlError error);
// http://www.w3.org/TR/webdatabase/#asynchronous-database-api
@Experimental() // deprecated
class SqlDatabase extends Interceptor native "Database" {
+ // To suppress missing implicit constructor warnings.
+ factory SqlDatabase._() { throw new UnsupportedError("Not supported"); }
/// Checks if this type is supported on the current platform.
static bool get supported => JS('bool', '!!(window.openDatabase)');
@@ -124,6 +126,8 @@ class SqlDatabase extends Interceptor native "Database" {
// http://www.w3.org/TR/webdatabase/#sqlerror
@Experimental() // deprecated
class SqlError extends Interceptor native "SQLError" {
+ // To suppress missing implicit constructor warnings.
+ factory SqlError._() { throw new UnsupportedError("Not supported"); }
@DomName('SQLError.CONSTRAINT_ERR')
@DocsEditable()
@@ -175,6 +179,8 @@ class SqlError extends Interceptor native "SQLError" {
// http://www.w3.org/TR/webdatabase/#sqlresultset
@Experimental() // deprecated
class SqlResultSet extends Interceptor native "SQLResultSet" {
+ // To suppress missing implicit constructor warnings.
+ factory SqlResultSet._() { throw new UnsupportedError("Not supported"); }
@DomName('SQLResultSet.insertId')
@DocsEditable()
@@ -198,6 +204,8 @@ class SqlResultSet extends Interceptor native "SQLResultSet" {
// http://www.w3.org/TR/webdatabase/#sqlresultsetrowlist
@Experimental() // deprecated
class SqlResultSetRowList extends Interceptor with ListMixin<Map>, ImmutableListMixin<Map> implements List native "SQLResultSetRowList" {
+ // To suppress missing implicit constructor warnings.
+ factory SqlResultSetRowList._() { throw new UnsupportedError("Not supported"); }
@DomName('SQLResultSetRowList.length')
@DocsEditable()
@@ -272,6 +280,8 @@ class SqlResultSetRowList extends Interceptor with ListMixin<Map>, ImmutableList
// http://www.w3.org/TR/webdatabase/#sqltransaction
@deprecated // deprecated
class SqlTransaction extends Interceptor native "SQLTransaction" {
+ // To suppress missing implicit constructor warnings.
+ factory SqlTransaction._() { throw new UnsupportedError("Not supported"); }
@DomName('SQLTransaction.executeSql')
@DocsEditable()
@@ -290,4 +300,6 @@ class SqlTransaction extends Interceptor native "SQLTransaction" {
// http://www.w3.org/TR/webdatabase/#sqltransactionsync
@Experimental() // deprecated
abstract class _SQLTransactionSync extends Interceptor native "SQLTransactionSync" {
+ // To suppress missing implicit constructor warnings.
+ factory _SQLTransactionSync._() { throw new UnsupportedError("Not supported"); }
}
« no previous file with comments | « sdk/lib/web_gl/dartium/web_gl_dartium.dart ('k') | sdk/lib/web_sql/dartium/web_sql_dartium.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698