Content Modularization Project: Network Service |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Issue descriptiondesign doc: https://docs.google.com/document/d/1wAHLw9h7gGuqJNCgG1mP1BmLtCGfZ2pys-PdZQ1vg7M/edit?pref=2&pli=1#heading=h.w406cmqiibno The initial goal is to isolate the networking related code in content/browser to a separate component. Then wrap it with Mojo interfaces such that it's possible to run it in a separate & sandboxed process. ⛆ |
|
|
Showing comments 165 - 264
of 264
Older ›
,
Oct 27 2017
,
Oct 31 2017
,
Nov 6 2017
,
Nov 7 2017
,
Nov 7 2017
Apologies, applied the wrong component in bulk.
,
Nov 7 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5eede2ad7561d98b03a4c4513a95a78f2fbe09c7 commit 5eede2ad7561d98b03a4c4513a95a78f2fbe09c7 Author: John Abd-El-Malek <jam@chromium.org> Date: Tue Nov 07 18:42:51 2017 Update network service browser_tests filter with new passing tests. A number of changes landed last week that fixed basic failures. BUG=598073 NOTRY=true TBR=mmenke@chromium.org Change-Id: I46d69a20556b7ea7fdab73ff081eb55197c814eb Reviewed-on: https://chromium-review.googlesource.com/756828 Reviewed-by: John Abd-El-Malek <jam@chromium.org> Commit-Queue: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#514520} [modify] https://crrev.com/5eede2ad7561d98b03a4c4513a95a78f2fbe09c7/testing/buildbot/filters/mojo.fyi.network_browser_tests.filter
,
Nov 7 2017
Migrating S13N meta bugs to Type=Task, so that they can be distinguished from technical work.
,
Nov 8 2017
,
Nov 10 2017
,
Nov 13 2017
,
Nov 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e634d9e791f279359915d1ebd7b76b9a09375bb9 commit e634d9e791f279359915d1ebd7b76b9a09375bb9 Author: John Abd-El-Malek <jam@chromium.org> Date: Mon Nov 13 22:29:20 2017 Annotate more browser_tests failures with network service. BUG=598073 TBR=yzshen@chromium.org NOTRY=true Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: Ibbafcf79f7bf10d1b96f252c94f1b90ec668cf6d Reviewed-on: https://chromium-review.googlesource.com/767025 Commit-Queue: John Abd-El-Malek <jam@chromium.org> Reviewed-by: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#516080} [modify] https://crrev.com/e634d9e791f279359915d1ebd7b76b9a09375bb9/testing/buildbot/filters/mojo.fyi.network_browser_tests.filter
,
Nov 14 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/18498ddacc6594ad6367fb74bd7a5918301cc1e2 commit 18498ddacc6594ad6367fb74bd7a5918301cc1e2 Author: John Abd-El-Malek <jam@chromium.org> Date: Tue Nov 14 02:28:23 2017 More updates to network service browser_tests filter file after r516080 BUG=598073 TBR=yzshen@chromium.org NOTRY=true Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: I6008e29f84337de9da48cbb8b1152f521107292d Reviewed-on: https://chromium-review.googlesource.com/768096 Commit-Queue: John Abd-El-Malek <jam@chromium.org> Reviewed-by: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#516150} [modify] https://crrev.com/18498ddacc6594ad6367fb74bd7a5918301cc1e2/testing/buildbot/filters/mojo.fyi.network_browser_tests.filter
,
Nov 15 2017
,
Nov 21 2017
,
Nov 22 2017
,
Nov 29 2017
,
Nov 29 2017
,
Nov 29 2017
,
Nov 29 2017
,
Nov 29 2017
,
Nov 29 2017
,
Nov 29 2017
,
Nov 29 2017
,
Dec 4 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8067cc0f4a81a3b5c4ca3fe6014481e7b78ade32 commit 8067cc0f4a81a3b5c4ca3fe6014481e7b78ade32 Author: John Abd-El-Malek <jam@chromium.org> Date: Mon Dec 04 23:23:20 2017 Add a feature flag to run network service in-process. This helps with debugging. Bug: 598073 Change-Id: I72d92c5b4870e4b1e985ae1a2442980b08dbe34c Reviewed-on: https://chromium-review.googlesource.com/806800 Reviewed-by: Ryan Hamilton <rch@chromium.org> Reviewed-by: Ken Rockot <rockot@chromium.org> Commit-Queue: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#521502} [modify] https://crrev.com/8067cc0f4a81a3b5c4ca3fe6014481e7b78ade32/chrome/browser/about_flags.cc [modify] https://crrev.com/8067cc0f4a81a3b5c4ca3fe6014481e7b78ade32/chrome/browser/flag_descriptions.cc [modify] https://crrev.com/8067cc0f4a81a3b5c4ca3fe6014481e7b78ade32/chrome/browser/flag_descriptions.h [modify] https://crrev.com/8067cc0f4a81a3b5c4ca3fe6014481e7b78ade32/components/network_session_configurator/common/BUILD.gn [modify] https://crrev.com/8067cc0f4a81a3b5c4ca3fe6014481e7b78ade32/components/network_session_configurator/common/network_features.h [add] https://crrev.com/8067cc0f4a81a3b5c4ca3fe6014481e7b78ade32/components/network_session_configurator/common/network_session_configurator_export.h [modify] https://crrev.com/8067cc0f4a81a3b5c4ca3fe6014481e7b78ade32/components/network_session_configurator/common/network_switches.h [modify] https://crrev.com/8067cc0f4a81a3b5c4ca3fe6014481e7b78ade32/content/browser/service_manager/service_manager_context.cc [modify] https://crrev.com/8067cc0f4a81a3b5c4ca3fe6014481e7b78ade32/content/public/common/content_features.cc [modify] https://crrev.com/8067cc0f4a81a3b5c4ca3fe6014481e7b78ade32/content/public/common/content_features.h [modify] https://crrev.com/8067cc0f4a81a3b5c4ca3fe6014481e7b78ade32/tools/metrics/histograms/enums.xml
,
Dec 13 2017
,
Dec 20 2017
,
Dec 22 2017
,
Dec 28 2017
,
Jan 10 2018
,
Jan 17 2018
,
Jan 17 2018
,
Jan 18 2018
,
Jan 18 2018
,
Jan 19 2018
,
Feb 8 2018
,
Feb 8 2018
,
Feb 8 2018
,
Feb 9 2018
,
Feb 16 2018
,
Feb 28 2018
,
Mar 6 2018
,
Mar 12 2018
,
Mar 15 2018
,
Mar 15 2018
,
Mar 22 2018
,
Mar 22 2018
,
Mar 22 2018
,
Mar 22 2018
,
Mar 22 2018
,
Mar 23 2018
,
Mar 28 2018
,
Mar 28 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7f520a8495ee33d9e4be72be42adab8efd0b96dd commit 7f520a8495ee33d9e4be72be42adab8efd0b96dd Author: Matt Menke <mmenke@chromium.org> Date: Wed Mar 28 21:38:37 2018 Add a note to URLFetcher that it shouldn't be use in content embedders. Also add a presubmit warning to trigger on new/modified uses of URLFetcher. Bug: 746977 , 598073 Change-Id: I962a9f9bdb0c8c533f5cc4e7ca5a919f917a89ca Reviewed-on: https://chromium-review.googlesource.com/984512 Reviewed-by: Helen Li <xunjieli@chromium.org> Reviewed-by: Daniel Cheng <dcheng@chromium.org> Commit-Queue: Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#546605} [modify] https://crrev.com/7f520a8495ee33d9e4be72be42adab8efd0b96dd/PRESUBMIT.py [modify] https://crrev.com/7f520a8495ee33d9e4be72be42adab8efd0b96dd/net/url_request/url_fetcher.h
,
Mar 30 2018
,
Mar 30 2018
,
Apr 5 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/59716d015d9d4a7870990a83419d7b5c3806a213 commit 59716d015d9d4a7870990a83419d7b5c3806a213 Author: Matt Menke <mmenke@chromium.org> Date: Thu Apr 05 12:45:53 2018 Make the net::URLFetcher presubmit check not require a presubmit bypass I misunderstood the boolean parameter. This check is intended to be advisory, rather than something that blocks landing CLs. Bug: 746977 , 598073 Change-Id: Ib7695f7bea15f1613a4f9e73e0258fa231340161 Reviewed-on: https://chromium-review.googlesource.com/992393 Reviewed-by: Dirk Pranke <dpranke@chromium.org> Commit-Queue: Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#548394} [modify] https://crrev.com/59716d015d9d4a7870990a83419d7b5c3806a213/PRESUBMIT.py
,
Apr 20 2018
,
Apr 26 2018
,
May 2 2018
,
May 2 2018
,
May 2 2018
,
May 3 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4b7d02f22b70c27f32fa5508f785ba310d08aad3 commit 4b7d02f22b70c27f32fa5508f785ba310d08aad3 Author: John Abd-El-Malek <jam@chromium.org> Date: Thu May 03 17:46:35 2018 Don't instantiate ChromeNetworkDelegate when network service is enabled. Bug: 598073 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: I59722f8aa6c1946c3f2cf00bbbad0d8c96e0d12f Reviewed-on: https://chromium-review.googlesource.com/1041486 Reviewed-by: Matt Menke <mmenke@chromium.org> Commit-Queue: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#555800} [modify] https://crrev.com/4b7d02f22b70c27f32fa5508f785ba310d08aad3/chrome/browser/io_thread.cc [modify] https://crrev.com/4b7d02f22b70c27f32fa5508f785ba310d08aad3/chrome/browser/profiles/profile_io_data.cc [modify] https://crrev.com/4b7d02f22b70c27f32fa5508f785ba310d08aad3/testing/buildbot/filters/mojo.fyi.network_browser_tests.filter
,
May 7 2018
,
May 7 2018
,
May 7 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ad0d50ec591d2f2e38dd76741c89c9e5743c1022 commit ad0d50ec591d2f2e38dd76741c89c9e5743c1022 Author: John Abd-El-Malek <jam@chromium.org> Date: Mon May 07 16:01:12 2018 Add network service tests to Mojo Windows FYI bot. Bug: 598073 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: I370ee1b06eb256072f502ee946e02b5032ec22fa Reviewed-on: https://chromium-review.googlesource.com/1035202 Reviewed-by: Dirk Pranke <dpranke@chromium.org> Reviewed-by: Jonathan Ross <jonross@chromium.org> Commit-Queue: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#556456} [modify] https://crrev.com/ad0d50ec591d2f2e38dd76741c89c9e5743c1022/chrome/browser/extensions/api/streams_private/streams_private_apitest.cc [modify] https://crrev.com/ad0d50ec591d2f2e38dd76741c89c9e5743c1022/testing/buildbot/chromium.fyi.json [modify] https://crrev.com/ad0d50ec591d2f2e38dd76741c89c9e5743c1022/testing/buildbot/filters/mojo.fyi.network_browser_tests.filter [modify] https://crrev.com/ad0d50ec591d2f2e38dd76741c89c9e5743c1022/testing/buildbot/test_suites.pyl [modify] https://crrev.com/ad0d50ec591d2f2e38dd76741c89c9e5743c1022/testing/buildbot/waterfalls.pyl
,
May 7 2018
,
May 8 2018
,
May 8 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/43db2f0f70498802a6b9a12a0a9fdc4fecd97aaf commit 43db2f0f70498802a6b9a12a0a9fdc4fecd97aaf Author: John Abd-El-Malek <jam@chromium.org> Date: Tue May 08 18:08:09 2018 Fix WebViewNewWindowInteractiveTest.NewWindow_WebRequestRemoveElement flake on Windows with network service. This passes all the time on Linux with network service, but fails most of the time on Windows. The problem is that due to timing differences with network service, |webview| is torn down while it still has its navigation in-progress. WebViewGuest::DidFinishNavigation then synthesizes a LoadAbort called and (I'm not sure why) onNewWindow gets called a second time. Bug: 598073 Change-Id: I7ca2acd5e4ec7ddbfc25f94184d83e1abdb25545 Reviewed-on: https://chromium-review.googlesource.com/1049011 Reviewed-by: Fady Samuel <fsamuel@chromium.org> Commit-Queue: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#556876} [modify] https://crrev.com/43db2f0f70498802a6b9a12a0a9fdc4fecd97aaf/chrome/test/data/extensions/platform_apps/web_view/newwindow/embedder.js
,
May 8 2018
,
May 9 2018
,
May 9 2018
,
May 9 2018
,
May 9 2018
,
May 9 2018
,
May 9 2018
,
May 9 2018
,
May 9 2018
,
May 16 2018
,
May 16 2018
,
May 16 2018
,
May 16 2018
,
May 16 2018
,
May 16 2018
,
May 16 2018
,
May 22 2018
,
May 22 2018
,
May 24 2018
,
May 30 2018
,
Jun 1 2018
,
Jun 27 2018
,
Aug 22
//NOLAUNCH - We are keeping this to the canary channel only in M70 and will not go beyond that. We will create a separate launch bug when if we decide to go beta in M71.
,
Aug 22
,
Aug 27
,
Aug 27
,
Aug 27
,
Oct 3
,
Oct 3
,
Oct 4
,
Oct 23
,
Nov 13
,
Jan 10
Showing comments 165 - 264
of 264
Older ›
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
►
Sign in to add a comment |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||