GitLab

C++11 provides unique_ptr<T[]>, and Chromium has implemented
scoped_ptr<T[]> to match its behavior during the transition period. As a
result, scoped_array<T> is now redundant and is being removed.

BUG=171111

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

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

This CL is the 4th in a series of changes that straddle webkit vs chromium repositories.
1) WK: Declare new virtual createSnapshotFile/didCreateSnapshotFile public api methods.
2) CR: Implement the new create method such that the didCreate method is invoked in response.
3) WK: Use the new create method and implement the new didCreate method.
4) CR: Cleanup the obsolete/deprecated blocks of code.

BUG=174200

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

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

BUG=170229
TEST=by hand as per repro on bug

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

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

This fixes a bug when reading from a HTTP mount, if the Content-Length header
is not available.

It occurs because the code was caching a pointer to a std::vector's buffer,
after it had resized it.

BUG=none
R=noelallen@chromium.org

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

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

BUG=

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

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

Hopefully this is the last one before the scoped_array removal can land.

BUG=171111

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

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

BUG=171111

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

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

AppCacheExecutableHandlers - parse manifest file and store/retrieve the 'bit' indicating executable.

BUG=101800

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

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

Local environment:
$ cat ~/.gyp/include.gypi
{
  'variables': {
    'debug_devtools': 1,
  },
}

The build fails with the following error:

../../chrome/browser/ui/webui/devtools_ui.cc: In constructor 'DevToolsUI::DevToolsUI(content::WebUI*)':
../../chrome/browser/ui/webui/devtools_ui.cc:239:59: error: cannot allocate an object of abstract type '{anonymous}::LocalhostDataSource'
../../chrome/browser/ui/webui/devtools_ui.cc:172:7: note:   because the following virtual functions are pure within '{anonymous}::LocalhostDataSource':
../../content/public/browser/url_data_source.h:40:23: note: 	virtual std::string content::URLDataSource::GetSource() const

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

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

BUG=none
TEST=bisect something where the regression range includes a blink roll.
Outputs working link for blink regression range.

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

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

If a layer has a replica, which has a mask, then the layer itself does not
need to clip to its bounding box. This dcheck verified that it would.

Add some new unit tests and pixel tests that verify behaviour of the mask
texture with masked replicas. The current behaviour is correct. The
replica is masked at the origin of the replicated layer.

Tests:
LayerTreeHostImplTest.ReflectionMaskLayerWithDifferentBounds
LayerTreeHostImplTest.ReflectionMaskLayerForSurfaceWithUnclippedChild
LayerTreeHostImplTest.MaskLayerForSurfaceWithClippedLayer
LayerTreeHostMasksPixelTest.MaskOfLayer
LayerTreeHostMasksPixelTest.ImageMaskOfLayer
LayerTreeHostMasksPixelTest.MaskOfClippedLayer
LayerTreeHostMasksPixelTest.MaskWithReplica
LayerTreeHostMasksPixelTest.MaskWithReplicaOfClippedLayer
LayerTreeHostMasksPixelTest.MaskOfReplica
LayerTreeHostMasksPixelTest.MaskOfReplicaOfClippedLayer

R=enne
BUG=171734

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

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

> Add a virtual keyboard webui at chrome://keyboard/
> 
> This is just the old virtual keyboard that was removed a couple of years
> ago, being used as a temporary stand-in.
> 
> BUG=222801
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=194579
> 
> Review URL: https://codereview.chromium.org/13652010

TBR=bryeung@chromium.org

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

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

This is just the old virtual keyboard that was removed a couple of years
ago, being used as a temporary stand-in.

BUG=222801

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=194579

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

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

QUIC crypto: move config objects.

Currently the client and server configs are setup and torn-down for each
connection. Since they are supposed to be per-client and per-server objects,
this change makes them parameters that are passed into the connection

Merge internal change: 44269387

QUIC crypto steps 6 and 7: per-server strike register.

This change adds a per-server strike-register that allows the server to
complete 0-RTT connections if the client has enough information cached.

Due to the fact that the per-server and per-client objects
(QuicCryptoServerConfig and QuicCryptoClientConfig) are currently setup and
torn down for each connection, there's no tests in this change for a 0-RTT
handshake because we can't do one yet. The next change will move these objects
into the right place so that 0-RTT handshakes can be tested.

This change also reminded me why I had a server nonce: without it the server
cannot terminate any connections if the strike-register fails. So the server
nonce is firmly back.

Merge internal change: 44228897

R=rch@chromium.org

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

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

BUG=231725
TEST=none

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

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

windows.h defines BEGIN_INTERFACE -- we need to undefine it before we redefine
it.

BUG=none
R=sbc@chromium.org

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

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

It is flaky on linux too. The failure tends to reproduce more often on linux
swarm slaves than linux_rel but this is really race-condition like.

TBR=jeremya@chromium.org
BUG=223467

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

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

