| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is so that it is clearer what the intended initial priority of
a URLRequest is.
It is also needed so that we can later enforce that if a URLRequest
is set to ignore limits, it has MAXIMUM_PRIORITY; otherwise, we'd have
to mandate that SetPriority() is called before set_load_flags(), which
is fiddly.
Also standardize on a single URLRequest constructor.
BUG=166689
R=jam@chromium.org, jamesr@chromium.org, joth@chromium.org, mmenke@chromium.org, scottbyer@chromium.org, sky@chromium.org, tommi@chromium.org
Review URL: https://codereview.chromium.org/51953002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@232219 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
cs.chromium.org has way fewer hits for
'message'.*\.',?$ file:gyp
than for
'message'.*[^\.]',?$ file:gyp
so standardize on the latter.
BUG=none
TEST=less ocd when watching build output
NOTRY=true
Review URL: https://codereview.chromium.org/51843002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@232166 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This change removes the dismiss button (reverting r213426 and r213521)
as well as the reshow delta (effectively making the delta 0).
BUG=none
Review URL: https://codereview.chromium.org/52583002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@232064 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
a UMA histogram.
As a first attempt, tracked the following
- Before starting metrics service,
- Before creating profile
- Before starting startup time bomb
- Before starting thread watcher
- Before starting main message loop
- Before starting shutdown time bomb
- clean shutdown.
R=jar@chromium.org
Review URL: https://codereview.chromium.org/28283002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@231334 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Switch to logging an event before calling into delegates, and after
they return instead of only in the case that they block. This allows
delegates to log events as they take turns blocking a network
request.
Also allows delegates to log events when delaying the start of
a request, and when delaying the start of reading the body of a
request (But not between normal body reads).
Also allows delegates to log delegate information that will not be
displayed to the user.
BUG=294920
Review URL: https://codereview.chromium.org/25097004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@231084 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adds a new GURL::GetContent() to retrieve the text after scheme: specifically
intended for use in non-hierarchical schemes (data: etc).
Doing the first part of https://codereview.chromium.org/23835019/
in a preparation CL.
BUG=291747
Review URL: https://codereview.chromium.org/23549039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@230781 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=310806
R=robertshield@chromium.org
Review URL: https://codereview.chromium.org/37173006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@230642 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This will make it possible to depend on components from content shell
BUG=247431
R=jam@chromium.org
TBR=ben@chromium.org
Review URL: https://codereview.chromium.org/28123002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229620 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Disables ChromeFrameTestWithWebServer.WidgetModeIE_ObjectFocus
>
> BUG=308847
> TEST=none
> R=grt@chromium.org
> TBR=grt@chromium.org
>
> Review URL: https://codereview.chromium.org/29863002
TBR=sky@chromium.org
Review URL: https://codereview.chromium.org/30103002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229488 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=308847
TEST=none
R=grt@chromium.org
TBR=grt@chromium.org
Review URL: https://codereview.chromium.org/29863002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229469 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The revert of r229285 at r229410 shoud resolve this for now.
> Disable ChromeFrameTestWithWebServer.WidgetModeIE_ObjectFocus. It regressed in [229275:229286]
>
> BUG=308847
>
> Review URL: https://codereview.chromium.org/28223004
TBR=pfeldman@chromium.org
Review URL: https://codereview.chromium.org/29313002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229427 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
in [229275:229286]
BUG=308847
Review URL: https://codereview.chromium.org/28223004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229320 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This removes another ifdef, which should be avoided in general.
This is a follow up to the clean up started in crrev.com/225862.
See discussion from https://codereview.chromium.org/25022004/
BUG=None
TEST=None, no behavior changed.
R=sky@chromium.org,asvitkine@chromium.org
TBR=darin # for trivial changes in content/
Review URL: https://codereview.chromium.org/25435002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@228347 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
scons hasn't been used in a long time.
BUG=305702
NOTRY=true
Review URL: https://codereview.chromium.org/26718002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@228308 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Most implementations (x11 aura with and without chromeos) do not explicitly
call aura::Env::GetInstance() to initialize it. These build work as is because
the call is done when performing other operations, such as adding observers.
In OZONE there are no such calls, so the OZONE build fails when trying to
create a compositor since the Compositor::Initialize hasn't been called.
This patch adds an explicit initialization for aura based builds to fix this
issue.
BUG=
Review URL: https://codereview.chromium.org/23882007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@227745 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is the chromium side of a blink change at
https://codereview.chromium.org/25870005
Until the corresponding blink CL above is re-landed (it's landed but reverted),
disable EUC-KR-related test temporarily.
This is a copy of https://codereview.chromium.org/25769005/ (lgtm'd and approved).
BUG=277027, 95963
TEST=browser test: BrowserEncodingTest.* and EncodingAlias*.
TBR=thakis,robertshield
Review URL: https://codereview.chromium.org/26439002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@227506 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When Chrome creates Google Update's OnDemandMachineAppsClass by way of
the elevation moniker, the eventual setup.exe's process security token
doesn't have the SeSecurityPrivilege. This prevents Chrome Frame from
reading and/or modifying the security descriptor on the Secure Mime
Handlers registry key. This had been resulting in error 7 when trying to
manually update a Chrome that shared binaries with Chrome Frame. With
this change, the values are probed before trying to modify them. Chances
are that they will already hold the correct data, so the update should
succeed.
BUG=224043
TEST=see comment #3 in bug
Review URL: https://codereview.chromium.org/25738003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@226810 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
|
|
|
|
|
|
|
|
|
| |
BUG=76112
R=grt@chromium.org
Review URL: https://codereview.chromium.org/23645012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225291 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ctor
ScopedVector supports Pass() method.
Use of Pass() makes it clear that the ownership of the readers moves to the newly created UploadDataStream.
BUG=None
TEST=git cl try
TBR=sky@chromium.org for content/browser, mpcomplete@chromium.org for chrome/browser/extensions/api/web_request
Review URL: https://chromiumcodereview.appspot.com/23929005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224960 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
It broke Google Chrome ChromeOS bot.
http://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20ChromeOS/builds/58548/steps/compile/logs/stdio#error1
FAILED: g++ ... -c ../../net/socket/ssl_client_socket_nss.cc -o obj/net/socket/net.ssl_client_socket_nss.o
../../net/socket/ssl_client_socket_nss.cc: In member function 'int net::SSLClientSocketNSS::DoVerifyCertComplete(int)':
../../net/socket/ssl_client_socket_nss.cc:3445:64:error: no matching function for call to 'net::TransportSecurityState::GetDomainState(const string&, bool&, net::TransportSecurityState::DomainState*)'
../../net/socket/ssl_client_socket_nss.cc:3445:64: note: candidate is:
../../net/http/transport_security_state.h:212:8: note: bool net::TransportSecurityState::GetDomainState(const string&, bool, bool, net::TransportSecurityState::DomainState*)
../../net/http/transport_security_state.h:212:8: note: candidate expects 4 arguments, 3 provided
> Don't persist HPKP if PrivacyMode is enabled.
>
> BUG=258667
>
> Review URL: https://chromiumcodereview.appspot.com/19269012
TBR=mef@chromium.org
Review URL: https://codereview.chromium.org/24251011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224275 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=258667
Review URL: https://chromiumcodereview.appspot.com/19269012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224269 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Re-enable device discovery. It should not regress perf after r224052.
> Revert "Enable device discovery notifications."
>
> This reverts commit 15e19ede41054676329caafa49f0b29065e5f1e3. This commit is
> being reverted since it causes a significant increase in memory usage.
>
> TBR=vitalybuka
> BUG=292754
>
> Review URL: https://codereview.chromium.org/23536066
TBR=teravest@chromium.org
Review URL: https://codereview.chromium.org/24191003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224162 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit 15e19ede41054676329caafa49f0b29065e5f1e3. This commit is
being reverted since it causes a significant increase in memory usage.
TBR=vitalybuka
BUG=292754
Review URL: https://codereview.chromium.org/23536066
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223894 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
|
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=None, no functional change.
TBR=jam@chromium.org
Review URL: https://chromiumcodereview.appspot.com/23658056
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223272 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Add kill switch for notifications.
Disable notifications for task manager test.
BUG=286157
Review URL: https://chromiumcodereview.appspot.com/23903024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223106 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
The page cyclers run with Telemetry now. They should not be baked into the chrome
binary itself. This causes unwanted dependencies on test code and bloats the
binary.
BUG=280157
Review URL: https://chromiumcodereview.appspot.com/23498004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@221201 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Changes APIs for StreamListenSocket, TCPListenSocket, UnixDomainSocket and derived test classes to use scoped_ptr instead of scoped_refptr.
BUG=263963
Review URL: https://chromiumcodereview.appspot.com/20142003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@220992 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This is to allow cc_perftests to share this code without depending on chrome/.
BUG=
Review URL: https://chromiumcodereview.appspot.com/23509002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@220803 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/19579005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@220612 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Broken by: https://codereview.chromium.org/23097007
I wish these were caught by the try bots.
BUG=266007
TBR=lambroslambrou@chromium.org
Review URL: https://codereview.chromium.org/23493012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@220345 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=None
TBR=jam@chromium.org
Review URL: https://chromiumcodereview.appspot.com/23112033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@219495 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This should address the TODO from tools/gn/secondary/base/BUILD.gn
BUG=None
TEST=test_support_perf
R=brettw@chromium.org
TBR=cpu, dalecurtis, akalin, thestig
Review URL: https://chromiumcodereview.appspot.com/22887041
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@219448 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
It is flaky on ie6. More details in http://crbug.com/255098#c11
BUG=255098
TBR=robertshield
Review URL: https://codereview.chromium.org/22859052
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@219013 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Flaky on ie7, more info on the bug.
BUG=277406
TBR=tommi@chromium.org
Review URL: https://codereview.chromium.org/22831028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218981 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=274679
TEST=N/A
TBR=cbentzel, jamesr, simonjam, tzik, stevet, mpcomplete
Review URL: https://chromiumcodereview.appspot.com/23064011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218893 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This issue was found by a linter.
R=tommi@chromium.org
BUG=
Review URL: https://chromiumcodereview.appspot.com/23375003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218813 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This issue was found by a linter.
R=tommi@chromium.org
BUG=
Review URL: https://chromiumcodereview.appspot.com/23375002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218786 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Previously: https://codereview.chromium.org/22909017/
TBR=jam@chromium.org
BUG=237249
Review URL: https://chromiumcodereview.appspot.com/23320004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218663 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This issue was found by a linter.
The value of 'hr' is ignored. Probably a nice idea to validate the CreateInstance success.
R=amit@chromium.org,tommi@chromium.org
BUG=
Review URL: https://chromiumcodereview.appspot.com/21948002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218662 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
GCF users the option to disable the meta tag check. That is, the tag that websites can use to enable GCF. This way the white/black lists are fully respected by GCF.
BUG=225971
TEST=Set HKEY_CURRENT_USER\Software\Google\ChromeFrame\SkipGCFMetaDataCheck to true to disable GCF on sites enabled through the meta data tag (fully respects white/black lists).
Contributed by joe.knoll@workday.com
TBR=joaodasilva@chromium.org,robertshield@chromium.org
Review URL: https://chromiumcodereview.appspot.com/23295005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218626 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The tab contents window title is updated from the page URL to the page
title in conjucting with sending out the
IA2_EVENT_DOCUMENT_LOAD_COMPLETE accessibility event. Usually the update
has already happened by the time the load event is processed by the
test, but not always. The tests now match both URL and title when
handling these load events.
BUG=276416
TEST=chrome_frame_tests.exe --gtest_filter=CF*/NavigationTransitionTest.JavascriptWindowOpen/* --gtest_repeat=20 passes when use_aura=1
Review URL: https://chromiumcodereview.appspot.com/22912010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218620 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=NONE
TEST=NONE
TBR=bauerb
Review URL: https://chromiumcodereview.appspot.com/23123009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218577 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=272072
Review URL: https://chromiumcodereview.appspot.com/22794009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@217875 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=271530
TEST=NONE
TBR=grt
Review URL: https://chromiumcodereview.appspot.com/22999004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@217577 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is failing on Android bots.
> net: add a test to ensure that our TLS handshake doesn't get too large.
>
> (I would like the test to assert that we have some headroom too, but we don't
> have any to assert!)
>
> BUG=none
> R=rsleevi@chromium.org, wtc@chromium.org
>
> Review URL: https://codereview.chromium.org/19557004
TBR=agl@chromium.org
Review URL: https://codereview.chromium.org/22371007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@216479 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
(I would like the test to assert that we have some headroom too, but we don't
have any to assert!)
BUG=none
R=rsleevi@chromium.org, wtc@chromium.org
Review URL: https://codereview.chromium.org/19557004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@216444 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Tests need both sets of entry point functions, ContentClient embedder
pointers, etc. Now in gyp, the three targets are content_app_browser for
browser-only, content_app_child, which is child-only, and
content_app_both which contains both.
TBR=joth@chromium.org, grt@chromium.org
R=jam@chromium.org
BUG=237249
Review URL: https://chromiumcodereview.appspot.com/21453003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215771 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL introduces an automated and centralized way to embed
compatibility manifest into all *.exe files. With this CL,
a potential risk of behavioural inconsistency between
production binaries and unit test binaries is resolved by
enforcing the same compatibility context.
This CL uses 'target_conditions' feature of gyp to inject
manifest settings into each executable target. One tricky
part is that some executables such as setup.exe and
mini_installer.exe require external manifest file instead of
embedded one when component build is enabled.
See http://crbug.com/127233 for this.
You can override the gyp variable
'win_exe_compatibility_manifest' locally for a given
executable target to embed a custom compatibility manifest.
BUG=260692
Review URL: https://chromiumcodereview.appspot.com/19275010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214427 0039d316-1c4b-4281-b951-d872f2087c98
|