GitLab

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

I also renamed classes to NestedAcceleratorXxx. I felt this is a bit more clearer than NestedDispatcher, especially in ui/wm/core. Please let me know if you disagree or have better suggestion. I'm happy to rename them.

BUG=None

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

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

This patch converts the following gyp targets to GN:
- favicon_base
- query_parser
- metro_viewer_messages -> metro_viewer
- native_theme

BUG=None
TEST=None
R=scottmg@chromium.org
TBR=ben@chromium.org
NOTRY=true

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

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

NOTRY=true

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

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

BUG=45482
R=thestig@chromium.org
NOTRY=true

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

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

TBR=jochen@chromium.org
NOTRY=true

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

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

this change:
- changes border-color instead of opacity of checkboxes on hover
- makes checked and hovered checkboxes look more similar
- makes hovered rows more noticeable

And I touched a few more things that weren't really noticeable
before I changed the background color:
- checkbox vertical alignment
- added padding in front of checkboxes

BUG=351565
R=dmazzoni@chromium.org
TEST=visible checkboxes

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

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

BUG=None
TEST=None

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

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

This is rudimentary implemenatation of AttachmentDownloader along with fake for tests.

BUG=376073
R=maniscalco@chromium.org

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

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

Identical logic change in both AudioSender and VideoSender: Send a sender
report just before each frame, up to the first 100 frames.  Then, they are
sent at ~1 second intervals thereafter.  However, if a receiver report
shows up, we halt the aggressive report sending immediately since we know
the data dependency (receiver side) has been satisfied.

Also, misc clean-ups:

1. RtpTimestampHelper threw away significant precision in its calculations
   (was truncating to millis).
2. Unit tests were using EXPECT with args in the wrong order (per gtest
   requirements).
3. Got rid of useless LocalRtcpXXXXXSenderFeedback class.

BUG=371482

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

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

scoped_array is deprecated in favor of scoped_ptr[]
as of https://codereview.chromium.org/14081006

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

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

BUG=

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

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

NOTRY=true
TBR=michaelbai@chromium.org

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

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

Also remove cruft and simplify code.

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

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

For tests I'm creating a MessagePumpGlib on a thread, destroying it,
then creating another MessagePumpGlib on another thread. The DCHECK
fires in this case because it only remembers the first thread. I made
it so the thread id is cleared when the MessagePumpGlib is destroyed.

BUG=none
TEST=none
R=sadrul@chromium.org
TBR=darin@chromium.org

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

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

BUG=368455

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

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

I was doing some benchmarking on tests on linux, and noticed that 10-20% of time was
in ThreadCheckerImpl::CalledOnValidThread. I looked into it and found that we're doing
a syscall to get the thread ID, which is not needed on any platform where pthread_t
is already an integer type.  Replacing that syscall with pthread_self() makes this particular
test (End2EndTest.BasicFakeSoftwareVideo) go from 488ms to 374ms, which is a 20% speedup.
While other tests might not use ThreadChecker quite as much, it should still be a pretty
good win across a large set of tests.

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

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

The vsync is unnecessary for ratelimiting or tear prevention, and can cause issues when multiple windows are displayed.

BUG=313104

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

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

Turn on ENABLE_NEW_GAMEPAD_API flag so that the new gamepad code is used.
Also remove the old code on Chromium side. The ENABLE_NEW_GAMEPAD_API is only
needed until the Blink side #ifdefs are removed.

BUG=344556

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

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

This change removes the NPAPI plugin from the webapp and switch Windows to use the Mac workflow where the user is prompted to download and install the host manually.

The removal of the dispatching code and the NPAPI target will be done in later CLs.

BUG=377046

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

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

This is in preparation for enabling HiDPI on Windows by default. Most of the changes here are replacing usage of
the ui::GetImageScale function with ui::GetScaleForScaleFactor.

