Bug 458167 - Fix bad usage of "it's" in comments: patch for content folder. r+sr=jst
authorDaniel Holbert <dholbert@cs.stanford.edu>
Fri, 23 Jan 2009 01:02:09 -0800
changeset 24112 c6dd3fd7c949ccfea36a2545db6bc4533d25f174
parent 24111 cf24acb3310264fbda1a854c20fe6567fd5ce642
child 24113 4d8381e446c48be37b75ce179d94d716d3baac61
push idunknown
push userunknown
push dateunknown
bugs458167
milestone1.9.2a1pre
Bug 458167 - Fix bad usage of "it's" in comments: patch for content folder. r+sr=jst
content/base/public/nsContentUtils.h
content/base/public/nsINodeInfo.h
content/base/src/nsContentIterator.cpp
content/base/src/nsDocument.h
content/base/src/nsHTMLContentSerializer.cpp
content/base/src/nsTreeWalker.cpp
content/base/src/nsTreeWalker.h
content/events/src/nsEventStateManager.h
content/html/content/public/nsIRadioControlElement.h
content/html/content/src/nsHTMLInputElement.cpp
content/html/content/src/nsHTMLLinkElement.cpp
content/html/content/src/nsHTMLTableElement.cpp
content/html/document/src/nsHTMLContentSink.cpp
content/media/video/src/nsMediaStream.cpp
content/smil/nsSMILAnimationController.cpp
content/smil/nsSMILTimedElement.cpp
content/xbl/src/nsBindingManager.cpp
content/xslt/src/xpath/txBooleanExpr.cpp
content/xslt/src/xpath/txExpr.h
--- a/content/base/public/nsContentUtils.h
+++ b/content/base/public/nsContentUtils.h
@@ -1624,17 +1624,17 @@ private:
 
   // The value 300 is not important, as long as it is bigger then anything
   // ever passed to Mutated().
   enum { eMaxMutations = 300 };
 
   
   // sMutationCount is a global mutation counter which is decreased by one at
   // every mutation. It is capped at 0 to avoid wrapping.
-  // It's value is always between 0 and 300, inclusive.
+  // Its value is always between 0 and 300, inclusive.
   static PRUint32 sMutationCount;
 };
 
 #define NS_AUTO_GCROOT_PASTE2(tok,line) tok##line
 #define NS_AUTO_GCROOT_PASTE(tok,line) \
   NS_AUTO_GCROOT_PASTE2(tok,line)
 #define NS_AUTO_GCROOT(ptr, result) \ \
   nsAutoGCRoot NS_AUTO_GCROOT_PASTE(_autoGCRoot_, __LINE__) \
--- a/content/base/public/nsINodeInfo.h
+++ b/content/base/public/nsINodeInfo.h
@@ -158,29 +158,29 @@ public:
     return mInner.mPrefix;
   }
 
   /*
    * Get the namespace URI for a node, if the node has a namespace URI.
    *
    * For the HTML element "<body>" in a HTML document this will return a null
    * string and for the XML element "<html:body>" (assuming that this element,
-   * or one of it's ancestors has an
+   * or one of its ancestors has an
    * xmlns:html='http://www.w3.org/1999/xhtml' attribute) this will return
    * the string "http://www.w3.org/1999/xhtml".
    */
   virtual nsresult GetNamespaceURI(nsAString& aNameSpaceURI) const = 0;
 
   /*
    * Get the namespace ID for a node if the node has a namespace, if not this
    * returns kNameSpaceID_None.
    *
    * For the HTML element "<body>" in a HTML document this will return
    * kNameSpaceID_None and for the XML element "<html:body>" (assuming that
-   * this element, or one of it's ancestors has an
+   * this element, or one of its ancestors has an
    * xmlns:html='http://www.w3.org/1999/xhtml' attribute) this will return
    * the namespace ID for "http://www.w3.org/1999/xhtml".
    */
   PRInt32 NamespaceID() const
   {
     return mInner.mNamespaceID;
   }
 
