GitLab

Review URL: https://codereview.chromium.org/137993011

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245149 0039d316-1c4b-4281-b951-d872f2087c98

BUG=316496
NOTRY=true
R=dewittj@chromium.org

Review URL: https://codereview.chromium.org/137993005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245148 0039d316-1c4b-4281-b951-d872f2087c98

This reverts commit 17dad9d5.

Although simple in principle, the Python C extension approach does not fit the
diversity of our infrastructure.

We will need a better method to address the speed requirements for
ColorHistogram.

BUG=333606,323813,334097,333361,334523
TBR=tonyg,maruel

Review URL: https://codereview.chromium.org/138143010

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245145 0039d316-1c4b-4281-b951-d872f2087c98

This relands digit@'s CL that completely reworks the traffic forwarding logic
and that should guarantee in particular that select() never blocks without a
good reason (i.e. that we are not leaking Forwarder instances due to the fact
that their internal thread would be blocked on select()).

The initial problem with r235213 was only the use of PipeNotifier which added
two file descriptors for each Forwarder instance (in addition to the two
sockets the Forwarder instance operates on). We were exceeding the file
descriptor limit (1024) on the intl_ko_th_vi Telemetry page set which is not
surprising since Chrome easily keeps 256 sockets around in its socket pool.
Therefore this change simply removes the PipeNotifier since we were not (yet)
using it anyway.

The initial problem was reproduced and the fix was tested as well with:
tools/perf/run_measurement -v --browser=android-chromium-testshell \
    --show-stdout page_cycler tools/perf/page_sets/intl_ko_th_vi.json

Note that a next step would be to have all the Forwarder instances operate on a
common thread owned by DeviceListener so that we don't end up with 256 threads.

BUG=332403

Review URL: https://codereview.chromium.org/137923004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245144 0039d316-1c4b-4281-b951-d872f2087c98

R=tonyg@chromium.org

Review URL: https://codereview.chromium.org/137913003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245143 0039d316-1c4b-4281-b951-d872f2087c98

BUG=178409
R=thakis@chromium.org

Review URL: https://codereview.chromium.org/139723002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245142 0039d316-1c4b-4281-b951-d872f2087c98

The one interesting bit is this makes mojo use the aura theme rather
than the android theme.

BUG=none
TEST=none
R=ben@chromium.org

Review URL: https://codereview.chromium.org/139623002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245141 0039d316-1c4b-4281-b951-d872f2087c98

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245140 0039d316-1c4b-4281-b951-d872f2087c98

libwebm will be used by chromoting client to generate WebM stream for
MediaSource API.

BUG=321825

Review URL: https://codereview.chromium.org/134643003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245139 0039d316-1c4b-4281-b951-d872f2087c98

On Android, viewing the data reduction settings screen would
trigger configuring the proxy, even if the user did not change
the data reduction proxy enabled pref. This CL limits
reconfigurations to actual pref changes, which reduces the work
Chrome needs to do.

BUG=181049

Review URL: https://codereview.chromium.org/124793003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245138 0039d316-1c4b-4281-b951-d872f2087c98

This adds a very basic surface class with tests under cc/surfaces
to establish gyp and DEPS rules. While this system is under early
development it shouldn't be used by anything outside of unit tests.

BUG=334090

Review URL: https://codereview.chromium.org/131893003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245137 0039d316-1c4b-4281-b951-d872f2087c98

icu.isolate file added in https://codereview.chromium.org/99073008/
is used where icudt.dll is directly listed for
'isolate_dependency_tracked'.

icu.isolate includes either icudtl.dat or icudt.dll depending on
'OS' and 'icu_use_data_file_flag'.

Besides, 'icu_use_data_file_flag' is added to build/isolate.gypi as a
config-variable.

Once the above icu-side CL is checked in, this CL will also includes
icu DEPS roll.

This is a prerequisite for the following CLs to use icudtl.dat :

Windows : https://codereview.chromium.org/99473012
Linux : https://codereview.chromium.org/102413007
Mac : https://codereview.chromium.org/109013004

