.flake8
author Andrew McCreight <continuation@gmail.com>
Mon, 22 Apr 2019 16:34:51 +0000
changeset 470376 3073770e06f157040f4c64951b7e8425e1ad7bbe
parent 467682 85a8b43922d85adbb493a1956c223b29d84bdcf6
child 474049 8f69c7eeb6fd147b0066756389b95a734be8ae86
permissions -rw-r--r--
Bug 1535403 - Take indirection into account for the CC optimizations for the outer window wrapper. r=peterv Most wrapper cached C++ objects are held alive by their wrapper. The cycle collector takes advantage of this in many classes and ignores the C++ object if the wrapper is marked black. However, this is not true for the outer window's wrapper. Instead, the outer window's wrapper keeps the inner window alive. The inner window usually keeps its outer window alive, but not after it has been unlinked. For reasons I do not yet understand, the outer window's wrapper can be kept alive after the inner window it is a proxy for is unlinked. This patch fixes the cycle collector optimization for the outer window by only applying it if the outer window still has a weak reference to the inner window, which it will until the inner no longer holds the outer alive. This in turn fixes, or at least helps fix, window leaks seen intermittently when the lifetime of outer windows and docshells are tied together. The code comment is based on a review comment by peterv. Differential Revision: https://phabricator.services.mozilla.com/D27981

[flake8]
max-line-length = 99
exclude =
    # These paths should be triaged and either fixed or moved to the list below.
    client.py,
    devtools/shared,
    dom/base,
    dom/bindings,
    dom/browser-element,
    dom/canvas,
    dom/encoding,
    dom/imptests,
    dom/security,
    dom/websocket,
    gfx/tests,
    layout/base/tests/marionette,
    layout/reftests/border-image,
    layout/reftests/fonts,
    layout/reftests/w3c-css,
    layout/style,
    media/libdav1d/generate_source.py,
    moz.configure,
    netwerk/dns/prepare_tlds.py,
    netwerk/protocol/http/make_incoming_tables.py,
    python/devtools/migrate-l10n/migrate/main.py,
    python/l10n/fluent_migrations,
    python/mozbuild/dumbmake,
    python/mozbuild/mozbuild,
    servo/components/style,
    testing/jsshell/benchmark.py,
    testing/marionette/mach_commands.py,
    testing/mozharness/docs,
    testing/mozharness/examples,
    testing/mozharness/external_tools,
    testing/mozharness/mach_commands.py,
    testing/mozharness/manifestparser,
    testing/mozharness/mozprocess,
    testing/mozharness/setup.py,
    testing/parse_build_tests_ccov.py,
    testing/runtimes/writeruntimes.py,
    testing/tools/iceserver/iceserver.py,
    testing/tools/view_gecko_profile/view_gecko_profile.py,
    testing/tools/websocketprocessbridge/websocketprocessbridge.py,
    testing/web-platform,
    toolkit/components/featuregates,
    toolkit/content/tests/chrome/file_about_networking_wsh.py,
    toolkit/crashreporter/tools/symbolstore.py,
    toolkit/crashreporter/tools/unit-symbolstore.py,
    toolkit/library/dependentlibs.py,
    toolkit/locales/generate_update_locale.py,
    toolkit/mozapps,
    toolkit/moz.configure,
    toolkit/nss.configure,

    # These paths are intentionally excluded (not necessarily for good reason).
    build/build-infer/build-infer.py,
    build/moz.configure/*.configure,
    build/pymake/,
    browser/extensions/mortar/ppapi/,
    browser/moz.configure,
    dom/canvas/test/webgl-conf/checkout/closure-library/,
    editor/libeditor/tests/browserscope/,
    intl/icu/,
    ipc/chromium/src/third_party/,
    js/*.configure,
    gfx/angle/,
    gfx/harfbuzz,
    gfx/skia/,
    memory/moz.configure,
    mobile/android/*.configure,
    node_modules,
    security/nss/,
    testing/marionette/harness/marionette_harness/runner/mixins,
    testing/marionette/harness/marionette_harness/tests,
    testing/mochitest/pywebsocket,
    testing/mozharness/configs/test/test_malformed.py,
    tools/lint/test/files,
    tools/infer/test/*.configure,
    tools/crashreporter/*.configure,
    .ycm_extra_conf.py,

# See:
#   - http://flake8.pycqa.org/en/latest/user/error-codes.html
#   - http://pep8.readthedocs.io/en/latest/intro.html#configuration
ignore =
    # These should be triaged and either fixed or moved to the list below.
    F632, F633, F811, E117, W504, W605, W606,
    # These are intentionally disabled (not necessarily for good reason).
    #   F723: syntax error in type comment
    #       text contains quotes which breaks our custom JSON formatter
    F723, E121, E123, E126, E129, E133, E226, E241, E242, E402, E704, E741, W503,

per-file-ignores =
    ipc/ipdl/*: F403, F405
    testing/firefox-ui/**/__init__.py: F401
    testing/marionette/**/__init__.py: F401
    testing/mozharness/configs/*: E124, E127, E128, E131, E231, E261, E265, E266, E501, W391