Added handling of $sitekey exceptions

lib/basedomain.js

 /*!
  * Parts of original code from ipv6.js <https://github.com/beaugunderson/javascript-ipv6>
  * Copyright 2011 Beau Gunderson
  * Available under MIT license <http://mths.be/mit>
  */
 
 const RE_V4 = /^(?:(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?|0x[0-9a-f][0-9a-f]?|0[0-7]{3})\.){3}(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?|0x[0-9a-f][0-9a-f]?|0[0-7]{3})$/i;
 const RE_V4_HEX = /^0x([0-9a-f]{8})$/i;
 const RE_V4_NUMERIC = /^[0-9]+$/;
 const RE_V4inV6 = /(?:(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.){3}(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)$/;
@@ skipping 147 matching lines @@
 }
 
 /**
  * Parses URLs and provides an interface similar to nsIURI in Gecko, see
  * https://developer.mozilla.org/en-US/docs/XPCOM_Interface_Reference/nsIURI.
  * TODO: Make sure the parsing actually works the same as nsStandardURL.
  * @constructor
  */
 function URI(/**String*/ spec)
 {
   this.spec = spec;

[Felix Dahlke, 2012/09/25 12:40:53]

Shouldn't this be this._spec? Seems to be the convention here.

[Wladimir Palant, 2012/09/25 14:09:41]

It's actually a public member like in nsIURI - that's how you get the string
representation of the URI object.

   this._schemeEnd = spec.indexOf(":");

[Felix Dahlke, 2012/09/25 12:40:53]

You could probably get rid of a lot of code here if you parse the spec using a
regex with capture groups.

[Wladimir Palant, 2012/09/25 14:09:41]

Maybe. Then again, that regexp will probably be absolutely unintelligible -
there are just too many optional parts. And there is a slight hope that this
version is actually faster (it does matter here).

   if (this._schemeEnd < 0)
     throw new Error("Invalid URI scheme");
 
-  if (spec[this._schemeEnd + 1] != "/" || spec[this._schemeEnd + 2] != "/")

[Felix Dahlke, 2012/09/25 12:40:53]

How about this:
if (spec.substr(this._schemeEnd + 1, 2) !== "//")

[Wladimir Palant, 2012/09/25 14:09:41]

Given that we are testing for string equality here and not whether we actually
deal with the same string instances I'll rather use != operator :)

+  if (spec.substr(this._schemeEnd + 1, 2) != "//")
     throw new Error("Unexpected URI structure");
 
   this._hostPortStart = this._schemeEnd + 3;
   this._hostPortEnd = spec.indexOf("/", this._hostPortStart);
   if (this._hostPortEnd < 0)
     throw new Error("Invalid URI host");
 
   var authEnd = spec.indexOf("@", this._hostPortStart);
   if (authEnd >= 0 && authEnd < this._hostPortEnd)
     this._hostPortStart = authEnd + 1;
@@ skipping 49 matching lines @@
   },
   get path()
   {
     return this.spec.substring(this._hostPortEnd);
   },
   get prePath()
   {
     return this.spec.substring(0, this._hostPortEnd);
   }
 };