MD Settings Desktop: Set embedder when adding exceptions.

chrome/browser/resources/settings/site_settings/add_site_dialog.js

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview
  * 'add-site-dialog' provides a dialog to add exceptions for a given Content
  * Settings category.
  */
 Polymer({
@@ skipping 43 matching lines @@
   /**
    * The tap handler for the Add [Site] button (adds the pattern and closes
    * the dialog).
    * @private
    */
   onSubmit_: function() {
     if (this.$.add.disabled)
       return;  // Can happen when Enter is pressed.
     var pattern = this.addPatternWildcard_(this.site_);
     this.setCategoryPermissionForOrigin(
-        pattern, '', this.category, this.allowException ?
+        pattern, pattern, this.category, this.allowException ?
             settings.PermissionValues.ALLOW : settings.PermissionValues.BLOCK);

[dschuyler, 2016/08/08 18:21:26]

IIUC, this is saying that we want this specific
pattern (by using same pattern in both pattern
args). If that is correct, then LGTM. If I'm 
not right, please tell me more about it
and I'll review it again.

[Finnur, 2016/08/09 11:19:54]

Yeah, that's correct. 

Using '' (as I was doing) is equivalent to wildcard, which causes the 'Embedded
on *' message to appear.

In the old content settings system, if you add [*.]google.com, that pattern gets
used in both places, so this matches that.

     this.$.dialog.close();
   },
 });