| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
field" to 2214.
> Use higher precision to store the login database's date_created field.
>
> When a password is created or updated, its creation_date is saved in the login
> database with precision seconds. The same data is stored in the password sync
> server with precision microseconds. As a result, the two passwords always
> appear to differ.
>
> I've changed the code so that the creation_date is stored in the login database
> with precision microseconds, and I've created a migration to update the
> existing login database entries. Prior to the migration, creation_date was a
> time_t, which has precision seconds and counts from the UTC epoch. After the
> migration, creation_date has precision microseconds, and counts from Chrome's
> platform independent epoch.
>
> Note that the very first time passwords are all synced, they will still appear
> to differ since the login database lost 6 decimal places of precision. This CL
> makes it so that on the second sync (and subsequent syncs), the passwords will
> no longer differ.
>
> BUG=408800
>
> Review URL: https://codereview.chromium.org/702893005
>
> Cr-Commit-Position: refs/heads/master@{#303783}
BUG=408800
TBR=gcasto@chromium.org
Review URL: https://codereview.chromium.org/726993002
Cr-Commit-Position: refs/branch-heads/2214@{#48}
Cr-Branched-From: 03655fd3f6d72165dc3c9bd2c89807305316fe6c-refs/heads/master@{#303346}
|
|
|
|
|
|
|
|
|
|
| |
(backported from mojo)
R=jam@chromium.org
Review URL: https://codereview.chromium.org/709793002
Cr-Commit-Position: refs/heads/master@{#303341}
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/708743002
Cr-Commit-Position: refs/heads/master@{#303295}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since we crash on invalid fd in ScopedFD::Close(), memory corruption
in a base::File() object can cause crashes; perhaps long after the memory
is actually corrupt.
This CL puts a checksum in memory near the fd on POSIX systems. The
checksum is different enough to be very unlikely to be correct after a
random memory write, but also very cheap to compute, so we can have
most operations on a file double quickly double check its integrity,
for earlier (and more useful for debugging) crashes.
This is intended to help us debug issue 424562. Once we have found out
if and where memory is being corrupted, this instrumentation code should
be removed.
R=thakis@chromium.org,pasko@chromium.org
BUG=424562
Review URL: https://codereview.chromium.org/702473009
Cr-Commit-Position: refs/heads/master@{#303251}
|
|
|
|
|
|
|
|
|
|
|
| |
An as-small-as-possible change has been made to JSONReader to test
this feature.
R=jamesr@chromium.org
Review URL: https://codereview.chromium.org/685603005
Cr-Commit-Position: refs/heads/master@{#303245}
|
|
|
|
|
|
|
|
| |
BUG=401560
Review URL: https://codereview.chromium.org/712443003
Cr-Commit-Position: refs/heads/master@{#303240}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In Browser process, if profiler timing field trial is "Enabled", enable profiler timing.
This is necessary to measure the performance impact of enabling profiler timing on Android, and, perhaps, rolling out profiler timing enabling afterwards.
Profiler timing is currently disabled on Android because of performance concerns, but the impact on the standard perf metricts is not yet known, hence this CL.
BUG=315070
Review URL: https://codereview.chromium.org/693103005
Cr-Commit-Position: refs/heads/master@{#303100}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/689553002
Cr-Commit-Position: refs/heads/master@{#303085}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Now, the PNaCL toolchain for Non-SFI build supports MSG_TRUNC and
MSG_CTRUNC, so this CL enables the related code.
BUG=427625, 358465
TEST=Ran trybots.
Review URL: https://codereview.chromium.org/703033003
Cr-Commit-Position: refs/heads/master@{#303065}
|
|
|
|
|
|
|
|
|
|
| |
Record the hardware model on Mac e.g. MacPro1,1 .
BUG=429487
Review URL: https://codereview.chromium.org/699173002
Cr-Commit-Position: refs/heads/master@{#303054}
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds a check to crazy::FindStartOffsetOfFileInZipFile which
ensures that a file is uncompressed in the zip file.
BUG=390618
Review URL: https://codereview.chromium.org/705613003
Cr-Commit-Position: refs/heads/master@{#303041}
|
|
|
|
|
|
|
|
|
|
|
| |
Objective-C interfaces are disallowed in base because of a bug related
to plugins.
There are no plugins on iOS, and //base/ios is not compiled for other
platforms.
Review URL: https://codereview.chromium.org/708513002
Cr-Commit-Position: refs/heads/master@{#303007}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds a public method
SetTaskRunner(scoped_refptr<SingleThreadTaskRunner>) to base::Timer
which changes the task runner used by the timer
(ThreadTaskRunnerHandler::Get() by default).
This change will enable scheduling timers on the queues of the
Blink scheduler (see https://codereview.chromium.org/637303003/).
Most importantly, we plan to apply this to the shared timer in
blink::Platform.
BUG=
Review URL: https://codereview.chromium.org/637983003
Cr-Commit-Position: refs/heads/master@{#302947}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds a "Used no map executable support fallback" value to the
"ChromiumAndroidLinker.LibraryLoadFromApkStatus" histogram. To do this,
an extra argument is added to crazy_library_open_in_zip_file() which
enables the fallback due to lack of support for mapping the APK file with
executable permission (https://codereview.chromium.org/692593002/).
BUG=398425,390618
Review URL: https://codereview.chromium.org/697953003
Cr-Commit-Position: refs/heads/master@{#302926}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
nacl_helper_nonsfi.
Now, OS_NACL_SFI and OS_NACL_NONSFI are introduced to build/build_config.h, and these are preferred to __native_client_nonsfi__ and OS_NACL combination.
BUG=358465
TEST=Ran trybots.
Review URL: https://codereview.chromium.org/704513003
Cr-Commit-Position: refs/heads/master@{#302907}
|
|
|
|
|
|
|
|
|
|
| |
activation.
BUG=430233
Review URL: https://codereview.chromium.org/700933002
Cr-Commit-Position: refs/heads/master@{#302813}
|
|
|
|
|
|
|
|
|
|
| |
This change exports TrialExists to Java layer which is used in https://chrome-internal-review.googlesource.com/#/c/182195/ to check whether a trial actually exists or not.
BUG=412156
Review URL: https://codereview.chromium.org/700043002
Cr-Commit-Position: refs/heads/master@{#302773}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
jankiness
Previous instrumentations showed that URLRequestJob::NotifyHeadersComplete
alone is responsible for ~57.1 janks per hour in IO thread. I need to instrument
the code inside it to find out which part causes jank.
This is a mechanical change that adds instrumentation required to locate the
source of jankiness (i.e. a long-running fragment of code executed as a part of
the task that causes jank) in the code. See the bug for details on what kind of
jank we are after.
A number of similar CLs were landed, and none of them caused issues. They've
helped to find and fix janky code. The code of the instrumentation is highly
optimized and is not expected to affect performance. The code simply creates a
diagnostic task which is identical to ones created by PostTask or IPC message
handlers. The task gets created only in developer build and in Canary channel.
BUG=423948
Review URL: https://codereview.chromium.org/696483002
Cr-Commit-Position: refs/heads/master@{#302694}
|
|
|
|
|
|
|
|
| |
BUG=105388
Review URL: https://codereview.chromium.org/678263003
Cr-Commit-Position: refs/heads/master@{#302483}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds support for copying the Chromium library from the APK file
to a separate file.
This patch is largly based on https://codereview.chromium.org/673093005/.
BUG=390618
Review URL: https://codereview.chromium.org/684163002
Cr-Commit-Position: refs/heads/master@{#302456}
|
|
|
|
|
|
|
|
|
|
|
|
| |
These are mostly mechanical changes found while attempting to compile
the Android build with libc++.
BUG=427718
TBR=darin@chromium.org
Review URL: https://codereview.chromium.org/693773003
Cr-Commit-Position: refs/heads/master@{#302425}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is useful for instrumentations required for jankiness investigations. See the .h file comment for details.
In fact, one of my investigations is blocked on this feature.
BUG=401560
Review URL: https://codereview.chromium.org/683283004
Cr-Commit-Position: refs/heads/master@{#302324}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
fix some unittests.
FieldTrialTest.DisableProbability was using Time::NowFromSystemTime - 1 year to generate a date before the build time. Instead use GetBuildTime - 1 year.
TransportSecurityState::IsBuildTimely is using GetBuildTime to determine if the build is older than 10 weeks. Disable this and return a default value if DONT_EMBED_BUILD_METADATA is defined (i.e. The build time is invalid) and we're not doing an official build.
CL based on https://codereview.chromium.org/685123005/ with some additional fixes for the unittests.
BUG=314403
Review URL: https://codereview.chromium.org/695523002
Cr-Commit-Position: refs/heads/master@{#302316}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Use NSWindowDidChangeOcclusionStateNotification (new in 10.9
Mavericks) to detect when the application is hidden, the window is
minimized or obscured by another window.
Send the WasHidden to the WebContentsImpl, because this will
take into account tab capture.
When RenderWidgetHostViewMac::WasHidden is called, the previous
behavior was to destroy the BrowserCompositorViewMac. This has
the undesirable effect of creating a white flash when a hidden
window is re-exposed.
To prevent this, only destroy the BrowserCompositorViewMac when
the the RenderWidgetHostViewMac is removed from all NSWindows.
Manage the state of BrowserCompositorViewMac explicitly as being
Active (previously whenever it existed), Suspended (the new state
where it is kept around, but the RWH and DFH are hidden), and
Destroyed (previously wherever it did not exist).
Move some 10.7 APIs to base because they belong there, not in
content.
BUG=310374
Review URL: https://codereview.chromium.org/684343002
Cr-Commit-Position: refs/heads/master@{#302160}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:1 of https://codereview.chromium.org/689713006/)
Reason for revert:
This seems to have broken the Linux MSan test bot.
[ RUN ] FileUtilTest.WriteFileWithMode
../../base/files/file_util_unittest.cc:2164: Failure
Value of: status.st_mode & 0777
Actual: 384
Expected: mode
Which is: 420
[ FAILED ] FileUtilTest.WriteFileWithMode (1 ms)
Original issue's description:
> Allow POSIX callers to specify a new file's mode.
>
> Retain the tight 0640 mode for most existing callers, but honor the user's
> umask for downloaded files.
>
> This is a re-land of https://codereview.chromium.org/639253004/.
>
> BUG=409416, 362887
> TBR=asanka,mdempsky,rvargas
>
> Committed: https://crrev.com/931e35070097af9837f79ff19f8668e45b92ed3a
> Cr-Commit-Position: refs/heads/master@{#302113}
TBR=palmer@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=409416, 362887
Review URL: https://codereview.chromium.org/693493004
Cr-Commit-Position: refs/heads/master@{#302143}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Retain the tight 0640 mode for most existing callers, but honor the user's
umask for downloaded files.
This is a re-land of https://codereview.chromium.org/639253004/.
BUG=409416, 362887
TBR=asanka,mdempsky,rvargas
Review URL: https://codereview.chromium.org/689713006
Cr-Commit-Position: refs/heads/master@{#302113}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Clean up redundant PRESUBMIT files.
linux_redux has been 100% red since at least August 2014,
and is presumably not useful.
BUG=380709
R=sergiyb@chromium.org
Review URL: https://codereview.chromium.org/685363002
Cr-Commit-Position: refs/heads/master@{#302067}
|
|
|
|
|
|
|
|
| |
BUG=427526
Review URL: https://codereview.chromium.org/680923002
Cr-Commit-Position: refs/heads/master@{#302064}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/684313002
Cr-Commit-Position: refs/heads/master@{#301979}
|
|
|
|
|
|
|
|
| |
BUG=427633
Review URL: https://codereview.chromium.org/685303002
Cr-Commit-Position: refs/heads/master@{#301973}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit 3457f738b0dc2dd7d67fa84148d72066db9e0519.
Breaks FileUtilTest.WriteFileWithMode on Android.
BUG=409416,362887
Review URL: https://codereview.chromium.org/687163005
Cr-Commit-Position: refs/heads/master@{#301968}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This will disable the instrumentations in all channels except Canary, and developer build. Instrumentations are only enabled in Browser process.
This will prevent Stable from sending increased amounts of UMA logs. Also, this will let us observe "true" (untouched by instrumentations) jankiness numbers every week in Dev channel.
BUG=401560
Review URL: https://codereview.chromium.org/686963002
Cr-Commit-Position: refs/heads/master@{#301950}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Retain the tight 0640 mode for most existing callers, but honor the user's
umask for downloaded files.
BUG=409416,362887
R=asanka@chromium.org, mdempsky@chromium.org, rvargas@chromium.org
Review URL: https://codereview.chromium.org/639253004
Cr-Commit-Position: refs/heads/master@{#301932}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a reland of https://codereview.chromium.org/669673004 after fixes.
This doesn't do anything yet, mostly adds supporting infrastructure.
R=lliabraa@chromium.org, mark@chromium.org
TBR=darin
BUG=426870
Review URL: https://codereview.chromium.org/688663003
Cr-Commit-Position: refs/heads/master@{#301850}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Readability only. No behavioural effect.
Fixes:
** Presubmit Warnings **
base/android/java/src/org/chromium/base/library_loader/LibraryLoader.java:188:96: '+' should be on a new line.
base/android/java/src/org/chromium/base/library_loader/LibraryLoader.java:242:66: '+' should be on a new line.
base/android/java/src/org/chromium/base/library_loader/LibraryLoader.java:334:94: '?' should be on a new line.
base/android/java/src/org/chromium/base/library_loader/Linker.java:245:69: '?' should be on a new line.
base/android/java/src/org/chromium/base/library_loader/Linker.java:343:67: '||' should be on a new line.
base/android/java/src/org/chromium/base/library_loader/Linker.java:509:66: '+' should be on a new line.
base/android/java/src/org/chromium/base/library_loader/Linker.java:764:80: '+' should be on a new line.
base/android/java/src/org/chromium/base/library_loader/Linker.java:836:83: '+' should be on a new line.
** Presubmit ERRORS **
base/android/java/src/org/chromium/base/library_loader/LibraryLoader.java:189: '"retrying without"' have incorrect indentation level 38, expected level should be 40.
base/android/java/src/org/chromium/base/library_loader/LibraryLoader.java:212: '&&' have incorrect indentation level 32, expected level should be 36.
base/android/java/src/org/chromium/base/library_loader/LibraryLoader.java:223: '&&' have incorrect indentation level 20, expected level should be 24.
base/android/java/src/org/chromium/base/library_loader/LibraryLoader.java:224: '&&' have incorrect indentation level 20, expected level should be 24.
base/android/java/src/org/chromium/base/library_loader/LibraryLoader.java:226: 'context' have incorrect indentation level 24, expected level should be 28.
base/android/java/src/org/chromium/base/library_loader/Linker.java:643: 'String' have incorrect indentation level 18, expected level should be 20.
base/android/java/src/org/chromium/base/library_loader/Linker.java:888: 'String' have incorrect indentation level 8, expected level should be 12.
base/android/java/src/org/chromium/base/library_loader/Linker.java:889: 'String' have incorrect indentation level 8, expected level should be 12.
base/android/java/src/org/chromium/base/library_loader/Linker.java:890: 'long' have incorrect indentation level 8, expected level should be 12.
base/android/java/src/org/chromium/base/library_loader/Linker.java:891: 'LibInfo' have incorrect indentation level 8, expected level should be 12.
base/android/java/src/org/chromium/base/library_loader/Linker.java:1082: '"org.chromium.base.android.linker.shared_relros"' have incorrect indentation level 8, expected level should be 12.
BUG=
Review URL: https://codereview.chromium.org/672363003
Cr-Commit-Position: refs/heads/master@{#301818}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Define operator<<(ostream&,Time) to permit Time, TimeDelta and TimeTicks
types to be used in logging assertions such as DCHECK_EQ().
BUG=425941
TEST=base_unittests
Review URL: https://codereview.chromium.org/669083002
Cr-Commit-Position: refs/heads/master@{#301815}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
file.
The following histogram is added:
ChromiumAndroidLinker.LibraryAlignedInApk
Whether the Chromium library is page aligned in the APK file. The
functionality is checked and reported during every Chromium browser process
start up if direct library loading from the APK file is enabled.
BUG=
Review URL: https://codereview.chromium.org/684453003
Cr-Commit-Position: refs/heads/master@{#301813}
|
|
|
|
|
|
|
|
|
|
|
| |
Namely, -Winconsistent-missing-override. No behavior change.
BUG=428099
TBR=blundell
Review URL: https://codereview.chromium.org/652233006
Cr-Commit-Position: refs/heads/master@{#301761}
|
|
|
|
|
|
|
|
| |
BUG=380419
Review URL: https://codereview.chromium.org/686553003
Cr-Commit-Position: refs/heads/master@{#301735}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
and suppress one very noisy warning.
When win_analyze=1 is specified then /analyze is added to the command
line to enable static code analysis with VC++.
WarnAsErrors is set to false to allow reporting of all errors.
Additional, WX- is added to disable WarnAsErrors more robustly, necessary because some projects override WarnAsErrors.
Additionally, many noisy but low-value warnings are suppressed.
Finally, _USING_V110_SDK71_ is un-set when win_analyze is true because
this is incompatible with /analyze.
The change to callback_internal.h suppresses an extremely noisy warning about a potentially incorrect use of sizeof():
base\callback_internal.h(80) : warning C6334: sizeof operator applied to an expression with an operator might yield unexpected results: Parentheses can be used to disambiguate certain usages.
With this change almost all of Chrome can be built with high but useful levels of warnings.
bug=427616
Review URL: https://codereview.chromium.org/676743003
Cr-Commit-Position: refs/heads/master@{#301723}
|
|
|
|
|
|
|
|
|
|
|
| |
No behavior change.
BUG=427584
TBR=phajdan
Review URL: https://codereview.chromium.org/683963002
Cr-Commit-Position: refs/heads/master@{#301710}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
and Canary channel.
This introduces TrackingProfile class that is equivalent to ScopedProfile what tracking is enabled and does nothing otherwise.
There is an idea to remove the single-parametered ScopedProfile constructor, but for this CL, I’m leaving it. Otherwise, I’d have to replace ~100 ScopedProfile instantiations throughout the code with TrackingProfile, and the substantial changes would be lost in these changes. I’ll do this in a separate CL. I replaced one ScopedProfile just to make sure this will compile.
BUG=401560
TBR=asanka@chromium.org
Review URL: https://codereview.chromium.org/663093005
Cr-Commit-Position: refs/heads/master@{#301695}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
On failure to load libchromium_android_linker.so, we retry with
libchromium_android_linker.cr.so. If the latter fails then only
the name 'libchromium_android_linker.cr.so' appears in the logcat.
This is confusing where (commonly) this is not a components build.
Add a warning to clarify that libchromium_android_linker.cr.so
is a load retry with a different name.
BUG=
Review URL: https://codereview.chromium.org/683163003
Cr-Commit-Position: refs/heads/master@{#301654}
|
|
|
|
|
|
|
|
|
|
|
|
| |
files as opposed to being statically linked to the binary.
This is not currently supported on any architecture.
BUG=421063
Review URL: https://codereview.chromium.org/594603003
Cr-Commit-Position: refs/heads/master@{#301595}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
On low-end devices, if Chrome was upgraded from a previous version,
Chrome decides to use normal mode to keep user experience consistent.
This is done by force '--low-end-device-mode=0' on both browser and render
processes.
However, when crazy linker is used, Linker.isUsed() triggers
SysUtils.detectLowEndDevice() call which checks CommandLine first.
Existing code invokes Linker.isUsed() before initilizing CommandLine,
which causes the render process ignores '--low-end-device-mode=0'.
This results the browser in normal mode, but render is in low-end device mode.
BUG=424228
Review URL: https://codereview.chromium.org/665273003
Cr-Commit-Position: refs/heads/master@{#301581}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/686573003
Cr-Commit-Position: refs/heads/master@{#301555}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/645343004
Cr-Commit-Position: refs/heads/master@{#301475}
|
|
|
|
|
|
|
|
|
|
|
| |
Not used except for cloud print.
BUG=426573
TBR=rvargas
Review URL: https://codereview.chromium.org/681533002
Cr-Commit-Position: refs/heads/master@{#301473}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ContentSettingsHandler is now made aware of Off The Record profiles
and will observe them as well, correcting bugs where changes from the
settings webUI would function incorrectly.
This is a rework of a previously landed fix [fix] which was reverted due to
a crash bug [crash] when OTR profiles already existed before settings
were opened.
[fix] https://codereview.chromium.org/585953003
[crash] https://code.google.com/p/chromium/issues/detail?id=417597
BUG=425079, 418931
Committed: https://crrev.com/c9ba380c606442a025a38eb67f4d35c65b1a293c
Cr-Commit-Position: refs/heads/master@{#301273}
Review URL: https://codereview.chromium.org/676083003
Cr-Commit-Position: refs/heads/master@{#301461}
|
|
|
|
|
|
|
|
|
| |
BUG=none
NOTRY=true
Review URL: https://codereview.chromium.org/686463002
Cr-Commit-Position: refs/heads/master@{#301417}
|