GitLab

This CL focusses on moving the following files:

webrequest_action.h/cc
webrequest_rule_registry.h/cc

from chrome/ to extensions/.

As part of the move, the following files need to move:

web_request_api_helpers.h/cc

and some minor re-factoring occurs (e.g.  moving a static function from web_request_api into web_request_api_helpers.

BUG=352293

TBR=darin@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#295719}

Avoid a crash caused by dereferencing an iterator after
erasing it.

BUG=415816

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

Cr-Commit-Position: refs/heads/master@{#295718}

Apparently these are also required for the swarming phase of
tests.

BUG=312586

R=jochen@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#295717}

This is a reland of https://codereview.chromium.org/588493002 using buffer on
bytearray before trying to call unpack_from because python 2.7.3 and earlier
doesn't handle unpacking a bytearray directly.

BUG=415491
R=sdefresne@chromium.org,cmasone@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#295716}

libc++ is the recommended C++ library for iOS development so use
it by default when targeting iOS.

clang ToT ships with all the required header and libraries to
support compiling with -stdlib=libc++ so remove the exception.

BUG=366744

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

Cr-Commit-Position: refs/heads/master@{#295715}

Remove unnecessary button on 'Bookmark all tabs' window which appears
before "Bookmarks Bar"
XIB Changes:
Removing the line in the following .xib file which is responsible for the appearance of empty button.
chrome/app/nibs/BookmarkAllTabs.xib

BUG=415554
R=asvitkine@chromium.org

TEST=
1) Launch chrome, open 2-3 tabs.
2) Right click on last tab and select 'Bookmark all tabs'
3) Empty button shouldn't be shown.

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