The patch to remove the HiDPI switch and enable it by default will follow. This patch has been incorporated from
girard's patch ( https://codereview.chromium.org/232483005/)

BUG=149881, 362245

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

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

BUG=376924

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

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

Since we had to change it on Win/Linux, I thought it would be good to be 
consistent and draw the frame on Mac as well.

BUG=372721

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

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

BUG=374686

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

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

BUG=n/a
NOTRY=true

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

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

Crash seems to indicate we're registering twice for a
notification. Perhaps we're calling ScheduleLoad and TabIsLoading for
the same tab. Only way I could see that happening is if ShowBrowser()
activated some other tab. This should help isolate if that is
happening.

BUG=368236
TEST=none
R=marja@chromium.org

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

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

To show the information about ServiceWorkers I added iframe of "chrome://serviceworker-internals/#iframe" to chrome://inspect.
And I used postMessage() to achieve the cross-domain messaging between chrome://inspect and chrome://serviceworker-internals.

BUG=358657

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

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

BUG=None

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

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

After r271640, they no longer need this information, since the browser
process sets it in crash reports.

TBR=jochen@chromium.org

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

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

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

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

Refactors the StatusChangeChecker class hierarchy to allow for more code
sharing.  Allows children of StatusChangeChecker to share logic around
performing a blocking wait, checking exit conditions, and timing out.

As before, the sharing is acheived through inheritance.  We should move
to a delegation pattern if this becomes too complicated, but I think the
use of inheritance still makes sense for now.

This is a pre-requisite for a future patch that adds a new kind of
StatusChangeChecker that does not listen to ProfileSyncService changes.
I expect that there will be many more kinds of StatusChangeCheckers
defined in the future.  This refactoring will help us share code between
them.

This is attached to issue 97780 and 95742 because it will enable us to
write more StatusChangeCheckers that we can use to refactor tests that
currently listen for snapshots and self-notifications.

BUG=97780,95742

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

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

The streamsPrivate API provides the response headers for the original URL request in the stream details. This passes them along to the plugin with the URL.

This also adds the ability for the plugin to cancel the stream, which is necessary if the plugin wants to re-request the original URL.

BUG=303491

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

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

Also, add an optimization to not decode frames if we are behind and multiple frames are currently decodable.

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

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

Turns MetricsLogChromeOS into a metrics::MetricsProvider. Splits the
ChromeOS-specific unittests out from the MetricsLog test into separate
ChromeOSMetricsProvider tests. Also moves LogChromeOSCrash() from
MetricsService to ChromeOSMetricsProvider.

BUG=374221
R=asvitkine@chromium.org
TBR=derat, pam

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

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

Revert of Don't expose --disable-gpu-compositing, only set internally. (https://codereview.chromium.org/300523002/)

Reason for revert:
Makes a bunch of virtual/softwarecompositing tests Timeout on Mac
https://code.google.com/p/chromium/issues/detail?id=377145

Original issue's description:
> Don't expose --disable-gpu-compositing, only set internally.
> 
> The flag isn't meant to be set externally, it doesn't work on most
> configs. It's only meant internally to make sure the renderer doesn't
> try to use GPU when the browser is in software.
> 
> BUG=None
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=272636

TBR=jbauman@chromium.org,piman@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=None

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

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

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

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

The test added in this CL DomDistillerTabUtilsBrowserTest.TestSwapWebContents
consistently fails on some bots, e.g.
http://build.chromium.org/p/chromium.mac/builders/Mac%2010.6%20Tests%20%28dbg%29%284%29/builds/41414
http://build.chromium.org/p/chromium.mac/builders/Mac10.7%20Tests%20%283%29/builds/27225
http://build.chromium.org/p/chromium.mac/builders/Mac10.6%20Tests%20%283%29/builds/48492

> Add support for distilling current WebContents
> 
> This CL adds the utilities needed for using the current WebContents when
> distilling web pages.
> 
> BUG=361939
> 
> Review URL: https://codereview.chromium.org/266073003

