GitLab

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

Adding metrics to track browser launches per primary/secondary profile. Adding metrics to track number of profiles per machine.

BUG=160771,106774
TEST=none

Review URL: http://codereview.chromium.org/8890054

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

BUG=none
TEST=builds on gcc 32 bit for linux
Review URL: http://codereview.chromium.org/8894028

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

BUG=None.
TEST=Click avatar menu and press Tab key to navigate through the menu.
Selected menu items should have a focus ring.

Review URL: http://codereview.chromium.org/8921003

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

This code was platform independent so could just be copied over into the
platform independent file with no changes.

Review URL: http://codereview.chromium.org/8899015

TBR=brettw@chromium.org
Review URL: http://codereview.chromium.org/8926003

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

Unit tests now instantiate their own AudioManager and can choose to
use the default one or provide their own mock implementation without
having to worry about conflicting with the singleton.

The teardown sequence of the AudioManager and its thread has been cleaned
up significantly and I don't think it has been completely tested before as
the audio thread was terminated before all objects that belonged to the
thread had a chance to do cleanup.  The AudioManager unit tests do not use
the actual audio thread, so this part seems to have been left out.

In Chrome, the AudioManager instance is now owned by BrowserProcessImpl
and always constructed on the UI thread.  This instance is then shared
in the same way that several other 'manager' type objects are shared to
'content' code, via content::ResourceContext.  Audio specific classes
do though receive a direct pointer to the AudioManager and are required
to do proper reference counting if they need to hold onto the instance.
I chose to use the ResourceContext rather than direct use of g_browser_process
to avoid requiring another singleton when writing relatively simple tests
that touch the AudioManager.

I added a couple of safeguards to guard against future regressions:
- Not more than one instance of the AudioManager should be created.
- The AudioManager should not be addrefed by its own thread.  This
  can basically become a circular reference and prevent deterministic
  shutdown.

Reviewers:  Of course you're free to review everything,
but here's the breakdown in terms of the bare minimum from
the standpoint of "Owners approval".  I'm asking Henrik to be the
main reviewer of the entire patch (sorry!).

Henrik: Everything minus the below, but it would be great if you could
        take a look at the whole thing, specifically media/audio.

Pawel:  I'd like you to take a generic look at this approach.
        The key areas as far as the singleton itself goes are in
        media/audio/audio_manager[_base].* and
        chrome/browser/browser_process*.*