Cr-Commit-Position: refs/heads/master@{#295714}

PulseAudio input can provide larger buffer size than what we ask for. This happens if the pulse internally increase its buffer size for various reasons. This patch handles it by dynamically allocate more memory to the FIFO.

BUG=411453
TEST=1. Have Jackd own a second (USB) microphone, and have
Pulseaudio connect to Jackd with the Pulseaudio Jack
module. There are many steps:
1a. Obtain a USB mic.
1b. Find out the ALSA name of the mic and of the default
audio.
1c. Create a file ~/.jackdrc containing:
/usr/bin/jackd -dalsa -d hw:<default audio device>
1d. Run "alsa_in -d hw:<USB mic name> &" (in background)
1e. Install pulseaudio-module-jack if not installed.
1f. Run "pactl load-module module-jack-source channels=2"
1g. Go into Pulseaudio and select Input, then Jack source.
1h. Install patchage, start it up and connect the USB mic
to the Pulseaudio JACK source.
2. Start up Chrome and begin a Hangout. Make sure that
Chrome is using the Default mic.
3. Have someone (or yourself on another computer) join the
Hangout and listen to you talk.

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

Cr-Commit-Position: refs/heads/master@{#295713}

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/65b4895..f782639

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=stephana@google.com

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

Cr-Commit-Position: refs/heads/master@{#295712}

If the shelf hides we should not immediately minimize a dragged panel. This panel can be dropped where it now is and detached.

BUG=393047
TEST=PanelWindowResizerTest.DetachThenHideShelf
TEST=Drag a panel window while switching profiles. There is no crash and the window is detached where it was when switching back.

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

Cr-Commit-Position: refs/heads/master@{#295711}

Applies to the horizontal divider line under the folder heading, and
above the contents switcher view, and the folder back button. Also set
the contents switcher background colour to the main background colour
(it doesn't make sense to have a different colour if the divider line
doesn't reach all the way to the edge).

BUG=411775

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

Cr-Commit-Position: refs/heads/master@{#295710}

Changing in the initialization order of WeakPtrFactory in src/gin module
such that all member variables should appear before the WeakPtrFactory
to ensure that any WeakPtrs to Controller are invalidated before its members
variable's destructor are executed, rendering them invalid.

BUG=303818

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

Cr-Commit-Position: refs/heads/master@{#295709}

apps::SavedDevicesService tracks USB devices that have been selected by
the user in the context of a given extension. Devices that can be
identified accurately after they have been reconnected because they have
a serial number are written out to ExtensionPrefs. All others are only
remembered until they are disconnected.

A new OnDisconnect observer function has been added to UsbDevice to
enable this monitoring.

BUG=346953

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

Cr-Commit-Position: refs/heads/master@{#295708}

BUG=NONE
TEST=Manually tested WebRTC clients and verified new UMA after close.

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

Cr-Commit-Position: refs/heads/master@{#295707}

Reland the code change for Issue 573963005: Clean up GestureEventDetails
constructors and fix unit tests:
https://codereview.chromium.org/573963005/.

The code does not change, just merge two commits into one.

BUG=350942
TBR=tdresser@chromium.org,jdduke@chromium.org,sadrul@chromium.org,sky@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#295706}

BUG=394187

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

Cr-Commit-Position: refs/heads/master@{#295705}

BUG=394990

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

Cr-Commit-Position: refs/heads/master@{#295704}

Fallback images should always have a source scale factor of 1.0, even on 300P
devices.

BUG=413300,415616
TEST=None

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

Cr-Commit-Position: refs/heads/master@{#295703}

Revert of Make ONCCertificateImporter async. (patchset #1 id:1 of https://codereview.chromium.org/582413002/)

Reason for revert:
Use-after-free.
https://code.google.com/p/chromium/issues/detail?id=415916

Original issue's description:
> Make ONCCertificateImporter async.
>
> This prepares for the new CertDatabase keyed service, which will have stricter threading restrictions. https://codereview.chromium.org/419013003/
> Before, ONCCertificateImporter accessed the NSSCertDatabase from the UI thread and blocked on certificate store operations.
> Now, the import itself is asychronous and calls back on completion.
>
> While there, this also removes the GMock of the importer.
>
> This is a reland of f08303014b165f6013fe33198cd798ebd9a4e925
>   refs/heads/master@{#295534}
> with the fixed destruction order in ONCCertificateImporterImplTest.
>
> The fix was reviewed in
>   https://codereview.chromium.org/589443002/
>
> TBR=eroman@chromium.org
> BUG=413219
>
> Committed: https://crrev.com/3b4ba221657f6b27e2156818bc445c885d87fc0e
> Cr-Commit-Position: refs/heads/master@{#295687}

TBR=joaodasilva@chromium.org,eroman@chromium.org,pneubeck@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=413219

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

Cr-Commit-Position: refs/heads/master@{#295702}

Initial code review: https://codereview.chromium.org/580553002/

BUG=none

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

Cr-Commit-Position: refs/heads/master@{#295701}

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/3c29c4d..65b4895

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=stephana@google.com

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

Cr-Commit-Position: refs/heads/master@{#295700}

Apply Xcode 6.0 fix to all following versions of Xcode (it's great that
gyp does support python string ordering).

BUG=385030

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

Cr-Commit-Position: refs/heads/master@{#295699}

The missing symbol on linux was: _ZN6webrtc18videocapturemodule16VideoCaptureImpl6CreateEiPKc

TBR=sergeyu@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#295698}

Previously, it had a fixed width, causing it to be very short on the
wider app list window style. Now it extends (almost) all the way to the
edge regardless of the app list's width.

BUG=415461

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

Cr-Commit-Position: refs/heads/master@{#295697}

BUG=159000
TBR=erg@chromium.org,mad@chromium.org
NOTRY=true

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

Cr-Commit-Position: refs/heads/master@{#295696}

Remove unused #includes, lower dependencies etc. in preparation for componentisation.

BUG=384873

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

Cr-Commit-Position: refs/heads/master@{#295695}

BUG=none
R=glider@chromium.org
NOTRY=true

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

Cr-Commit-Position: refs/heads/master@{#295694}

BUG=415949
TBR=ericzeng@chromium.org
NOTRY=true

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

Cr-Commit-Position: refs/heads/master@{#295693}

BUG=162827

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

Cr-Commit-Position: refs/heads/master@{#295692}

Revert of Update V8 to version 3.29.78 (based on bleeding_edge revision r24051). (patchset #1 id:1 of https://codereview.chromium.org/583023003/)

Reason for revert:
This is causing crashes in Perf bots.

See crbug.com/415943 for details.

Original issue's description:
> Update V8 to version 3.29.78 (based on bleeding_edge revision r24051).
>
> TBR=machenbach@chromium.org
>
> Committed: https://crrev.com/48c3e5ebfd2640061ed488216f1d275d17c887bd
> Cr-Commit-Position: refs/heads/master@{#295676}

TBR=machenbach@chromium.org,v8-autoroll@chromium.org
NOTREECHECKS=true
NOTRY=true

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

Cr-Commit-Position: refs/heads/master@{#295691}

Cr-Commit-Position: refs/heads/master@{#295690}

Revert of mojo: Add deserialization to python structs. (patchset #3 id:40001 of https://codereview.chromium.org/588493002/)

Reason for revert:
Linux Tests mojo_python_bindings_tests failure
http://build.chromium.org/p/chromium.linux/builders/Linux%20Tests/builds/13712

======================================================================
ERROR: testDefaultsTestSerializationDeserialization (bindings_serialization_deserialization_unittest.SerializationDeserializationTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/mnt/data/b/build/slave/Linux_Tests/build/src/mojo/python/tests/bindings_serialization_deserialization_unittest.py", line 125, in testDefaultsTestSerializationDeserialization
    v2 = sample_service_mojom.DefaultsTest.Deserialize(data, handles)
  File "/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/python/mojo/bindings/reflection.py", line 113, in Deserialize
    serialization_object.Deserialize(fields, data, handles)
  File "/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/python/mojo/bindings/serialization.py", line 82, in Deserialize
    (_, version) = HEADER_STRUCT.unpack_from(data)
TypeError: unpack_from() argument 1 must be string or read-only buffer, not bytearray

Original issue's description:
> mojo: Add deserialization to python structs.
>
> This is a reland of https://codereview.chromium.org/578263003 using
> buffer instead of memoryview, because on some early version of python
> 2.7, memoryview cannot be used with struct.unpack_from.
>
> BUG=415491
> R=sdefresne@chromium.org,cmasone@chromium.org
>
> Committed: https://crrev.com/37dd5f484593b1c8b92b825a9648edf7fb6944e6
> Cr-Commit-Position: refs/heads/master@{#295684}

TBR=cmasone@chromium.org,sdefresne@chromium.org,qsr@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=415491

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

Cr-Commit-Position: refs/heads/master@{#295689}

Revert of mojo: fix python unittests. (patchset #1 id:1 of https://codereview.chromium.org/586683002/)

Reason for revert:
Linux Tests mojo_python_bindings_tests failure
http://build.chromium.org/p/chromium.linux/builders/Linux%20Tests/builds/13712

======================================================================
ERROR: testDefaultsTestSerializationDeserialization (bindings_serialization_deserialization_unittest.SerializationDeserializationTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/mnt/data/b/build/slave/Linux_Tests/build/src/mojo/python/tests/bindings_serialization_deserialization_unittest.py", line 125, in testDefaultsTestSerializationDeserialization
    v2 = sample_service_mojom.DefaultsTest.Deserialize(data, handles)
  File "/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/python/mojo/bindings/reflection.py", line 113, in Deserialize
    serialization_object.Deserialize(fields, data, handles)
  File "/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/python/mojo/bindings/serialization.py", line 82, in Deserialize
    (_, version) = HEADER_STRUCT.unpack_from(data)
TypeError: unpack_from() argument 1 must be string or read-only buffer, not bytearray

Original issue's description:
> mojo: fix python unittests.
>
> TBR=sdefresne@chromium.org
>
> Committed: https://chromium.googlesource.com/chromium/src/+/80ced42d624a1eec8e8a2f0c0c760ab576d65ead

TBR=sdefresne@chromium.org,qsr@chromium.org
NOTREECHECKS=true
NOTRY=true

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

Cr-Commit-Position: refs/heads/master@{#295688}

This prepares for the new CertDatabase keyed service, which will have stricter threading restrictions. https://codereview.chromium.org/419013003/
Before, ONCCertificateImporter accessed the NSSCertDatabase from the UI thread and blocked on certificate store operations.
Now, the import itself is asychronous and calls back on completion.

While there, this also removes the GMock of the importer.

This is a reland of f08303014b165f6013fe33198cd798ebd9a4e925
  refs/heads/master@{#295534}
with the fixed destruction order in ONCCertificateImporterImplTest.

The fix was reviewed in
  https://codereview.chromium.org/589443002/

TBR=eroman@chromium.org
BUG=413219

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

Cr-Commit-Position: refs/heads/master@{#295687}

Move PageTransition enumeration from //content/public/common to //ui/base
so that it can be used by iOS and layered components that are used on iOS.

BUG=372373
R=avi@chromium.org, ben@chromium.org, wfh@chromium.org
TBR=ben, nasko

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

Cr-Commit-Position: refs/heads/master@{#295686}

TBR=sdefresne@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#295685}

This is a reland of https://codereview.chromium.org/578263003 using
buffer instead of memoryview, because on some early version of python
2.7, memoryview cannot be used with struct.unpack_from.

BUG=415491
R=sdefresne@chromium.org,cmasone@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#295684}

Revert of Make ONCCertificateImporter async. (patchset #5 id:190001 of https://codereview.chromium.org/547553005/)

Reason for revert:
Use-after-free.
https://code.google.com/p/chromium/issues/detail?id=415916

Original issue's description:
> Make ONCCertificateImporter async.
>
> This prepares for the new CertDatabase keyed service, which will have stricter threading restrictions. https://codereview.chromium.org/419013003/
>
> Before, ONCCertificateImporter accessed the NSSCertDatabase from the UI thread and blocked on certificate store operations.
>
> Now, the import itself is asychronous and calls back on completion.
>
> While there, this also removes the GMock of the importer.
>
> BUG=413219
>
> Committed: https://crrev.com/bc656c0e7b7bd67fb28e5a880d21b9510ebd3e3a
> Cr-Commit-Position: refs/heads/master@{#295534}

TBR=joaodasilva@chromium.org,eroman@chromium.org,pneubeck@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=413219

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

Cr-Commit-Position: refs/heads/master@{#295683}

TBR=kjellander@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#295682}

It was using chromium resources when it should have used chrome resources, and vice versa.

BUG=None

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

Cr-Commit-Position: refs/heads/master@{#295681}

For idle times < 1000us, we sent notifications with 0ms. This patch
adds a check for idle_time_in_ms to be non-zero and re-enables the
feature by default.

R=jochen@chromium.org
BUG=414815

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

Cr-Commit-Position: refs/heads/master@{#295680}