From 3d3c0fe487cb9b6fd2c708b91b3505c5d786b128 Mon Sep 17 00:00:00 2001
From: Jonathan Pevarnek <jpevarnek@google.com>
Date: Tue, 4 Aug 2015 17:44:40 -0400
Subject: [PATCH 1/2] Stop trying to update advanced settings autogrow textarea

The autogrow textarea has been removed.  The attempt to update it is
causing a runtime error.  This removes that attempt (and moves a few
other lines around)

Fixes #1797
---
 src/generic_ui/polymer/advanced-settings.ts | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/generic_ui/polymer/advanced-settings.ts b/src/generic_ui/polymer/advanced-settings.ts
index 5c0797bcbe..352a281554 100644
--- a/src/generic_ui/polymer/advanced-settings.ts
+++ b/src/generic_ui/polymer/advanced-settings.ts
@@ -39,13 +39,11 @@ Polymer({
   checkSettings_: function(oldSettings :any, newSettings :any) {
     for (var key in oldSettings){
       if (!(key in newSettings)) {
-        this.status = StatusState.KEY_VALUE_ERROR;
         return false;
       }
     }
     for (var key in newSettings){
       if (!(key in oldSettings)) {
-        this.status = StatusState.KEY_VALUE_ERROR;
         return false;
       }
     }
@@ -54,14 +52,16 @@ Polymer({
   setAdvancedSettings: function() {
     try {
       var newSettings = JSON.parse(this.settings);
-      if (this.checkSettings_(ui_context.model.globalSettings, newSettings)) {
-        ui_context.model.globalSettings = newSettings; 
-        this.status = StatusState.SET;
-        ui_context.core.updateGlobalSettings(ui_context.model.globalSettings);
-
-        this.settings = this.jsonifySettings_(ui_context.model.globalSettings);
-        this.$.advancedSettingsPaperTextarea.update();
+      if (!this.checkSettings_(ui_context.model.globalSettings, newSettings)) {
+        this.status = StatusState.KEY_VALUE_ERROR;
+        return;
       }
+
+      ui_context.model.globalSettings = newSettings;
+      this.status = StatusState.SET;
+      ui_context.core.updateGlobalSettings(ui_context.model.globalSettings);
+
+      this.settings = this.jsonifySettings_(ui_context.model.globalSettings);
     } catch (e) {
       this.status = StatusState.PARSE_ERROR;
     }

From f783dd7f5a7be0c4ab2a809ee99e4267fe353c1d Mon Sep 17 00:00:00 2001
From: Jonathan Pevarnek <jpevarnek@google.com>
Date: Tue, 4 Aug 2015 17:44:49 -0400
Subject: [PATCH 2/2] Fix the styling for advanced-setting on Firefox

On Firefox, the advanced-setting textarea was not filling the whole
space, this adds height: 100% to it to fix that issue.

Fixes #1798
---
 src/generic_ui/polymer/advanced-settings.html | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/src/generic_ui/polymer/advanced-settings.html b/src/generic_ui/polymer/advanced-settings.html
index b28e00e254..78faa0dd1b 100644
--- a/src/generic_ui/polymer/advanced-settings.html
+++ b/src/generic_ui/polymer/advanced-settings.html
@@ -1,6 +1,5 @@
 <link rel='import' href='../../bower/polymer/polymer.html'>
 <link rel='import' href='../../bower/paper-input/paper-input-decorator.html'>
-<link rel='import' href='../../bower/paper-input/paper-autogrow-textarea.html'>
 <link rel='import' href='../../bower/core-label/core-label.html'>
 <link rel='import' href='../../bower/core-tooltip/core-tooltip.html'>
 <link rel='import' href='../../bower/core-overlay/core-overlay.html'>
@@ -64,16 +63,12 @@
       background-color: rgb(240,240,240);
       padding-left: 10px;
       padding-top: 0;
-      padding-bottom: 0;
       margin-bottom: 0.75em;
       margin-top: 0.75em;
     }
-    #advancedSettingsDecorator {
-      height: 340px;
-    }
-    #advancedSettingsPaperTextarea {
-      height: 340px;
-      margin: 0em;
+    textarea[fit] {
+      /* fit does not properly size textareas on Firefox */
+      height: 100%;
     }
     .advancedSettingsText {
       font-size: 12px;
@@ -151,7 +146,7 @@
         <div id='formContainer' flex vertical layout>
           <p class='inputLabel'>{{ 'EDIT_ADVANCED_SETTINGS' | $$ }}</p>
           <paper-input-decorator id='advancedSettingsDecorator' label='{{ "EDIT_ADVANCED_SETTINGS_PLACEHOLDER" | $$ }}' flex>
-            <textarea id='advancedSettingsInput' value='{{ settings }}' fit>
+            <textarea value='{{ settings }}' fit>
             </textarea>
           </paper-input-decorator>
           <p id='confirmSetAdvancedSettings' class='advancedSettingsText' hidden?='{{ status !== StatusState.SET }}'>