--- a/content/base/src/nsContentIterator.cpp
+++ b/content/base/src/nsContentIterator.cpp
@@ -1343,17 +1343,17 @@ nsresult nsContentSubtreeIterator::Init(
 
   if (nodeBefore || nodeAfter)
   {
     MakeEmpty();
     return NS_OK;
   }
 
   // cool, we have the first node in the range.  Now we walk
-  // up it's ancestors to find the most senior that is still
+  // up its ancestors to find the most senior that is still
   // in the range.  That's the real first node.
   if (NS_FAILED(GetTopAncestorInRange(firstCandidate, address_of(mFirst))))
     return NS_ERROR_FAILURE;
 
   // now to find the last node
   aRange->GetEndOffset(&indx);
   PRInt32 numChildren = nEndP->GetChildCount();
 
@@ -1394,17 +1394,17 @@ nsresult nsContentSubtreeIterator::Init(
 
   if (nodeBefore || nodeAfter)
   {
     MakeEmpty();
     return NS_OK;
   }
 
   // cool, we have the last node in the range.  Now we walk
-  // up it's ancestors to find the most senior that is still
+  // up its ancestors to find the most senior that is still
   // in the range.  That's the real first node.
   if (NS_FAILED(GetTopAncestorInRange(lastCandidate, address_of(mLast))))
     return NS_ERROR_FAILURE;
   
   mCurNode = mFirst;
 
   return NS_OK;
 }
--- a/content/base/src/nsDocument.h
+++ b/content/base/src/nsDocument.h
@@ -660,17 +660,17 @@ public:
    * HTML META tags).
    */
   virtual void GetHeaderData(nsIAtom* aHeaderField, nsAString& aData) const;
   virtual void SetHeaderData(nsIAtom* aheaderField,
                              const nsAString& aData);
 
   /**
    * Create a new presentation shell that will use aContext for
-   * it's presentation context (presentation context's <b>must not</b> be
+   * its presentation context (presentation context's <b>must not</b> be
    * shared among multiple presentation shell's).
    */
   virtual nsresult CreateShell(nsPresContext* aContext,
                                nsIViewManager* aViewManager,
                                nsStyleSet* aStyleSet,
                                nsIPresShell** aInstancePtrResult);
   virtual PRBool DeleteShell(nsIPresShell* aShell);
   virtual nsIPresShell *GetPrimaryShell() const;
--- a/content/base/src/nsHTMLContentSerializer.cpp
+++ b/content/base/src/nsHTMLContentSerializer.cpp
@@ -670,17 +670,17 @@ nsHTMLContentSerializer::AppendElementSt
   nsAutoString nameStr;
   name->ToString(nameStr);
   AppendToString(nameStr.get(), -1, aStr);
 
   // Need to keep track of OL and LI elements in order to get ordinal number 
   // for the LI.
   if (mIsCopying && name == nsGkAtoms::ol){
     // We are copying and current node is an OL;
-    // Store it's start attribute value in olState->startVal.
+    // Store its start attribute value in olState->startVal.
     nsAutoString start;
     PRInt32 startAttrVal = 0;
     aElement->GetAttribute(NS_LITERAL_STRING("start"), start);
     if (!start.IsEmpty()){
       PRInt32 rv = 0;
       startAttrVal = start.ToInteger(&rv);
       //If OL has "start" attribute, first LI element has to start with that value
       //Therefore subtracting 1 as all the LI elements are incrementing it before using it;
--- a/content/base/src/nsTreeWalker.cpp
+++ b/content/base/src/nsTreeWalker.cpp
@@ -540,17 +540,17 @@ nsTreeWalker::ChildOf(nsINode* aNode,
         }
     }
 
     *_retval = nsnull;
     return NS_OK;
 }
 
 /*
- * Gets the child index of a node within it's parent. Gets a possible index
+ * Gets the child index of a node within its parent. Gets a possible index
  * from mPossibleIndexes to gain speed. If the value in mPossibleIndexes
  * isn't correct it'll get the index the usual way
  * @param aParent   in which to get the index
  * @param aChild    node to get the index of
  * @param aIndexPos position in mPossibleIndexes that contains the possible.
  *                  index
  * @returns         resulting index
  */
--- a/content/base/src/nsTreeWalker.h
+++ b/content/base/src/nsTreeWalker.h
@@ -142,17 +142,17 @@ private:
      */
     nsresult ChildOf(nsINode* aNode,
                      PRInt32 childNum,
                      PRBool aReversed,
                      PRInt32 aIndexPos,
                      nsINode** _retval);
 
     /*
-     * Gets the child index of a node within it's parent. Gets a possible index
+     * Gets the child index of a node within its parent. Gets a possible index
      * from mPossibleIndexes to gain speed. If the value in mPossibleIndexes
      * isn't correct it'll get the index the usual way.
      * @param aParent   in which to get the index
      * @param aChild    node to get the index of
      * @param aIndexPos position in mPossibleIndexes that contains the possible.
      *                  index
      * @returns         resulting index
      */
