author Iain Ireland <>
Fri, 11 Jan 2019 18:05:36 +0000
changeset 453523 025feea5945bffc625e9cca2e23be51ee8670315
parent 452446 f0a91d36587266d7454a450c6044d573664fbed5
permissions -rw-r--r--
Bug 1480390: Move ForOfIterClose logic inside TryNoteIter r=tcampbell This patch was intended to be a pure refactoring of existing code with no side-effects, moving the logic for handling for-of/for-of-iterclose trynotes inside TryNoteIter to avoid duplicating logic in all users of TryNoteIter. However, it turns out that there was a subtle preexisting bug in TryNoteIter that is fixed by the refactoring. Specifically, the logic to skip from a for-of-iterclose to its enclosing for-of must run before the logic to skip trynotes based on stack depth. Otherwise, the stack depth code may filter out the enclosing for-of (see the attached test case for an example) and we will skip too many try-notes. Differential Revision:

/* -*- Mode: C++; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
/* This Source Code Form is subject to the terms of the Mozilla Public
 * License, v. 2.0. If a copy of the MPL was not distributed with this
 * file, You can obtain one at */
#ifndef nsIFragmentContentSink_h___
#define nsIFragmentContentSink_h___

#include "nsISupports.h"

namespace mozilla {
namespace dom {
class Document;
class DocumentFragment;
}  // namespace dom
}  // namespace mozilla

#define NS_I_FRAGMENT_CONTENT_SINK_IID               \
  {                                                  \
    0x1a8ce30b, 0x63fc, 0x441a, {                    \
      0xa3, 0xaa, 0xf7, 0x16, 0xc0, 0xfe, 0x96, 0x69 \
    }                                                \

 * The fragment sink allows a client to parse a fragment of sink, possibly
 * surrounded in context. Also see nsIParser::ParseFragment().
 * Note: once you've parsed a fragment, the fragment sink must be re-set on
 * the parser in order to parse another fragment.
class nsIFragmentContentSink : public nsISupports {
   * This method is used to obtain the fragment created by
   * a fragment content sink and to release resources held by the parser.
   * The sink drops its reference to the fragment.
  NS_IMETHOD FinishFragmentParsing(mozilla::dom::DocumentFragment** aFragment) =

   * This method is used to set the target document for this fragment
   * sink.  This document's nodeinfo manager will be used to create
   * the content objects.  This MUST be called before the sink is used.
   * @param aDocument the document the new nodes will belong to
   * (should not be null)
  NS_IMETHOD SetTargetDocument(mozilla::dom::Document*) = 0;

   * This method is used to indicate to the sink that we're done building
   * the context and should start paying attention to the incoming content
  NS_IMETHOD WillBuildContent() = 0;

   * This method is used to indicate to the sink that we're done building
   * The real content. This is useful if you want to parse additional context
   * (such as an end context).
  NS_IMETHOD DidBuildContent() = 0;

   * This method is a total hack to help with parsing fragments. It is called to
   * tell the fragment sink that a container from the context will be delivered
   * after the call to WillBuildContent(). This is only relevent for HTML
   * fragments that use nsHTMLTokenizer/CNavDTD.
  NS_IMETHOD IgnoreFirstContainer() = 0;

   * Sets whether scripts elements are marked as unexecutable.
  NS_IMETHOD SetPreventScriptExecution(bool aPreventScriptExecution) = 0;


nsresult NS_NewXMLFragmentContentSink(
    nsIFragmentContentSink** aInstancePtrResult);