author Jan Henning <jh+bugzilla@buttercookie.de>
Thu, 07 Feb 2019 20:41:16 +0000
changeset 512996 95b884dcbd6f001b990a6353980280050fab9ac6
parent 505383 6f3709b3878117466168c40affa7bca0b60cf75b
child 543493 c160562971eb15d2a3de71a011f223baedc46b3d
permissions -rw-r--r--
Bug 1496684 - Dispatch commonly expected startup notifications when opening a GeckoView window. r=snorp a=lizzard Once a webextension using a blocking WebRequest listener has started loading, all network connections covered by the extension's manifest are held until the extension is ready the process them. One condition for the extension being ready apparently includes browser startup having progressed far enough, as signified by "browser-delayed-startup-finished" having been dispatched. Therefore, we have to start sending that notification when opening a new Gecko- View window, too, and copy Fennec's InitLater() system for that. Unlike Fennec, we cannot tie registration of those InitLater() runnables to the initial content load having progressed far enough because of a) e10s, which makes that approach neither easily possible nor really sensible, as content will load in a different process in that case, and b) because we're racing with extension startup here - if extensions are loaded quick enough to block even the initial page load, we'd be deadlocked: We cannot send the notification until the page finishes loading, but the page cannot load until we send the notification. Fennec isn't affected by the latter problem because "sessionstore-windows-restored", which Fennec will send in any case, serves as an alternative pathway for completing extension startup. And unlike Desktop, we don't really have any chrome content to paint, so we cannot tie delayed initialisation to a paint listener for that, either. Therefore, we simply fire off a runnable at the *end* of geckoview.js's startup() method, which should give more pressing initialisation tasks enough of a headstart. For completeness, we're also adding the "browser-idle-startup-tasks-finished" notification and thereby solve bug 1465832 as well, allowing the ScriptPreloader to detect which scripts are commonly loaded during GeckoView startup and to start caching and pre-parsing them. Differential Revision: https://phabricator.services.mozilla.com/D18865

/* -*- 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 http://mozilla.org/MPL/2.0/. */

#ifndef nsBinaryStream_h___
#define nsBinaryStream_h___

#include "nsCOMPtr.h"
#include "nsAString.h"
#include "nsIObjectInputStream.h"
#include "nsIObjectOutputStream.h"
#include "nsIStreamBufferAccess.h"

#define NS_BINARYOUTPUTSTREAM_CID                    \
  { /* 86c37b9a-74e7-4672-844e-6e7dd83ba484 */       \
    0x86c37b9a, 0x74e7, 0x4672, {                    \
      0x84, 0x4e, 0x6e, 0x7d, 0xd8, 0x3b, 0xa4, 0x84 \
    }                                                \

#define NS_BINARYOUTPUTSTREAM_CONTRACTID "@mozilla.org/binaryoutputstream;1"

// Derive from nsIObjectOutputStream so this class can be used as a superclass
// by nsObjectOutputStream.
class nsBinaryOutputStream final : public nsIObjectOutputStream {
  nsBinaryOutputStream() {}

  friend already_AddRefed<nsIObjectOutputStream> NS_NewObjectOutputStream(

  // nsISupports methods

  // nsIOutputStream methods

  // nsIBinaryOutputStream methods

  // nsIObjectOutputStream methods

  // Call Write(), ensuring that all proffered data is written
  nsresult WriteFully(const char* aBuf, uint32_t aCount);

  nsCOMPtr<nsIOutputStream> mOutputStream;
  nsCOMPtr<nsIStreamBufferAccess> mBufferAccess;

  // virtual dtor since subclasses call our Release()
  virtual ~nsBinaryOutputStream() {}

#define NS_BINARYINPUTSTREAM_CID                     \
  { /* c521a612-2aad-46db-b6ab-3b821fb150b1 */       \
    0xc521a612, 0x2aad, 0x46db, {                    \
      0xb6, 0xab, 0x3b, 0x82, 0x1f, 0xb1, 0x50, 0xb1 \
    }                                                \

#define NS_BINARYINPUTSTREAM_CONTRACTID "@mozilla.org/binaryinputstream;1"

class nsBinaryInputStream final : public nsIObjectInputStream {
  nsBinaryInputStream() {}

  friend already_AddRefed<nsIObjectInputStream> NS_NewObjectInputStream(

  // nsISupports methods

  // nsIInputStream methods

  // nsIBinaryInputStream methods

  // nsIObjectInputStream methods

  nsCOMPtr<nsIInputStream> mInputStream;
  nsCOMPtr<nsIStreamBufferAccess> mBufferAccess;

  // virtual dtor since subclasses call our Release()
  virtual ~nsBinaryInputStream() {}

#endif  // nsBinaryStream_h___