| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=
Review URL: https://codereview.chromium.org/25657007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@226593 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Print a usage message if called with no (valid) arguments.
* Determine the script name by stripping before either 'base' or
'third_party', or strip nothing if neither name can be found in
the path.
* Allow overriding the script name, and use this in the tests to be
path-agnostic.
TBR=rmcilroy@chromium.org
Review URL: https://codereview.chromium.org/23702057
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@226503 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ObserverListThreadSafe will call PostTask on all observer callbacks,
even those on the same thread. So correct the comment in
MemoryPressureListener which states callbacks on the same thread as the
system broadcast are called synchronously.
BUG=
NOTRY=true
Review URL: https://codereview.chromium.org/25676002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@226497 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=298036
NOTRY=True
Review URL: https://codereview.chromium.org/24533002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@226473 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=covered
Review URL: https://codereview.chromium.org/25286002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@226421 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
non-Mac non-NaCl posix
Both use a relative time, and it's important that this relative time is consistent with the wall clock when the system time gets adjusted (e.g. NTP, tlsdate, etc.). The monotonic clock, when available, has that property. Unfortunately, by default, pthread_cond_timedwait takes the system time which doesn't have that property.
On Linux/Chrome OS, we use pthread_condattr_setclock which lets us use the monotonic clock. On Android, we use the non-portable pthread_cond_timedwait_monotonic_np which has the same effect.
Unfortunately, neither is supported by NaCl.
Mac can use the non-portable pthread_cond_timedwait_relative_np which uses a relative time.
BUG=293736
R=thakis@chromium.org
Review URL: https://codereview.chromium.org/24158005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@226378 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
TraceEvent's copy constructor has a side-effect that the ownership
of the convertable values will be passed from the source TraceEvent
to the target TraceEvent.
Handle ECHO_TO_CONSOLE before the event is added to the buffer.
The copy constructor will be obsoleted soon to avoid such issues.
BUG=302676
TEST=Run cc_unittests with --trace_to_console
Review URL: https://codereview.chromium.org/25453005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@226319 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=299282
R=sky@chromium.org
Review URL: https://codereview.chromium.org/24996007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@226025 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Reason: PerfTimer is used by production code, so it doesn't make sense to have
it as test only.
- Rename it to ElapsedTimer.
- Move into base namespace.
- Move into timer/ directory.
BUG=None
TEST=None, no functional change
R=brettw@chromium.org
TBR=grt@chromium.org # for chrome_frame
Review URL: https://codereview.chromium.org/24265005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225898 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also, use lsb-release/CHROMEOS_RELEASE_NAME instead of the
USER env variable.
This CL depends on https://codereview.chromium.org/23588009/
BUG=270135
For trivial base/ changes:
R=derat@chromium.org, satorux@chromium.org, sky@chromium.org
TBR=brettw@chromium.org
Review URL: https://codereview.chromium.org/23904025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225761 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
- skip dependent tests if a PRE_ test failed
- simplify ParallelTestLauncher code now that we don't need sequencing
BUG=236893
R=jam@chromium.org
Review URL: https://codereview.chromium.org/24892002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225741 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=None
R=thakis@chromium.org
Review URL: https://codereview.chromium.org/24767002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225681 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This is to share common x11 code for ui/x and chromeos/display.
R=thakis@chromium.org
Review URL: https://codereview.chromium.org/24160005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225662 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
NOTRY=true
Review URL: https://codereview.chromium.org/24899002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225630 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=266903
TBR=derat@chromium.org, petewil@chromium.org
Review URL: https://codereview.chromium.org/24878002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225596 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This moves all parsing and caching of the contents of /etc/lsb-release
to base::SysInfo. This eliminates redundant parsing from:
* LsbReleaseLogSource
* VersionLoader
* StatisticsProvider
BUG=266903
For chrome_main_deleagate.cc (code removal only)
R=brettw@chromium.org, derat@chromium.org, pneubeck@chromium.org, xiyuan@chromium.org
TBR=cpu@chromium.org
Review URL: https://codereview.chromium.org/23588009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225581 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TBR=darin@chromium.org
R=ajwong@chromium.org, darin@chromium.org
Review URL: https://codereview.chromium.org/24648002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225534 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
OS X
BUG=298197
TBR=mark@chromium.org
Review URL: https://codereview.chromium.org/24742002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225474 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Other projects will be using the JNI bindings.
- Move GetMethodIDFromClassName down to content/browser/renderer_host/java,
the only place where it's used.
- Remove a few methods that are no longer in use.
- Trim down dependencies.
BUG=
Review URL: https://codereview.chromium.org/23835020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225471 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The tests were removed in
https://codereview.chromium.org/10151017
and
https://codereview.chromium.org/10830318
but are still referenced as friends.
BUG=143239
Review URL: https://codereview.chromium.org/24571002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225445 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This data can be collected on ChromiumOS when the user consents to it; this should help us collect more data about what shouldn't be happening in prod that happens anyways.
BUG=288014
Review URL: https://codereview.chromium.org/24313004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225383 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
It turns out they may happen.
BUG=236893
R=sky@chromium.org
Review URL: https://codereview.chromium.org/24472002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225283 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
slashes as arguments rather than switches.
The old command line parsing is still run so some code could get confused later since / will also be treated as a switch if you ask for it. This seems not as likely to happen that the label will match a switch. If it does, we'll have to change the CommandLine class to make the "/" behavior optional.
Fix a "desc" crash attributing blame due to a null pointer.
BUG=
R=scottmg@chromium.org, thakis@chromium.org
Review URL: https://codereview.chromium.org/24613003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225277 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
[[Making changes to '_toolset=="target"' section in system.gyp to fix
cros compiling.]]
The linux_aura port didn't deal with multiple monitors very well because
it was treating the X root window as one big display. When xrandr is
present, get the screen areas from it, and exposes this data back to
chrome.
This patch also factors out the EDID parser than chromeos was using into
a common directory. Like chromeos, we use it to assign stable display
IDs.
BUG=287972
TBR=brettw@chromium.org, derat@chromium.org, sky@chromium.org
First Review URL: https://codereview.chromium.org/23536057
Review URL: https://codereview.chromium.org/24459002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225254 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Found by clang's new -Wunused-const-variable.
BUG=290204
R=akalin@chromium.org, brettw@chromium.org, sergeyu@chromium.org, thestig@chromium.org
Review URL: https://codereview.chromium.org/24616002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225235 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Copy the thread name to the stack for debugging purposes.
>
> The this ptr in Thread is getting smashed after the MessageLoop ends, which
> leaves very little to go on. This is kind of a stab in the dark, but hopefully
> it makes something bleed in a general direction.
>
> BUG=274705
> TEST=Minidumps contain 47 bytes of thread name data way high on the stack.
> R=mark@chromium.org
>
> Review URL: https://codereview.chromium.org/23606030
TBR=rsesek@chromium.org
Review URL: https://codereview.chromium.org/24508003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225228 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This was removed as part of removing the import process in issue 219419.
BUG=219419
R=battre@chromium.org
Review URL: https://codereview.chromium.org/24453002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225185 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=268984
R=ajwong@chromium.org, erikwright@chromium.org, jam@chromium.org
Review URL: https://codereview.chromium.org/23514056
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225184 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Compiled on linux_chromeos, but doesn't compile in cros.
> linux_aura: Implement most of DesktopScreenX11.
>
> The linux_aura port didn't deal with multiple monitors very well because
> it was treating the X root window as one big display. When xrandr is
> present, get the screen areas from it, and exposes this data back to
> chrome.
>
> This patch also factors out the EDID parser than chromeos was using into
> a common directory. Like chromeos, we use it to assign stable display
> IDs.
>
> BUG=287972
> R=brettw@chromium.org, derat@chromium.org, sky@chromium.org
>
> Review URL: https://codereview.chromium.org/23536057
TBR=erg@chromium.org
Review URL: https://codereview.chromium.org/24365012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225061 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The linux_aura port didn't deal with multiple monitors very well because
it was treating the X root window as one big display. When xrandr is
present, get the screen areas from it, and exposes this data back to
chrome.
This patch also factors out the EDID parser than chromeos was using into
a common directory. Like chromeos, we use it to assign stable display
IDs.
BUG=287972
R=brettw@chromium.org, derat@chromium.org, sky@chromium.org
Review URL: https://codereview.chromium.org/23536057
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225054 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Otherwise when increasing action timeout some tests actually start taking
longer (they probably sleep).
Also change test status to timeout for tests in a batch that completed but
took longer than the timeout so that the outcome is not dependent on test
order and similar factors.
BUG=236893
R=sky@chromium.org
Review URL: https://codereview.chromium.org/24255017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225013 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The ObserverList macro (FOR_EACH_OBSERVER) uses an iterator named |it|, which can in some cases conflict with locally defined variables.
Make the iterator variable name slightly more unique
BUG=None
R=jar@chromium.org
Review URL: https://codereview.chromium.org/23672052
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224985 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
R=darin@chromium.org
Review URL: https://codereview.chromium.org/24350011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224853 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This makes sure it's also available during destructions.
While the immediate problem happened only with ParallelTestLauncher,
I'm proactively fixing similar pattern in other places.
BUG=236893
R=sky@chromium.org
Review URL: https://codereview.chromium.org/24398002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224838 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=247382
Review URL: https://chromiumcodereview.appspot.com/22791013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224744 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://chromiumcodereview.appspot.com/16206002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224546 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This still defaults to serialized execution while I perform more testing.
BUG=236893
R=sky@chromium.org
Review URL: https://codereview.chromium.org/23757033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224200 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This was discovered by the InstantExtended field trial that was
using very small group percentages and discovered several groups
of the same size were getting different user counts.
BUG=290438
TEST=New unit test that fails without the fix.
Review URL: https://chromiumcodereview.appspot.com/23710041
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224011 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Tests which use WorkerPool have no way to pump it during teardown, so they will
sometimes have leaks of PostTaskAndReply Closures that we can't do anything
about. Since we're deprecating WorkerPool anyway, we can just annotate those
leaks and forget about them.
BUG=290897,248513
R=ajwong@chromium.org
Review URL: https://chromiumcodereview.appspot.com/23514070
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223960 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
With thread-local tracing (r221766), TraceLog::Flush() is async and
needs to be called from a thread having a message loop. It also requires
that tracing has been stopped.
BrowserShutdownProfileDumper is a special caller that TraceLog::Flush()
was originally called from a thread without message loop.
Now start another thread for flushing, and call TraceLog::SetDisabled()
before TraceLog::Flush().
The calling thread needs to wait for the completion of the flush,
otherwise the browser may shutdown before all trace events written.
Temporarily allow the thread to wait in ThreadRestrictions.
BUG=none
TEST=manual run chrome with --trace-shutdown. Should not assert. Should generate correct trace file.
Review URL: https://chromiumcodereview.appspot.com/24047007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223819 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Trace memory, swap, ZRAM, and disk activity and
place the data in JSON to be displayed by trace-viewer.
BUG=236763
TEST=base_unittests TraceSystemStatsMonitorTest.*
Review URL: https://chromiumcodereview.appspot.com/22836004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223806 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=None
Review URL: https://chromiumcodereview.appspot.com/22911017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223727 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL partially reverts r194205, which aimed to support
IME for Ash. With this partial rollback, we can assume
that all the instances of Aura-Win Chrome running in desktop
use IMM32 regardless of the OS version.
Given that Ash IME support is not functional yet, there is
no strong reason to force TSF only on Win8+. Actually our
IMM32 implemenation is more robust than that of TSF. This
way should lower the risk of successful launch of Aura for
Windows in M31.
BUG=164964
TEST=manually confirmed that we can use IME with Win8 Aura build
Review URL: https://chromiumcodereview.appspot.com/23875022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223589 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The determination of whether or not to put up a sad tab page is made
based on the termination status of the renderer process.
The termination status of the renderer process is checked when
the channel between the browser and the renderer stops working.
There is a race here because there is no reason to believe that the
renderer process has finished terminating at the time that the
browser detects that its channel to the renderer is dead.
If renderer process is believed to be still running at the moment when
the decision to put up the sad tab page is taken, then no sad tab will
be put up, the view of the renderer will be removed (eventually), and
a transparent window will result.
This bug was previously fixed on Linux with with
https://chromiumcodereview.appspot.com/11316261
this patch expands that fix to Mac.
In particular, if the renderer process is known to be dead, wait for the
process to terminate before taking its termination status. To be sure
that the browser does not wait forever on a renderer that is not exiting,
send a kill signal to the renderer before doing the wait.
BUG=167538
Review URL: https://chromiumcodereview.appspot.com/23866011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223526 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Soon PerfTimer will move into base/timer/ directory as it is already used by
production code.
This splits the logging stuff into their owns files and also moves
PerfTimeLogger into its own files.
BUG=None
TEST=base_unittests, ipc_perftests, etc...
R=brettw@chromium.org
Review URL: https://chromiumcodereview.appspot.com/23985006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223486 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Use the ability to track activity changes by registering a listener on
the application.
Also, this will manage the state of various activities on the java side.
BUG=286071
Review URL: https://chromiumcodereview.appspot.com/23522032
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223437 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
webview's cookies file, for the first instance of the app, so
cookies are not lost.
BUG=b/9962984
TBRing safebrowsing_service and profile_impl_io_data changes since these are mechanical in nature.
TBR=mattm@chromium.org,mmenke@chromiumorg
adding no try since we have comments that are longer than 80 chars, and since this is allowed under style guidance.
NOTRY=true
Review URL: https://chromiumcodereview.appspot.com/23964011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223400 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
R=thakis@chromium.org
Review URL: https://codereview.chromium.org/23526052
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223370 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
it fails with EINTR. Specifically, in this case, the FD will already have been
closed.
BUG=269623
R=thakis@chromium.org
Review URL: https://codereview.chromium.org/23455051
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223369 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=155525
Review URL: https://chromiumcodereview.appspot.com/24123005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223365 0039d316-1c4b-4281-b951-d872f2087c98
|