GitLab

BUG=455084

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

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

BUG=428729

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

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

AwURLRequestContextGetter::SetKeyOnIO now calls into the
DataReductionProxyAuthRequestHandler that is managed by
DataReductionProxyIOData.

BUG=453519

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

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

This is a pure code moving around CL.  No logic change at all.

So we add helper functions to pack/unpack strings to/from a bucket, so
the code generator is now much simpler.

This is to get the pack/unpack functions to be used by glGetUniformIndices.

BUG=429053
TEST=gpu_unittests, webgl_conformance
R=sievers@chromium.org

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

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

cache sends a CHLO and another connection that doesn't wait to load
server config from disk cache and sends INCHOATE_HELLO.

This is not enabled. QuicStreamFactory tests with this flag enabled
and disabled. Tested chrome and all unit tests with this flag enabled.

R=rch@chromium.org

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

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

This unindents the lines about PID/network namespaces, since those now
apply to both the setuid or unprivileged namespace sandbox.

BUG=312380

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

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

Making the (deprecation-in-progress) congestion field write only.

This is a bit of a hack because technically it's "required" for
QUIC_VERSION_23 but as all the internal servers and all of chrome builds
picking up this change consider the field optional despite the version,
it should just Work (TM).

I was hoping to stop sending the congestion fields for v24 clients but
that is not safe until the internal server fleet doesn't consider it a
required field for negotiated()=true.  Bah.

No longer reading the (deprecated) congestion fields for QUIC.

Merge internal change: 85158634
https://codereview.chromium.org/898233003/

Improving junk packet handling for QUIC by ignoring packets with client
port 0.

Merge internal change: 85067984
https://codereview.chromium.org/885713009/

Removing deprecated flag FLAGS_quic_use_initial_rtt_for_stats.

Merge internal change: 84999682
https://codereview.chromium.org/883393008/

Adding more error logging for failed QUIC writes.

Merge internal change: 84989992
https://codereview.chromium.org/898243002/

Further cleanup to QuicAckNotifier and QuicAckNotifierManager.
No functional change.

Merge internal change: 84784925
https://codereview.chromium.org/880403006/

Remove an unneeded hash_set from QuicAckNotifierManager.
No change in behavior.

Merge internal change: 84637544
https://codereview.chromium.org/872403007/

Minor cleanup and optimization of QuicConnection::IsConnectionClose.

Merge internal change: 84624803
https://codereview.chromium.org/903973002/

Rename QuicAckNotifier's AddSequenceNumber to OnSerializedPacket.
No functional change.

Merge internal change: 84624660
https://codereview.chromium.org/867293004/

R=rch@chromium.org

