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

Unified Diff: editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/preferences/DartBasePreferencePage.java

Issue 23496048: Add additional hints to the analyzer to catch the 'x is double' dart2js bug. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Rebase to latest & merge Created 7 years, 3 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
Index: editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/preferences/DartBasePreferencePage.java
diff --git a/editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/preferences/DartBasePreferencePage.java b/editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/preferences/DartBasePreferencePage.java
index 798af4845622080f203e588855fd66e2ce535efc..f325d5a6697d261c7ee656b01d2be836dff270ad 100644
--- a/editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/preferences/DartBasePreferencePage.java
+++ b/editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/preferences/DartBasePreferencePage.java
@@ -59,7 +59,6 @@ public class DartBasePreferencePage extends PreferencePage implements IWorkbench
private Button enableFolding;
private Button enableAutoCompletion;
private Button runPubAutoCheck;
- private Button enableHints;
public DartBasePreferencePage() {
setPreferenceStore(DartToolsPlugin.getDefault().getPreferenceStore());
@@ -124,12 +123,6 @@ public class DartBasePreferencePage extends PreferencePage implements IWorkbench
IEclipsePreferences prefs = DartCore.getPlugin().getPrefs();
if (prefs != null) {
prefs.putBoolean(DartCore.PUB_AUTO_RUN_PREFERENCE, runPubAutoCheck.getSelection());
- boolean hints = DartCore.getPlugin().isHintsEnabled();
- prefs.putBoolean(DartCore.ENABLE_HINTS_PREFERENCE, enableHints.getSelection());
- if (hints != enableHints.getSelection()) {
- // trigger processing for hints
- DartCore.getProjectManager().setHintOption(enableHints.getSelection());
- }
try {
DartCore.getPlugin().savePrefs();
} catch (CoreException e) {
@@ -148,7 +141,7 @@ public class DartBasePreferencePage extends PreferencePage implements IWorkbench
composite);
GridLayoutFactory.fillDefaults().spacing(0, 8).margins(0, 10).applyTo(composite);
- // General preferences
+ // General group
Group generalGroup = new Group(composite, SWT.NONE);
generalGroup.setText(PreferencesMessages.DartBasePreferencePage_general);
GridDataFactory.fillDefaults().grab(true, false).align(SWT.FILL, SWT.BEGINNING).applyTo(
@@ -225,12 +218,7 @@ public class DartBasePreferencePage extends PreferencePage implements IWorkbench
PreferencesMessages.DartBasePreferencePage_enable_code_folding_tooltip);
GridDataFactory.fillDefaults().span(2, 1).applyTo(enableFolding);
- enableHints = createCheckBox(
- generalGroup,
- PreferencesMessages.DartBasePreferencePage_enable_hints,
- PreferencesMessages.DartBasePreferencePage_enable_hints_tooltip);
- GridDataFactory.fillDefaults().span(2, 1).applyTo(enableHints);
-
+ // Save actions group
Group saveGroup = new Group(composite, SWT.NONE);
saveGroup.setText(PreferencesMessages.DartBasePreferencePage_save);
GridDataFactory.fillDefaults().grab(true, false).align(SWT.FILL, SWT.BEGINNING).applyTo(
@@ -243,6 +231,7 @@ public class DartBasePreferencePage extends PreferencePage implements IWorkbench
PreferencesMessages.DartBasePreferencePage_trailing_ws_details);
GridDataFactory.fillDefaults().applyTo(removeTrailingWhitespaceCheck);
+ // Pub group
Group pubGroup = new Group(composite, SWT.NONE);
pubGroup.setText(PreferencesMessages.DartBasePreferencePage_pub);
GridDataFactory.fillDefaults().grab(true, false).align(SWT.FILL, SWT.BEGINNING).applyTo(
@@ -255,6 +244,7 @@ public class DartBasePreferencePage extends PreferencePage implements IWorkbench
PreferencesMessages.DartBasePreferencePage_pub_auto_details);
GridDataFactory.fillDefaults().applyTo(runPubAutoCheck);
+ // init
initFromPrefs();
return composite;
@@ -296,7 +286,6 @@ public class DartBasePreferencePage extends PreferencePage implements IWorkbench
IEclipsePreferences prefs = DartCore.getPlugin().getPrefs();
if (prefs != null) {
- enableHints.setSelection(prefs.getBoolean(DartCore.ENABLE_HINTS_PREFERENCE, true));
runPubAutoCheck.setSelection(prefs.getBoolean(DartCore.PUB_AUTO_RUN_PREFERENCE, true));
}

Powered by Google App Engine
This is Rietveld 408576698