New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 766077 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Sep 19
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Handle Background Fetch in WebView

Project Member Reported by delph...@chromium.org, Sep 18 2017

Issue description

Background Fetch currently is not supported WebView and so will not function correctly.

Before we ship, we should either:
a) Enable support for it directly.
b) Explicitly disable it so that WebView does not act if it exists with fetches mysteriously failing.

Bug filed as per  comments here:
https://chromium-review.googlesource.com/c/chromium/src/+/647751/18
 
Components: Blink>BackgroundFetch

Comment 2 by peter@chromium.org, Sep 25 2017

Cc: timvolod...@chromium.org
We'll take option (b), as this boils down to the same reason that the Push API, Notification API, Background Sync and co are not supported: they need input from the hosting app, for which we need a dedicated mechanism.
Owner: ----
Status: Available (was: Assigned)
Labels: BlocksMVP
Project Member

Comment 5 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c9354ad9b7bffc134f9b04667a564d74598e8765

commit c9354ad9b7bffc134f9b04667a564d74598e8765
Author: Rayan Kanso <rayankans@chromium.org>
Date: Tue Sep 04 13:47:21 2018

[Background Fetch] Disable Background Fetch on WebView.

Bug:  766077 
Change-Id: Ie5cf7a94036dfa74787465ef210510b2b5993ed9
Reviewed-on: https://chromium-review.googlesource.com/1202209
Reviewed-by: Tobias Sargeant <tobiasjs@chromium.org>
Reviewed-by: Mugdha Lakhani <nator@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Commit-Queue: Rayan Kanso <rayankans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#588503}
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/android_webview/lib/aw_main_delegate.cc
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/content/browser/renderer_host/render_process_host_impl.cc
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/content/child/runtime_features.cc
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/content/public/common/content_switches.cc
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/content/public/common/content_switches.h
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/third_party/blink/public/platform/web_runtime_features.h
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/third_party/blink/renderer/platform/exported/web_runtime_features.cc

Project Member

Comment 6 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c9354ad9b7bffc134f9b04667a564d74598e8765

commit c9354ad9b7bffc134f9b04667a564d74598e8765
Author: Rayan Kanso <rayankans@chromium.org>
Date: Tue Sep 04 13:47:21 2018

[Background Fetch] Disable Background Fetch on WebView.

Bug:  766077 
Change-Id: Ie5cf7a94036dfa74787465ef210510b2b5993ed9
Reviewed-on: https://chromium-review.googlesource.com/1202209
Reviewed-by: Tobias Sargeant <tobiasjs@chromium.org>
Reviewed-by: Mugdha Lakhani <nator@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Commit-Queue: Rayan Kanso <rayankans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#588503}
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/android_webview/lib/aw_main_delegate.cc
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/content/browser/renderer_host/render_process_host_impl.cc
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/content/child/runtime_features.cc
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/content/public/common/content_switches.cc
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/content/public/common/content_switches.h
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/third_party/blink/public/platform/web_runtime_features.h
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/third_party/blink/renderer/platform/exported/web_runtime_features.cc

Project Member

Comment 7 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c9354ad9b7bffc134f9b04667a564d74598e8765

commit c9354ad9b7bffc134f9b04667a564d74598e8765
Author: Rayan Kanso <rayankans@chromium.org>
Date: Tue Sep 04 13:47:21 2018

[Background Fetch] Disable Background Fetch on WebView.

Bug:  766077 
Change-Id: Ie5cf7a94036dfa74787465ef210510b2b5993ed9
Reviewed-on: https://chromium-review.googlesource.com/1202209
Reviewed-by: Tobias Sargeant <tobiasjs@chromium.org>
Reviewed-by: Mugdha Lakhani <nator@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Commit-Queue: Rayan Kanso <rayankans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#588503}
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/android_webview/lib/aw_main_delegate.cc
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/content/browser/renderer_host/render_process_host_impl.cc
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/content/child/runtime_features.cc
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/content/public/common/content_switches.cc
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/content/public/common/content_switches.h
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/third_party/blink/public/platform/web_runtime_features.h
[modify] https://crrev.com/c9354ad9b7bffc134f9b04667a564d74598e8765/third_party/blink/renderer/platform/exported/web_runtime_features.cc

Status: Fixed (was: Available)

Sign in to add a comment