--- a/content/events/src/nsEventStateManager.h
+++ b/content/events/src/nsEventStateManager.h
@@ -252,17 +252,17 @@ protected:
    * on descendant docshells first, then on the ancestor (with |aBubbledFrom|
    * set to the docshell associated with |this|), until something matches.
    *
    * @param aPresContext the presentation context
    * @param aEvent the key event
    * @param aStatus the event status
    * @param aBubbledFrom is used by an ancestor to avoid calling HandleAccessKey()
    *        on the child the call originally came from, i.e. this is the child
-   *        that recursively called us in it's Up phase. The initial caller
+   *        that recursively called us in its Up phase. The initial caller
    *        passes |nsnull| here. This is to avoid an infinite loop.
    * @param aAccessKeyState Normal, Down or Up processing phase (see enums
    *        above). The initial event receiver uses 'normal', then 'down' when
    *        processing children and Up when recursively calling its ancestor.
    * @param aModifierMask modifier mask for the key event
    */
   void HandleAccessKey(nsPresContext* aPresContext,
                        nsKeyEvent* aEvent,
--- a/content/html/content/public/nsIRadioControlElement.h
+++ b/content/html/content/public/nsIRadioControlElement.h
@@ -73,17 +73,17 @@ public:
    * @param aOnlyOnThisRadioButton set to TRUE to only set the property only on
    *        the radio button in question; otherwise it will set the property on
    *        all other radio buttons in its group just to be sure they are
    *        consistent.
    */
   NS_IMETHOD SetCheckedChangedInternal(PRBool aCheckedChanged) = 0;
 
   /**
-   * Let an entire radio group know that it's "checked" property has been
+   * Let an entire radio group know that its "checked" property has been
    * changed by JS or by the user (calls SetCheckedChangedInternal multiple
    * times via a visitor)
    *
    * @param aCheckedChanged whether the checked property has been changed.
    */
   NS_IMETHOD SetCheckedChanged(PRBool aCheckedChanged) = 0;
 
   /**
--- a/content/html/content/src/nsHTMLInputElement.cpp
+++ b/content/html/content/src/nsHTMLInputElement.cpp
@@ -2518,17 +2518,17 @@ nsHTMLInputElement::SubmitNamesValues(ns
     // Get our default value, which is the same as our default label
     nsXPIDLString defaultValue;
     nsContentUtils::GetLocalizedString(nsContentUtils::eFORMS_PROPERTIES,
                                        "Submit", defaultValue);
     value = defaultValue;
   }
       
   //
-  // Submit file if it's input type=file and this encoding method accepts files
+  // Submit file if its input type=file and this encoding method accepts files
   //
   if (mType == NS_FORM_INPUT_FILE) {
     //
     // Open the file
     //
     nsCOMPtr<nsIFile> file;
  
     rv = GetFile(getter_AddRefs(file));
--- a/content/html/content/src/nsHTMLLinkElement.cpp
+++ b/content/html/content/src/nsHTMLLinkElement.cpp
@@ -252,17 +252,17 @@ void
 nsHTMLLinkElement::CreateAndDispatchEvent(nsIDocument* aDoc,
                                           const nsAString& aEventName)
 {
   if (!aDoc)
     return;
 
   // In the unlikely case that both rev is specified *and* rel=stylesheet,
   // this code will cause the event to fire, on the principle that maybe the
-  // page really does want to specify that it's author is a stylesheet. Since
+  // page really does want to specify that its author is a stylesheet. Since
   // this should never actually happen and the performance hit is minimal,
   // doing the "right" thing costs virtually nothing here, even if it doesn't
   // make much sense.
   static nsIContent::AttrValuesArray strings[] =
     {&nsGkAtoms::_empty, &nsGkAtoms::stylesheet, nsnull};
 
   if (!nsContentUtils::HasNonEmptyAttr(this, kNameSpaceID_None,
                                        nsGkAtoms::rev) &&
--- a/content/html/content/src/nsHTMLTableElement.cpp
+++ b/content/html/content/src/nsHTMLTableElement.cpp
@@ -748,17 +748,17 @@ nsHTMLTableElement::DeleteCaption()
   return NS_OK;
 }
 
 NS_IMETHODIMP
 nsHTMLTableElement::InsertRow(PRInt32 aIndex, nsIDOMHTMLElement** aValue)
 {
   /* get the ref row at aIndex
      if there is one, 
-       get it's parent
+       get its parent
        insert the new row just before the ref row
      else
        get the first row group
        insert the new row as its first child
   */
   *aValue = nsnull;
 
   if (aIndex < -1) {
--- a/content/html/document/src/nsHTMLContentSink.cpp
+++ b/content/html/document/src/nsHTMLContentSink.cpp
@@ -1597,17 +1597,17 @@ NS_IMPL_ISUPPORTS_INHERITED2(HTMLContent
 
 static PRBool
 IsScriptEnabled(nsIDocument *aDoc, nsIDocShell *aContainer)
 {
   NS_ENSURE_TRUE(aDoc && aContainer, PR_TRUE);
 
   nsCOMPtr<nsIScriptGlobalObject> globalObject = aDoc->GetScriptGlobalObject();
 
-  // Getting context is tricky if the document hasn't had it's
+  // Getting context is tricky if the document hasn't had its
   // GlobalObject set yet
   if (!globalObject) {
     nsCOMPtr<nsIScriptGlobalObjectOwner> owner = do_GetInterface(aContainer);
     NS_ENSURE_TRUE(owner, PR_TRUE);
 
     globalObject = owner->GetScriptGlobalObject();
     NS_ENSURE_TRUE(globalObject, PR_TRUE);
   }
--- a/content/media/video/src/nsMediaStream.cpp
+++ b/content/media/video/src/nsMediaStream.cpp
@@ -394,17 +394,17 @@ public:
   virtual void     Suspend();
   virtual void     Resume();
 
   // Return PR_TRUE if the stream has been cancelled.
   PRBool IsCancelled() const;
 
   // This must be called on the main thread only, and at a
   // time when the strategy is not reading from the current
-  // channel/stream. It's primary purpose is to be called from
+  // channel/stream. Its primary purpose is to be called from
   // a Seek to reset to the new byte range request http channel.
   void Reset(nsIChannel* aChannel, 
              nsChannelToPipeListener* aListener, 
              nsIInputStream* aStream);
 
 private:
   // Listener attached to channel to constantly download the
   // media data asynchronously and store it in the pipe. The 
--- a/content/smil/nsSMILAnimationController.cpp
+++ b/content/smil/nsSMILAnimationController.cpp
@@ -284,17 +284,17 @@ nsSMILAnimationController::DoSample(PRBo
                                          aSkipUnchangedContainers };
   mChildContainerTable.EnumerateEntries(SampleTimeContainer, &tcParams);
 
   // STEP 2: (i)  Sample the timed elements AND
   //         (ii) Create a table of compositors
   // 
   // (i) Here we sample the timed elements (fetched from the
   // nsISMILAnimationElements) which determine from the active time if the
-  // element is active and what it's simple time etc. is. This information is
+  // element is active and what its simple time etc. is. This information is
   // then passed to its time client (nsSMILAnimationFunction).
   //
   // (ii) During the same loop we also build up a table that contains one
   // compositor for each animated attribute and which maps animated elements to
   // the corresponding compositor for their target attribute.
   //
   // Note that this compositor table needs to be allocated on the heap so we can
   // store it until the next sample. This lets us find out which elements were
--- a/content/smil/nsSMILTimedElement.cpp
+++ b/content/smil/nsSMILTimedElement.cpp
@@ -124,17 +124,17 @@ nsSMILTimedElement::BeginElementAt(doubl
   //                      BeginElementAt
   //
   // After BeginElementAt(0) is called a new begin instance time is added to the
   // list. Depending on the restart mode this may generate a new interval,
   // possiblying ending the current interval early.
   //
   // Intuitively this change should take effect before the subsequent call to
   // BeginElementAt however to get this to take effect we need to drive the
-  // state engine through it's sequence active-waiting-active by calling Sample.
+  // state engine through its sequence active-waiting-active by calling Sample.
   //
   // When we get the second call to BeginElementAt the element should be in the
   // active state and hence the new begin instance time will be ignored because
   // it is before the beginning of the (new) current interval. SMIL says we do
   // not change the begin of a current interval once it is active.
   //
   // See also:
   // http://www.w3.org/TR/SMIL3/smil-timing.html#Timing-BeginEnd-Restart
--- a/content/xbl/src/nsBindingManager.cpp
+++ b/content/xbl/src/nsBindingManager.cpp
@@ -886,17 +886,17 @@ nsBindingManager::RemoveLayeredBinding(n
   // Finally remove the binding...
   // XXXbz this doesn't remove the implementation!  Should fix!  Until
   // then we need the explicit UnhookEventHandlers here.
   binding->UnhookEventHandlers();
   binding->ChangeDocument(doc, nsnull);
   SetBinding(aContent, nsnull);
   binding->MarkForDeath();
   
-  // ...and recreate it's frames. We need to do this since the frames may have
+  // ...and recreate its frames. We need to do this since the frames may have
   // been removed and style may have changed due to the removal of the
   // anonymous children.
   // XXXbz this should be using the current doc (if any), not the owner doc.
   nsIPresShell *presShell = doc->GetPrimaryShell();
   NS_ENSURE_TRUE(presShell, NS_ERROR_FAILURE);
 
   return presShell->RecreateFramesFor(aContent);;
 }
--- a/content/xslt/src/xpath/txBooleanExpr.cpp
+++ b/content/xslt/src/xpath/txBooleanExpr.cpp
@@ -34,17 +34,17 @@
  * the provisions above, a recipient may use your version of this file under
  * the terms of any one of the MPL, the GPL or the LGPL.
  *
  * ***** END LICENSE BLOCK ***** */
 
 
 /**
  * Represents a BooleanExpr, a binary expression that
- * performs a boolean operation between it's lvalue and rvalue.
+ * performs a boolean operation between its lvalue and rvalue.
 **/
 
 #include "txExpr.h"
 #include "txIXPathContext.h"
 
 /**
  * Evaluates this Expr based on the given context node and processor state
  * @param context the context node for evaluation of this Expr
--- a/content/xslt/src/xpath/txExpr.h
+++ b/content/xslt/src/xpath/txExpr.h
@@ -713,17 +713,17 @@ public:
 
     TX_DECL_EXPR
 
 private:
     nsRefPtr<txAExprResult> mValue;
 };
 
 /**
- * Represents an UnaryExpr. Returns the negative value of it's expr.
+ * Represents an UnaryExpr. Returns the negative value of its expr.
 **/
 class UnaryExpr : public Expr {
 
 public:
 
     UnaryExpr(Expr* aExpr)
         : expr(aExpr)
     {
@@ -732,17 +732,17 @@ public:
     TX_DECL_EXPR
 
 private:
     nsAutoPtr<Expr> expr;
 }; //-- UnaryExpr
 
 /**
  * Represents a BooleanExpr, a binary expression that
- * performs a boolean operation between it's lvalue and rvalue.
+ * performs a boolean operation between its lvalue and rvalue.
 **/
 class BooleanExpr : public Expr
 {
 public:
 
     //-- BooleanExpr Types
     enum _BooleanExprType { AND = 1, OR };
 
@@ -757,17 +757,17 @@ public:
 
 private:
     nsAutoPtr<Expr> leftExpr, rightExpr;
     short op;
 }; //-- BooleanExpr
 
 /**
  * Represents a MultiplicativeExpr, a binary expression that
- * performs a multiplicative operation between it's lvalue and rvalue:
+ * performs a multiplicative operation between its lvalue and rvalue:
  *  *   : multiply
  * mod  : modulus
  * div  : divide
  *
 **/
 class txNumberExpr : public Expr
 {
 public:
@@ -784,18 +784,18 @@ public:
     TX_DECL_EXPR
 
 private:
     nsAutoPtr<Expr> mLeftExpr, mRightExpr;
     eOp mOp;
 }; //-- MultiplicativeExpr
 
 /**
- * Represents a RelationalExpr, an expression that compares it's lvalue
- * to it's rvalue using:
+ * Represents a RelationalExpr, an expression that compares its lvalue
+ * to its rvalue using:
  * =  : equal to
  * <  : less than
  * >  : greater than
  * <= : less than or equal to
  * >= : greater than or equal to
  *
 **/
 class RelationalExpr : public Expr