Committed: https://crrev.com/ff331b65e0aef293ed6f1d0954b83ae17cc66d28
Cr-Commit-Position: refs/heads/master@{#315086}

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

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

BUG=456214

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

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

Revert of Linux: Enable multi-threading with the gold linker. (patchset #2 id:20001 of https://codereview.chromium.org/894463003/)

Reason for revert:
Several CrOS folks are reporting linker errors locally, even though the CrOS buildbots are green.

Original issue's description:
> Linux: Enable multi-threading with the gold linker.
>
> BUG=161942
>
> Committed: https://crrev.com/226488abee2adbdb6e54e5bfca1c3b2f52f2da2d
> Cr-Commit-Position: refs/heads/master@{#314991}

TBR=thakis@chromium.org,tzik@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=161942

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

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

TBR=machenbach@chromium.org

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

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

Need to turn it off to reduce the power used by the display when idling.

BUG=chrome-os-partner:35458
TEST=Wait for the displays to turn off and check with 'modetest -c' that the DPMS state is off

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

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

We want to restrict certain behavior (such as network reporting) to
auto-launched kiosk apps. This change updates KioskAppManager to track
whether a given app was auto-launched (which is different from whether a
given app is *currently* set to auto-launch, because policies can change
while an app is running).

BUG=452968

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

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

Instead of using the id of the worker-thread to keep track of the thread to
run the image-loader callbacks on, use the task-runner of the thread instead.

BUG=none
R=mkwst@chromium.org

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

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

BUG=441466

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

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

BUG=456001

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

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

If a quad is fully transparent then it shouldn't block overlays. For
now we allow some SOLID_COLOR quads through

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

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

Cause:
The crash opt-in setting is sometimes controlled by domain policy, which will
prevent the user from enabling it.  However, this is not communicated to the
user properly.

Fix:
- Shows a tooltip "This setting is controlled by your domain policy" if
  necessary.
- Hides the check box if the domain doesn't allow usage stats collection
  as an empty disabled check box offers no value to the user.

BUG=388864

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

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

* Use AppBannerInfoBars instead of AppBannerViews for Android
  app promos.

- Button doesn't install anything.

- Layout is still incorrect.

BUG=453170

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

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

Currently when user signs out GCMInvalidationBridge stays registered
with GCMDriver preventing it from shutting down.
GCMInvalidationBridge needs to unregister when user signs out.

BUG=446349
R=jianli@chromium.org

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

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

BUG=none

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

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

This reverts commit 101f015e.

Causing http://crbug.com/456224.

While this original CL should be fine, it's not; we need
to revert until we understand why.

R=marshall@chromium.org
BUG=456224

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

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

BUG=

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/02c8fd0..d042358

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

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

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

Fix a use after free crasher in the ReadAsync task initiated on Windows by the FileStream::Context::Read operation.

The crash was reported by the DrMemory bot and based on the stack happens because the OVERLAPPED structure passed into
the ReadFile call is invalid.

Proposed fix is the following:-
1. Have two flags io_complete_for_read_received_ and async_read_completed_ which track whether the IO completion was
   received for a Read and whether we received a notification on the calling thread that the ReadFile call returned.
   We invoke the user callback only when both these flags are true.

2. We have another flag async_read_initiated_ which is set to true if an asynchonous Read was initated. We use this
   to not set the async_in_progress_ flag to false until both notifications as per 1 above are received.

3. All flags above are reset when we invoke the user callback. That now happens in the InvokeUserCallback function.

4. We need to save the result in a member as the callback is invoked later.

5. Removed the Weak pointer member from the Context class as this is not needed because the Context instance should remain
   valid until the pending Read operation completes.

BUG=455066

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

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

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

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

Revert of Roll WebRTC 8220:8272, Libjingle 8221:8270 (patchset #1 id:1 of https://codereview.chromium.org/902183002/)

Reason for revert:
Suspected to cause the test failures on Mac:
http://build.chromium.org/p/chromium.mac/builders/Mac10.9%20Tests%20%28dbg%29/builds/2725

Original issue's description:
> Roll WebRTC 8220:8272, Libjingle 8221:8270
>
> WebRTC 8220:8272
> Changes: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+log/cfe4b49..0506804
>
> Libjingle 8221:8270
> Changes: https://chromium.googlesource.com/external/webrtc/trunk/talk.git/+log/18f533d..184c120
>
> TBR=magjed@chromium.org
>
> Committed: https://crrev.com/53b0c831ff6731665a6881fe380e8097402ccf0b
> Cr-Commit-Position: refs/heads/master@{#315066}

TBR=magjed@chromium.org,tommi@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

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

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

This CL ports the GATT API bindings from the D-Bus API in our Chrome OS fork to
the API definition in doc/gatt-api.txt in BlueZ upstream.

TBR=keybuk@chromium.org
BUG=440968

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

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

1) Add a drop down revealing a popup panel with import details (with is a WIP, but the current state should be sufficient to land).

2) Update scanner to publish updates on all scan activity (so the details reflects valid information while scanning). In doing this I pulled all of the deduping logic up out of ScanResult into MediaScanner...improving separation of concern between the classes.

3) Rework update model to eliminate a flicker resulting from the brief update to show zero results when a file selection is transitioning from n to n+/-1.

Sadly, this change results in a net loss of readability, especially WRT having good clear decision points. I've added comments to help out in this respect.

FYI, I'm OOO for the rest of the day. I think this CL is good to land, rough though it may be.

In a followup CL:

1) I'll make some subsequent improvements to unittests in the wake of removing the "GetCommandUpdate" coverage (removed because GetCommandUpdate exists no more).

2) This change still exhibits one odd behavior, ignoring entirely a selection when it consistes of all deduped files...in that case we *do* want to see zero results. Will address at the time of test improvements.

BUG=420680
TEST=browser_test: FileManagerJsTest.*

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

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

This CL:
* Enables chrome.networkingPrivate in chrome://network
* Replaces networkConfig with direct calls to chrome.networkingPrivate
* Moves the getShillProperties implementation (which is not available in networkingPrivate) into the network_ui.js and network_ui.cc files

BUG=453895
For webui resources:
TBR=xiyuan@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/ea6ed3c31..722374e

TBR=apavlov@chromium.org,dpranke@chromium.org

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

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