My http://crrev.com/192493 caused a regression:
FindBarView doesn't select-all on tab-switch focus restoration.

Restore the FindBarView::SearchTextfieldView class with RestoreFocus override.
( this restores the previous select-all behavior to fix the regression )

BUG=232290
TEST=Switching back to a tab that had the find bar focused (from a tab that didn't have the find bar focused) restores find bar focus and selects the entire text content.
R=finnur@chromium.org
TBR=ben@chromium.org

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

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

This provides a replacement for the inner class of SerializedVar for serializing a PP_Var. It achieves 2 purposes: 1) it refactors the design of SerializedVar::Inner which was growing increasingly messy (and was going to become nastier) 2) it provides support for serializing/deserializing dictionary and array PP_Vars. To serialize a PP_Var, first the transitive closure of that var is computed (all nodes referenced) and then each var in the transitive closure is written to the message. Some trickiness arises from the fact that a PP_Var can be either a primitive or a reference and references have to be maintained when transmitting over the wire. The comments in the header files provide a description of the approach in more detail.

BUG=

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

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

Previously P2PSocketHostTcp was calling Write() even when another write is
pending.

BUG=232046

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

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

Serializing calls to getAuthToken will eliminate unnecessary calls to
GAIA and consolidate UI flows.

There is are two queues of requests for each (extension, scopes)
tuple, one for the non-interactive IssueToken flow and one for the
interactive prompt.

BUG=228908
(step #1 of the bug description)

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

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

R=enne
BUG=171734

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

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

This change puts the final signatures containing doubles in place so that
they can be used by Blink code. After this change lands Blink will be updated
to call these methods instead of the temporary xxxFloat() methods.

BUG=227156
TEST=All existing tests still pass. No user visible behavior has changed.

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

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

logged into (or has been logged into).
re-submission of https://codereview.chromium.org/13903018

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

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

This short fix in the deprecated bluetooth ChromeOS implementation
shows a trusted device (but not paired) as paired. This allows testing
the Verbatim mouse in the current implementation.

BUG=232145
TEST=Connect to a Verbatim device (connected but not paired). Device shows up in the settings list.

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

Patch from Alex Deymo <deymo@chromium.org>.

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

With fixed include path.

TBR=cpu@chromium.org
BUG=225822

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

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

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

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

> Pepper: Autogenerate thunk for PPB_URL_Loader.
> 
> This requires another annotation:
>   "always_set_output_parameters"
> 
> This annotation ensures that the thunk implementation will zero-out the value of all output parameters on failure.
> 
> Tested:
>   Built chrome and browser_tests.
> 
> BUG=
> 
> Review URL: https://codereview.chromium.org/14007010

TBR=teravest@chromium.org

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

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

In r174057, enne@ added support for implicit testing to scoped_ptr<>. Removes
these in chrome/browser/extensions/.

BUG=232084

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

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

This is needed for stacking panels. The foreground panel windows will turn off the shadow while the background stack window will provider the shadow.

BUG=223155,177770
TEST=none due to no functionality change yet

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

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

This requires another annotation:
  "always_set_output_parameters"

This annotation ensures that the thunk implementation will zero-out the value of all output parameters on failure.

Tested:
  Built chrome and browser_tests.

BUG=

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

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

BUG=None
TEST=Ran tests locally with the flag.

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

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

In r174057, ajwong@ added support for implicit testing to scoped_ptr<>. Removes
these in ash/.

BUG=232084

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

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

In r174057, ajwong@ added support for implicit testing to scoped_ptr<>. Removes
these in content/.

BUG=232084

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

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

BUG=227416
TEST=WebMParserTest.ZeroPaddedStrings

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

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

R=danakj@chromium.org
BUG=231458

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

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

The method is logically const, all callers are already const, and I
wanted to call this from a const method.

BUG=

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

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

Persists-across-all-workspace flag should not cause workspace switch when
showing a transient child of a maximized transient parent.

BUG=232119
TEST=Covered in a new test.

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

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

The CL turned out to be innocent, the bots turned green without the revert.

> Revert 194578 "Add ash SessionStateDelegate"
> 
> > Add ash SessionStateDelegate
> > 
> > This CL refactors the ShellDelegate by adding a SessionStateDelegate to
> > which methods dealing with the session state can be moved. This cleans up
> > the huge ShellDelegate interface and paves the way for further Chrome OS
> > multiprofile work which will need to add several new methods related to
> > the session state.
> > 
> > This CL is only the first step. Several other methods should also move to
> > SessionStateDelegate but I do not want to overburden a single CL.
> > 
> > BUG=None
> > TEST=Manual and browser/unit tests
> > 
> > TBR=sky (for c/b/idle_chromeos.cc and c/chrome_browser_ui.gypi)
> > 
> > Review URL: https://codereview.chromium.org/14295008
> 
> TBR=bartfab@google.com
> 
> Review URL: https://codereview.chromium.org/14200034

TBR=gavinp@chromium.org

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

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

BUG=138275

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

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