BUG=72633
TEST=All tests utilizing isolate pass on bots.

Review URL: https://codereview.chromium.org/124143002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245136 0039d316-1c4b-4281-b951-d872f2087c98

    
This change splits ChromiumEnv into two classes: ChromiumEnv/ChromiumEnvPosix.
ChromiumEnvWin32 was also added which uses the Win32 API - except for logging.

The default is to create POSIX environments for all platforms for non-IDB
LevelDB users. For IDB we create a Windows environment on Windows, else a
POSIX environment like before.

BUG=222623

Review URL: https://codereview.chromium.org/113373002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245135 0039d316-1c4b-4281-b951-d872f2087c98

The policy protobufs were duplicated at chrome/browser/policy/proto
so that the protofiles ebuild of ChromeOS could still fetch them.
That ebuild has been fixed in a separate CL at:

  https://chromium-review.googlesource.com/#/c/182300/

BUG=327345

Review URL: https://codereview.chromium.org/137903004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245133 0039d316-1c4b-4281-b951-d872f2087c98

Currently there's a symbol retrieval in place every time someone asks for
a NSString* inside the AVFoundation library. Change this so that the
AVFoundationInternal retrieves them on construction and caches them. This
CL should not change the behaviour of the avfoundation_glue.

BUG=323536, 288562

Review URL: https://codereview.chromium.org/137903003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245132 0039d316-1c4b-4281-b951-d872f2087c98

Move the bulk of the files in content/browser/aura to
content/browser/compositor, since they're not aura-specific.

Build all of those files in the Mac build, and add ui/compositor
to the dependencies of the Mac build. These files will soon only
be excluded from the Android build.

BUG=314190

Review URL: https://codereview.chromium.org/137893007

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245131 0039d316-1c4b-4281-b951-d872f2087c98

Android requires us to call restart input when input node changes.
Previously we relied on onHandleGesture which is not the right way
to do it.

BUG=242715

Review URL: https://codereview.chromium.org/56643003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245130 0039d316-1c4b-4281-b951-d872f2087c98

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245128 0039d316-1c4b-4281-b951-d872f2087c98

This is a regression introduced by 97028206 (bug 299224).

BUG= 
TEST=Go to youtube.com or dailymotion.com. Start playing Flash video. Middle-click one of suggested link to videos to open new tab in background. Activate the tab you've just opened. Expected: Flash video should be playing in active tab. Current: Flash video is not playing until you scroll the page.

Review URL: https://codereview.chromium.org/137883003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245127 0039d316-1c4b-4281-b951-d872f2087c98

When using the legacy API both continuos and provisional_results are
false, and in the JS API they are by default false but can be set to
true independently. The DCHECKs didn't allow for this and erroneously
triggered for some API usages; like the implementation on google.com.

Review URL: https://codereview.chromium.org/137853010

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245125 0039d316-1c4b-4281-b951-d872f2087c98

WebGraphicsContext3D::reshape..() resizes the default backbuffer, but the
TestPlugin doesn't use the default backbuffer so this call is a no-op.

BUG=181120

Review URL: https://codereview.chromium.org/137613002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245124 0039d316-1c4b-4281-b951-d872f2087c98

This CL implements nacl_irt_thread and nacl_irt_tls.

BUG=https://code.google.com/p/nativeclient/issues/detail?id=3734
TEST=Tried to call a newly added function from plugin via AT_SYSINFO.

Review URL: https://codereview.chromium.org/133683011

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245123 0039d316-1c4b-4281-b951-d872f2087c98

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245122 0039d316-1c4b-4281-b951-d872f2087c98

Includes the following CLs:

Remove LDAP support from NSS.
https://codereview.chromium.org/130403004/

R=rsleevi@chromium.org
BUG=289680
TEST=none

Review URL: https://codereview.chromium.org/137603002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245121 0039d316-1c4b-4281-b951-d872f2087c98

BUG=331730
TEST=Nope.

Review URL: https://codereview.chromium.org/137583004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245120 0039d316-1c4b-4281-b951-d872f2087c98