Revert of Landing Recent QUIC Changes. (patchset #1 id:1 of https://codereview.chromium.org/903013002/)

Reason for revert:
Broke the build

Original issue's description:
> Landing Recent QUIC Changes.
>
> Making the (deprecation-in-progress) congestion field write only.
>
> This is a bit of a hack because technically it's "required" for
> QUIC_VERSION_23 but as all the internal servers and all of chrome builds
> picking up this change consider the field optional despite the version,
> it should just Work (TM).
>
> I was hoping to stop sending the congestion fields for v24 clients but
> that is not safe until the internal server fleet doesn't consider it a
> required field for negotiated()=true.  Bah.
>
> No longer reading the (deprecated) congestion fields for QUIC.
>
> Merge internal change: 85158634
> https://codereview.chromium.org/898233003/
>
> Improving junk packet handling for QUIC by ignoring packets with client
> port 0.
>
> Merge internal change: 85067984
> https://codereview.chromium.org/885713009/
>
> Removing deprecated flag FLAGS_quic_use_initial_rtt_for_stats.
>
> Merge internal change: 84999682
> https://codereview.chromium.org/883393008/
>
> Adding more error logging for failed QUIC writes.
>
> Merge internal change: 84989992
> https://codereview.chromium.org/898243002/
>
> Further cleanup to QuicAckNotifier and QuicAckNotifierManager.
> No functional change.
>
> Merge internal change: 84784925
> https://codereview.chromium.org/880403006/
>
> Remove an unneeded hash_set from QuicAckNotifierManager.
> No change in behavior.
>
> Merge internal change: 84637544
> https://codereview.chromium.org/872403007/
>
> Minor cleanup and optimization of QuicConnection::IsConnectionClose.
>
> Merge internal change: 84624803
> https://codereview.chromium.org/903973002/
>
> Rename QuicAckNotifier's AddSequenceNumber to OnSerializedPacket.
> No functional change.
>
> Merge internal change: 84624660
> https://codereview.chromium.org/867293004/
>
> R=rch@chromium.org
>
> Committed: https://crrev.com/ff331b65e0aef293ed6f1d0954b83ae17cc66d28
> Cr-Commit-Position: refs/heads/master@{#315086}

TBR=rch@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

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

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

It doesn't need to run in Chrome's browser_tests.

BUG=388893
TEST=browser_tests and extensions_browsertests for Bluetooth*

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

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

NSWindowFullScreenButton is a 10.7 addition and calling
-standardWindowButton: with it on 10.6 results in log
spam of the form:

"unrecognized button type 7"

BUG=none

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

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

This is a re-land of https://codereview.chromium.org/862113005/

This CL introduces JNI plumbing for one method of the FakeServer's
verification API. One test (testUploadTypedUrl) has been added to use
the API; it types a URL on the client and verifies that the URL was
sent to the server.

BUG=NONE

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

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

The number of times the current received packet had a lower sequence number
than the previously received packet sequence number, and the size of the
current packet is larger than the size of the previous packet.

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

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

Making the (deprecation-in-progress) congestion field write only.

This is a bit of a hack because technically it's "required" for
QUIC_VERSION_23 but as all the internal servers and all of chrome builds
picking up this change consider the field optional despite the version,
it should just Work (TM).

I was hoping to stop sending the congestion fields for v24 clients but
that is not safe until the internal server fleet doesn't consider it a
required field for negotiated()=true.  Bah.

No longer reading the (deprecated) congestion fields for QUIC.

Merge internal change: 85158634
https://codereview.chromium.org/898233003/

Improving junk packet handling for QUIC by ignoring packets with client
port 0.

Merge internal change: 85067984
https://codereview.chromium.org/885713009/

Removing deprecated flag FLAGS_quic_use_initial_rtt_for_stats.

Merge internal change: 84999682
https://codereview.chromium.org/883393008/

Adding more error logging for failed QUIC writes.

Merge internal change: 84989992
https://codereview.chromium.org/898243002/

Further cleanup to QuicAckNotifier and QuicAckNotifierManager.
No functional change.

Merge internal change: 84784925
https://codereview.chromium.org/880403006/

Remove an unneeded hash_set from QuicAckNotifierManager.
No change in behavior.

Merge internal change: 84637544
https://codereview.chromium.org/872403007/

Minor cleanup and optimization of QuicConnection::IsConnectionClose.

Merge internal change: 84624803
https://codereview.chromium.org/903973002/

Rename QuicAckNotifier's AddSequenceNumber to OnSerializedPacket.
No functional change.

Merge internal change: 84624660
https://codereview.chromium.org/867293004/

R=rch@chromium.org

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

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

Switch Chrome on Android to use libc++ instead of stlport. WebView AOSP
builds will continue to use stlport for the time being.

Note that this change is exploratory, and likely to be reverted before
branch point (ensuring WebView and Chrome remain in lockstep with
respect to standard library dependencies).

BUG=427718
TBR=cpu@chromium.org

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

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

CloudPolicyClient previously only allowed a single outstanding request at once,
and issuing a new request would cause the previous request to silently fail.
This behavior made sense for policy fetches, but not for other types of
out-of-band requests (status uploads, certificate uploads, etc).

CloudPolicyClient now allows multiple requests at once, and differentiates
between policy-fetch/registration related requests and out-of-band requests.

BUG=452563

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

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

BUG=417463
R=thakis@chromium.org

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

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

BUG=none

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

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