Satish: content/browser/speech/*
        media/audio/audio_manager_base.* (new reference counting code)

Andrew: content/browser/renderer_host/media/*
        content/renderer/media/webrtc_audio_device_unittest.cc (Owner)

Avi:    content/browser/renderer_host/render_process_host_impl.cc
        content/browser/resource_context.*

William: chrome/browser/profiles/profile_io_data.cc
	 chrome/browser/browser_process*.*

Robert:  This is basically a heads up. I hope that I didn't break the OpenBSD
         implementation, but unfortunately I have no way of knowing for sure.

Shijing: Please take a look at AudioManagerLinux.  I replaced the set of
         active streams with a simple counter.

BUG=105249
TEST=content_unittests, media_unittests, browser_tests.
Review URL: http://codereview.chromium.org/8818012

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

This application icon index values are being changed from ints to StringOrdinals to decrease the potential number of syncs required when icons are moved, by only needing to change 1 StringOrdinal instead of all the integers.


BUG=61447
TEST=Open up an instance of chromium with multiple application icons and ensure
that icons can still have their page and order changed. There should be no
behaviour change.


Review URL: http://codereview.chromium.org/8198003

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

This bugs in isMark and isLetter lead 'init' feature to be applied
even when a syllable is not at the beginning of a word.

See http://www.microsoft.com/typography/otfntdev/bengalot/features.htm
for 'init' feature application in Bengali.

It turned out that the upstream also fixed this (see https://bugs.freedesktop.org/show_bug.cgi?id=30319) exactly the same way.

Change the upstream reference to 93d8573ddcf97648983a1c1b5cdda8a3ec6d4432
in README.chromium and remove 2 patches in chromium.patch because
they're all landed in the upstream @93d8573ddcf97648
Two removed patches are:

  - Fix OOB access (in Tibetan shaper) @ff0612c2e7df1b86fc
  - Fix crash! (in harfbuzz-gpos.c) @81c8ef785b079980a

In addition, the following upstream fix is included by going to 93d8573ddc.

  - Fix bug in contrib/harfbuzz-unicode.cc @93d8573ddcf9

Webkit bug to add a layout test for 'init' is
   https://bugs.webkit.org/show_bug.cgi?id=74310

BUG=107031
TEST=With Lohit Bengali font installed (the default on CrOS devices), go to http://i18nl10n.com/fonts/beng3.html and the result should be the same as the left in the screenshot attached to the bug instead of the right. A webkit layout test will be added and will be mentioned in the bug later.
Review URL: http://codereview.chromium.org/8895014

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

This is a reland of r113823, hopefully without breaking the Pepper tests.

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

This goes most of the way to re-enabling chrome_frame_net_tests by making it a content::BrowserMainParts.  The tests are still disabled until we are able to test them on IE8 and older; on IE9 all tests currently pass except for a few that hang.

Additionally, this makes chrome_frame_net_tests determine protocol version using the exact same version information as npchrome_frame.dll does, so that version mismatch due to the RC dependency problems we have sometimes seen should not happen any more with this test suite.

While in there, I also updated the poor_mans_trybot.bat script.

BUG=105435

Review URL: http://codereview.chromium.org/8894008

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

The current code is for update time run, so it can generate correct list per OS.  However, we changed it to run locally and check the generated list in, so we need to have OS conditions for tests.

Also, add DEBUG/RELEASE modifier so we can mark test failures better.

BUG=
TEST=gpu_tests built
R=kbr
Review URL: http://codereview.chromium.org/8885003

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

This code was platform independent so could just be copied over into the
platform independent file with no changes.

Review URL: http://codereview.chromium.org/8899015

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

BUG=100026,107181
TEST=none
TBR=vrk

Review URL: http://codereview.chromium.org/8920010

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

If the user had a custom profile name then clicking on "Edit profile" would reset the name to their GAIA name.

The problem was that we were incorrectly detecting that the user had changed their profile picture to their GAIA picture. Clicking on the GAIA picture causes us to set the profile name to the GAIA name.

Fix was to only send profile picture changed events if the user actually changed their profile picture.

BUG=106382
TEST=Reproduced the bug. Applied my fix and verified that it no longer reproduced.


Review URL: http://codereview.chromium.org/8905001

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

base::Bind: Convert chrome_benchmarking_message_filter.cc.

BUG=none
TEST=none
R=csilv,ajwong

Review URL: http://codereview.chromium.org/8897026

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

Test gathers the EPP and TTP metrics for video playback under
constrained network conditions.  The test uses the Constrained Network
Server (CNS) to accomplish this.

 - Starts CNS.
 - Iterates over test matrix of network constraints.
 - Reports results in a manner consumable by chromium.perf_av.
 - Stops CNS.

BUG=106257
TEST=Ran test locally.


Review URL: http://codereview.chromium.org/8802030

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

Also, move some URL constants to url_constants.h/cc.

BUG=103324
TEST=Go to chrome://settings/personal. Sign in to sync. Choose "Sync Everything". The learn more link should go to http://...answer=185277.


Review URL: http://codereview.chromium.org/8589014

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

BUG=77155, 103574
TEST=none

Review URL: http://codereview.chromium.org/8872036

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

BUG=106795
TEST=Virtual Me2Me still works

Review URL: http://codereview.chromium.org/8863010

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

The new truncate mode is needed by my upcoming changes to
rewrite CanvasSkia::DrawText().

Part of the change is changing the binary search code to a
more traditional form that also makes the truncate mode
work correctly. This is covered by the new unit tests.

BUG=105550
TEST=New and existing tests in text_elider_unittest.cc.
Review URL: http://codereview.chromium.org/8917011

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

BUG=105811
TEST=Inspect checkmark position in settings window for all platforms.
Review URL: http://codereview.chromium.org/8918012

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

levels requirested. Also ClearLevel now uses glTexSubImage2D to clear the level if the
texture is immutable.  Finally, rearranged the code in doTexSubImage2D to correctly
bypass clearing a level if the bitmaps supplied matches the dimensions of the texture level.
BUG=106894
Review URL: http://codereview.chromium.org/8872061

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

New in this roll: ASan! (for mac and linux64 for now)

BUG=none
TEST=none
TBR=hans

Review URL: http://codereview.chromium.org/8918007

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

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

Control group (whitespace change) is http://codereview.chromium.org/8917009
Review URL: http://codereview.chromium.org/8922003

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

base::Bind: Convert browsing_data_remover.h.

BUG=none
TEST=none
R=csilv,ajwong

Review URL: http://codereview.chromium.org/8896024

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

This includes the Chrome 16 release notes.
It also includes updates to the manifest doc
to include the requirements field
which the release notes reference.

Review URL: http://codereview.chromium.org/8743020

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

BUG=http://code.google.com/p/chromium-os/issues/detail?id=24025
TEST=If the user sets the name of a printer in cloud print that name should be the displayed name of that printer in the print preview page.


Review URL: http://codereview.chromium.org/8896014

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

Revert "Disable RendererAccessibilityBrowserTest.CrossPlatform* tests"

This reverts commit 2a31da98908e3e42b323f4d67b957fc8b85941e3.

and

ensures that we replace the test tree when receiving layout complete notifications.
Only pass on the test tree when the tree actually has a non-empty subtree. This indicates a non-blank test page.

BUG=106934
TEST=try bots.
TBR=jam,sergyu

Review URL: http://codereview.chromium.org/8895018

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

BUG=chromium-os:23281
TEST=Manual: see bug description

Review URL: http://codereview.chromium.org/8923004

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

Follow-up to 113090 to see which ModelChangingSyncerCommand triggers
a perf regression.

BUG=97832
TEST=
TBR=tim

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113319

Review URL: http://codereview.chromium.org/8820019

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

BUG=106786
R=jamesr@chromium.org,jbates@chromium.org


Review URL: http://codereview.chromium.org/8898010

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

chromeos is the target in a few places.
Make this assumption explicit.
Also enable nacl in the non-chromeos case so we can start debugging.

Review URL: http://codereview.chromium.org/8776009

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

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

Review URL: http://codereview.chromium.org/8899018

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

BUG=104160
TEST=Covered by ChunkDemuxer unittests.


Review URL: http://codereview.chromium.org/8775035

TBR=acolwell@chromium.org
Review URL: http://codereview.chromium.org/8921010

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

This variable is no longer used, so remove it.

BUG= none
TEST= build still works

R=sehr@google.com

Review URL: http://codereview.chromium.org/8893023

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

BUG=104160
TEST=Covered by ChunkDemuxer unittests.


Review URL: http://codereview.chromium.org/8775035

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

I also added an unittest entry that checks the accelerator is processed only once. This test has failed before.
This CL depends on
http://codereview.chromium.org/8834014/


BUG=106702
TEST=Ran aura_shell_unittest


Review URL: http://codereview.chromium.org/8833012

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

of any of the permanent nodes can be toggled.

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


Review URL: http://codereview.chromium.org/8828006

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