author Nicholas Nethercote <>
Fri, 03 Mar 2017 15:32:11 +1100
changeset 374836 4e196d802c7be7f3a3c147cc6c5b6406656584b1
parent 370542 7870b72807809a9ce4fb0165f9b10fdcb1f78aba
permissions -rw-r--r--
Bug 1344118 - Fix the profiler's sleeping threads optimization. r=jseward. When ProfilerBuffer::reset() is called, DuplicateLastSample() will start failing for all sleeping threads because there will be no prior thread data in the buffer to duplicate. But the sampling loop doesn't detect such failure. This causes two problems: - Missing samples. - CPU usage goes through the roof, because each time around the sampling loop the length of the failing search increases. The fix is simple: detect failure in the sampling loop and do a normal sample in that case. The patch also removes ThreadInfo::DuplicateLastSample(), because it just calls onto ProfileBuffer::DuplicateLastSample().

/* -*- Mode: C++; tab-width: 8; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
/* vim: set ts=8 sts=2 et sw=2 tw=80: */
/* 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 mozilla_dom_HTMLTableSectionElement_h
#define mozilla_dom_HTMLTableSectionElement_h

#include "mozilla/Attributes.h"
#include "nsGenericHTMLElement.h"
#include "nsContentList.h" // For ctor.

namespace mozilla {
namespace dom {

class HTMLTableSectionElement final : public nsGenericHTMLElement
  explicit HTMLTableSectionElement(already_AddRefed<mozilla::dom::NodeInfo>& aNodeInfo)
    : nsGenericHTMLElement(aNodeInfo)

  // nsISupports

  nsIHTMLCollection* Rows();
    InsertRow(int32_t aIndex, ErrorResult& aError);
  void DeleteRow(int32_t aValue, ErrorResult& aError);

  void GetAlign(DOMString& aAlign)
    GetHTMLAttr(nsGkAtoms::align, aAlign);
  void SetAlign(const nsAString& aAlign, ErrorResult& aError)
    SetHTMLAttr(nsGkAtoms::align, aAlign, aError);
  void GetCh(DOMString& aCh)
    GetHTMLAttr(nsGkAtoms::_char, aCh);
  void SetCh(const nsAString& aCh, ErrorResult& aError)
    SetHTMLAttr(nsGkAtoms::_char, aCh, aError);
  void GetChOff(DOMString& aChOff)
    GetHTMLAttr(nsGkAtoms::charoff, aChOff);
  void SetChOff(const nsAString& aChOff, ErrorResult& aError)
    SetHTMLAttr(nsGkAtoms::charoff, aChOff, aError);
  void GetVAlign(DOMString& aVAlign)
    GetHTMLAttr(nsGkAtoms::valign, aVAlign);
  void SetVAlign(const nsAString& aVAlign, ErrorResult& aError)
    SetHTMLAttr(nsGkAtoms::valign, aVAlign, aError);

  virtual bool ParseAttribute(int32_t aNamespaceID,
                              nsIAtom* aAttribute,
                              const nsAString& aValue,
                              nsAttrValue& aResult) override;
  virtual nsMapRuleToAttributesFunc GetAttributeMappingFunction() const override;
  NS_IMETHOD_(bool) IsAttributeMapped(const nsIAtom* aAttribute) const override;

  virtual nsresult Clone(mozilla::dom::NodeInfo *aNodeInfo, nsINode **aResult) const override;

  virtual ~HTMLTableSectionElement();

  virtual JSObject* WrapNode(JSContext *aCx, JS::Handle<JSObject*> aGivenProto) override;

  RefPtr<nsContentList> mRows;

  static void MapAttributesIntoRule(const nsMappedAttributes* aAttributes,
                                    GenericSpecifiedValues* aGenericData);

} // namespace dom
} // namespace mozilla

#endif /* mozilla_dom_HTMLTableSectionElement_h */