This avoids invoking a null callback when an untrusted renderer
sends an out-of-sequence message.

BUG=334038

Review URL: https://codereview.chromium.org/137503002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245119 0039d316-1c4b-4281-b951-d872f2087c98

As part of effort to support multiple fonts, this CL changes methods which are now taking gfx::Font so they will take gfx::FontList instead.

See https://docs.google.com/a/chromium.org/document/d/1D_25fp9B8b9aZJORfAjDIFq61NWvUquZ5xmKH-VcC4k/view

BUG=265485
TEST=Run unit_tests, ui_unittests, views_unittests

Review URL: https://codereview.chromium.org/119273002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245118 0039d316-1c4b-4281-b951-d872f2087c98

BUG=334317

Review URL: https://codereview.chromium.org/137443009

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245117 0039d316-1c4b-4281-b951-d872f2087c98

When the change is rolled in, I'll rebaseline and enable the tests.

BUG=329407
TBR=dominicc,esprehn
NOTRY=true

Review URL: https://codereview.chromium.org/138003010

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245116 0039d316-1c4b-4281-b951-d872f2087c98

The OVERRIDE needs to be right after the method name, and before other
attributes (e.g. WARN_UNUSED_RESULT) or gcc 4.8 gives a build error.
Clang does not seem to mind it though.

Review URL: https://codereview.chromium.org/139703005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245115 0039d316-1c4b-4281-b951-d872f2087c98

BUG=330082

Review URL: https://codereview.chromium.org/138933002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245114 0039d316-1c4b-4281-b951-d872f2087c98

BUG=334049

Review URL: https://codereview.chromium.org/138913002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245113 0039d316-1c4b-4281-b951-d872f2087c98

There are some unnecessary includes in generated thunks. We should remove these
for clarity, and (hopefully) to reduce build time. A small test showed that
ppb_view_thunk.cc built 18% faster after the change, but that's well within the
noise on my local system.

BUG=334727

Review URL: https://codereview.chromium.org/137353003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245111 0039d316-1c4b-4281-b951-d872f2087c98

Send multiple queries to the network in short intervals to account for potential
packet loss on noisy networks.

BUG=332629

Review URL: https://codereview.chromium.org/137343005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245110 0039d316-1c4b-4281-b951-d872f2087c98

that have changed from the previous context.

The most CPU consuming operations on the GPU thread for
Android are virtual context switches. We attempt to reduce the CPU usage by
context switches with this patch.

BUG=244701

Review URL: https://codereview.chromium.org/118203002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245109 0039d316-1c4b-4281-b951-d872f2087c98

Android mediaplayer won't be able to handle blob urls.
After xhr requests creates a temporary file with blob url, we need to pass the file path to the media player so it can play the file.

TBR=tedchoc
BUG=253465

Review URL: https://codereview.chromium.org/98823003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245108 0039d316-1c4b-4281-b951-d872f2087c98

There is too many abstractions in our surface classes to
determine 'real' SwapBuffers calls in traces. This adds
traces in all the relevant places where we do 'real' swaps.

BUG=334063

Review URL: https://codereview.chromium.org/137823002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245107 0039d316-1c4b-4281-b951-d872f2087c98

BUG=194304
NOTRY=true

Review URL: https://codereview.chromium.org/137803002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245106 0039d316-1c4b-4281-b951-d872f2087c98

Before, we regarded any HTML form on the landing page after a login attempt as failure.

After this CL, it's only a failure, if the new form has the same action as the old one.

BUG=329860

Review URL: https://codereview.chromium.org/137783003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245105 0039d316-1c4b-4281-b951-d872f2087c98

The change was needed because when there was a lot of information to be
displayed, and a if the query was not pasted, then the incremental search
took too long and the partial results rendering was unnecesarily refreshing
the results pane when what was actually needed was just the full query.

BUG=333414

Review URL: https://codereview.chromium.org/137753002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245104 0039d316-1c4b-4281-b951-d872f2087c98