Remove editor button default parameters

Closes PLAT-4660

Test Plan:
- Install an LTI 1 tool that uses
  com.instructure.Editor.contents and
  com.instructure.Editor.selection
- Launch the tool from an RCE that has content and
  a selection
- Verify the custom fields are sent
- Verify the custom fields are not duplicated
  in the launch without the "custom_" prefix

Change-Id: Ie77e2479f29c89ec8331f2cbf68bbdeb2de96168
Reviewed-on: https://gerrit.instructure.com/201204
Tested-by: Jenkins
Reviewed-by: Clint Furse <cfurse@instructure.com>
QA-Review: Clint Furse <cfurse@instructure.com>
Product-Review: Weston Dransfield <wdransfield@instructure.com>

lib/lti/privacy_level_expander.rb

@@ -20,7 +20,7 @@ module Lti
   class PrivacyLevelExpander
     EMAIL_ONLY = %w(Person.email.primary).freeze
     INLCUDE_NAME = %w(Person.name.given Person.name.full Person.name.family).freeze
-    PUBLIC = %w(Person.sourcedId CourseOffering.sourcedId com.instructure.Editor.selection com.instructure.Editor.contents).
+    PUBLIC = %w(Person.sourcedId CourseOffering.sourcedId).
       concat(EMAIL_ONLY).
       concat(INLCUDE_NAME).
       freeze

spec/lib/lti/privacy_level_expander_spec.rb

@@ -81,9 +81,7 @@ describe Lti::PrivacyLevelExpander do
                              Person.email.primary
                              Person.name.given
                              Person.name.full
-                             Person.name.family
-                             com.instructure.Editor.selection
-                             com.instructure.Editor.contents)
+                             Person.name.family)
         expect(helper.supported_parameters).to match_array expected_params
       end
     end