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

Issue 654804 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jan 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows , Mac
Pri: 3
Type: Bug



Sign in to add a comment

FATAL: extension_updater.cc(228) Check failed: target_delay >= TimeDelta::FromSeconds(1)

Project Member Reported by elawrence@chromium.org, Oct 11 2016

Issue description

Version: 56.2880
OS: Win7

What steps will reproduce the problem?

I ran c:\src\c\src\out\default\chrome.exe --user-data-dir=C:\temp\ browsed a bit, and closed the browser.

I then ran C:\src\c\src\out\default\chrome.exe without any arguments and it crashed on startup:

---------------------------
Fatal error
---------------------------
[5196:4168:1011/121941:FATAL:extension_updater.cc(228)] Check failed: target_delay >= TimeDelta::FromSeconds(1). 
Backtrace:
	base::win::RegistryValueIterator::ValueSize [0x1007B857+336407]
	base::win::RegistryValueIterator::ValueSize [0x100CAC7B+661051]
	policy::PolicyDetails::operator= [0x061428A1+47211930]
	policy::PolicyDetails::operator= [0x06142E5F+47213400]
	policy::PolicyDetails::operator= [0x060F2BD3+46885068]
	policy::PolicyDetails::operator= [0x062502CE+48316359]
	policy::PolicyDetails::operator= [0x060EECDA+46868947]
	policy::PolicyDetails::operator= [0x064BC4B0+50856361]
	policy::PolicyDetails::operator= [0x064BC956+50857551]
	policy::PolicyDetails::operator= [0x0496346C+22180197]
	policy::PolicyDetails::operator= [0x04963226+22179615]
	policy::PolicyDetails::operator= [0x0496116C+22171237]
	policy::PolicyDetails::operator= [0x0496252D+22176294]
	policy::PolicyDetails::operator= [0x04965565+22188638]
	policy::PolicyDetails::operator= [0x07163A7C+64124789]
	policy::PolicyDetails::operator= [0x04B2F4BB+24064436]
	policy::PolicyDetails::operator= [0x04B33DDD+24083158]
	policy::PolicyDetails::operator= [0x04B33104+24079869]
	content::BrowserContext::GetStoragePartitionForSite [0x115EB037+15529059]
	content::BrowserContext::GetStoragePartitionForSite [0x115E1512+15489342]
	content::BrowserContext::GetStoragePartitionForSite [0x115E18A4+15490256]
	content::BrowserContext::GetStoragePartitionForSite [0x115E1ACB+15490807]
	content::BrowserContext::GetStoragePartitionForSite [0x115EB4E4+15530256]
	content::BrowserContext::GetStoragePartitionForSite [0x11D41CAF+23224539]
	content::BrowserContext::GetStoragePartitionForSite [0x1225DC1E+28581962]
	content::BrowserContext::GetStoragePartitionForSite [0x115E78A9+15514837]
	content::BrowserContext::GetStoragePartitionForSite [0x115F0C4E+15552634]
	content::BrowserContext::GetStoragePartitionForSite [0x115DEE4F+15479419]
	content::BrowserContext::GetStoragePartitionForSite [0x12EE7757+41728899]
	content::BrowserContext::GetStoragePartitionForSite [0x12EE7618+41728580]
	content::BrowserContext::GetStoragePartitionForSite [0x12EE53F4+41719840]
	policy::PolicyDetails::operator= [0x036EAD92+2812555]
	policy::PolicyDetails::operator= [0x0042E00F+114020]
	policy::PolicyDetails::operator= [0x00428E1C+93041]
	policy::PolicyDetails::operator= [0x004C7FEE+744771]
	policy::PolicyDetails::operator= [0x004C7E50+744357]
	policy::PolicyDetails::operator= [0x004C7CED+744002]
	policy::PolicyDetails::operator= [0x004C8008+744797]
	BaseThreadInitThunk [0x765D336A+18]
	RtlInitializeExceptionChain [0x77C59902+99]
	RtlInitializeExceptionChain [0x77C598D5+54]
---------------------------
OK   
---------------------------

 
Perhaps this:

  // Read the persisted next check time, and use that if it isn't in the past
  // or too far in the future (this can happen with system clock changes).
  if (saved_next > now &&
      saved_next < now + TimeDelta::FromSeconds(frequency_seconds_)) {
    return saved_next - now;
  }

Needs to be something like:

  // Read the persisted next check time, and use that if it isn't in the past
  // or too far in the future (this can happen with system clock changes).
  if (saved_next > now + TimeDelta::FromSeconds(1) &&
      saved_next < now + TimeDelta::FromSeconds(frequency_seconds_)) {
    return saved_next - now;
  }

?
Owner: asargent@chromium.org
Status: Assigned (was: Untriaged)
Labels: OS-Mac
Just ran into this on Mac.
Chromium build from: f518237b7651d1eeb115bffcc51f11ee0e9078cc with some minor [unrelated] changes.

Wasn't doing anything special. Had launched recent builds of Chromium many times before.
Status: Started (was: Assigned)
FYI, I have a CL fixing this going through review, hoping to land soon:

https://codereview.chromium.org/2562963003/



Status: Fixed (was: Started)
The above referenced CL landed a while ago - not sure why the commit bot didn't notify on the bug, but marking as Fixed. 

Sign in to add a comment