TBR=nyquist@chromium.org

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

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

The CL broke asan/lsan bots on linux, and chromiumos e.g.

LevelDBPrefStoreTest.BasicObserver (run #1):
Direct leak of 8 byte(s) in 1 object(s) allocated from:
    #0 0x5babfb in operator new(unsigned long) /usr/local/google/work/chromium/src/third_party/llvm/projects/compiler-rt/lib/asan/asan_new_delete.cc:55
    #1 0x61f3699 in LevelDBPrefStore::OnStorageRead(scoped_ptr\u003CLevelDBPrefStore::ReadingResults, base::DefaultDeleter\u003CLevelDBPrefStore::ReadingResults> >) chrome/browser/prefs/leveldb_pref_store.cc:406
    #2 0x61f2ea7 in LevelDBPrefStore::ReadPrefs() chrome/browser/prefs/leveldb_pref_store.cc:329
    #3 0x336efee in LevelDBPrefStoreTest_BasicObserver_Test::TestBody() chrome/browser/prefs/leveldb_pref_store_unittest.cc:97
...

Indirect leak of 4096 byte(s) in 1 object(s) allocated from:
    #0 0x5bad4b in operator new[](unsigned long) /usr/local/google/work/chromium/src/third_party/llvm/projects/compiler-rt/lib/asan/asan_new_delete.cc:57
    #1 0x7e66db5 in AllocateNewBlock third_party/leveldatabase/src/util/arena.cc:62
    #2 0x7e66db5 in leveldb::Arena::AllocateFallback(unsigned long) third_party/leveldatabase/src/util/arena.cc:33
    #3 0x7e2a88a in NewNode third_party/leveldatabase/src/db/skiplist.h:182
    #4 0x7e2a88a in leveldb::SkipList\u003Cchar const*, leveldb::MemTable::KeyComparator>::SkipList(leveldb::MemTable::KeyComparator, leveldb::Arena*) third_party/leveldatabase/src/db/skiplist.h:325
    #5 0x7e2a76e in leveldb::MemTable::MemTable(leveldb::InternalKeyComparator const&) third_party/leveldatabase/src/db/memtable.cc:24
    #6 0x7e084ce in leveldb::DBImpl::DBImpl(leveldb::Options const&, std::string const&) third_party/leveldatabase/src/db/db_impl.cc:120
    #7 0x7e1bfb0 in leveldb::DB::Open(leveldb::Options const&, std::string const&, leveldb::DB**) third_party/leveldatabase/src/db/db_impl.cc:1449
    #8 0x61ef0a2 in LevelDBPrefStore::OpenDB(base::FilePath const&, LevelDBPrefStore::ReadingResults*) chrome/browser/prefs/leveldb_pref_store.cc:113
    #9 0x61efcae in LevelDBPrefStore::DoReading(base::FilePath const&) chrome/browser/prefs/leveldb_pref_store.cc:158
    #10 0x61f2dc0 in LevelDBPrefStore::ReadPrefs() chrome/browser/prefs/leveldb_pref_store.cc:325
...

> Implementation of leveldb-backed PrefStore.
> 
> This is not hooked up yet, migration code from Json-backed stores is needed, among other things.
> 
> BUG=362814
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271602
> 
> Review URL: https://codereview.chromium.org/169323003

TBR=dgrogan@chromium.org

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

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

Instead, each HttpNetworkSession is given its own immutable copies on
construction.  Other than spdy_enabled, none of the globals were
changed before this CL, anyways.

Also, setting spdy_enabled back to true after setting it to false no
longer clears the NPN list.

spdy_enabled is still a global because group policy can set it to
false at runtime.

BUG=372533
R=joaodasilva@chromium.org, rch@chromium.org, sergeyu@chromium.org, sgurun@chromium.org, sky@chromium.org, ttuttle@chromium.org

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

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