GitLab

BUG=170375

Review URL: https://chromiumcodereview.appspot.com/23856005

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

BUG=285584

Review URL: https://chromiumcodereview.appspot.com/23803005

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

When we early-out of a frame that has no damage, we didn't
correctly detect if the frame had incomplete tiles. This
patch only updates the flag if a swap actually occurs.

To prevent this kind of bug in the future, this patch
always sets the did-swap-use-incomplete-tile flag when it is
known, rather than resetting it to false in one place and
hoping it will always be set again correctly in all cases.

The test times-out without this patch.

BUG=284810

Review URL: https://chromiumcodereview.appspot.com/23686011

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

This CL fixes the attached file 'size to big' error to show the maximum size allowed. Also reduced the size of the screenshot (when zoomed) slightly because on some platforms it doesn't fit into the feedback page width, displacing all the elements and moving one line down.

R=xiyuan@chromium.org
BUG=285993

Review URL: https://chromiumcodereview.appspot.com/23536038

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

By changing 1.f to 1.0, this allows the inverse contents
scale to cancel out properly in the transformation matrix.

BUG=259154

Review URL: https://chromiumcodereview.appspot.com/23572033

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

This CL tags every request NativeMessagingHostTest.All sends and process the responses out of order since the native messaging host does not guarantee the order. It also removes the call log because of the same reason - the order is not reliable unless the client wait for the resonse before sending another request.

Also with this CL NativeMessagingHost delays shutdown until all replies have been sent.

BUG=284704

Review URL: https://chromiumcodereview.appspot.com/23461030

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

Incorporate printer list from cloudprint and modify cloud print page to match
mocks, adding necessary functionality along the way.

BUG=286157

Review URL: https://chromiumcodereview.appspot.com/23903011

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

Start using BlobStorageContext (and family) as a replacement for BlobStorageController. The new class manages blob data by uuid and seperately maintains a mapping from of public blob urls to uuids.

Added a methods to create an additional mapping of old-style 'private urls' to new-style uuids to help with transitioning, blink needs to be retrofitted too, but that has to happen separately.

Changed blob processing classes to take BlobDataHandles as input instead of GURLs. Not long after IPC deserizliation, get a BlobDataHandle and pass that around. 
- FileSystemOperation.Write (and famlity) for FileWriter support 
- ResourceDispatcherHost (and famility) for blob url requests 
- (PostMessage handling and WebIntent plumbing ultimately needs to be treated in this way too) 


Switched to using string uuids instead of urls in IPC messages (and structs carried in ipc messages).
Until blink is retrofitted, the chromiums side supports both forms of identifying blobs (old-style private urls and new-style uuids).
- FileSystemHostMsg_Write 
- webkit_base::DataElement (and consumers: BlobData, ResourceRequestBody) 
- blob registry/building ipc messages 

Implemented the WebKit::WebBlobRegistryImpl such that it can be invoked on any renderer/worker thread. 

Got more explicit about naming: FileSystemURL vs BlobUUID vs PublicBlobURL.

Note: There are corresponding changes to be made in Blink, after which, the things annotated as deprecated in this CL can be removed.

TBR=inferno
BUG=174200

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

Review URL: https://chromiumcodereview.appspot.com/23223003

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

BUG=286446

Review URL: https://chromiumcodereview.appspot.com/23522025

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

Configure Views autofill dialogs to be closed when interstitial WebUI
is displayed (e.g. for SSL warnings prior to page load).

BUG=240575

Review URL: https://chromiumcodereview.appspot.com/23493025

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

Now a reserved name is not allowed on any component of the path.

BUG=none
test=net_unittests

R=asanka@chromium.org

Review URL: https://chromiumcodereview.appspot.com/23464037

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

Removing one of the events.

BUG=285999

Review URL: https://chromiumcodereview.appspot.com/23449027

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

The lifetime of the login webui can exceed that of the AppLaunchController,
so we need to do proper cleanup.

BUG=286406

Review URL: https://chromiumcodereview.appspot.com/23967008

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

Take advantage of the ASYNC variants of the event tracing macros when
tracing model association times.  This makes the model association
tracing much more reliable and easier to read.

BUG=286541

Review URL: https://chromiumcodereview.appspot.com/23907005

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

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

Previously, we were using the windows assets. As is, this is functional,
but will require further patches to get the button spacing correct.

BUG=281788

Review URL: https://chromiumcodereview.appspot.com/23993003

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

This CL implements DesktopMediaPicker for the Mac used by the Desktop Capture API to present the user with a list of sources (including live thumbnails) to use.

BUG=237907

Review URL: https://chromiumcodereview.appspot.com/23944003

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

This patch allows both the X11 and the Gtk message-pumps to be built
in a linux-gtk build. A subsequent patch will use the X11 message-pump
for the GPU process, while continue to use the Gtk message-pump for the
browser process.

BUG=145600
R=ccameron@chromium.org, oshima@chromium.org, piman@chromium.org, thakis@chromium.org

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

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

Changing the --enable-sctp-data-channels flag to --disable-sctp-data-channels so that SCTP is turned on by default.

BUG=248486

Review URL: https://chromiumcodereview.appspot.com/24042002

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

Blink trybots don't need to zip build.

TBR=dpranke
BUG=230080

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

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

This removes access of finder_options.output_profile_path from android_browser_backend.

BUG=269131
TEST=trybot

Review URL: https://chromiumcodereview.appspot.com/23466013

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

Allows java bookmark UIs to update to changes to the underlying model.

BUG=247381

Review URL: https://chromiumcodereview.appspot.com/23849003

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

-move command logging from http_handler to execute session command
-fix bug where didn't delete thread if session creation failed
-add logging for prefs
-LOG to driver log if on appropriate thread
BUG=none

Review URL: https://chromiumcodereview.appspot.com/23467010

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

BUG=77656

Review URL: https://chromiumcodereview.appspot.com/23471007

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

Attaching a file to feedback is crashing the app since we fail on the AttachedFile validate (since we don't actually send blobs over, the attachedFile.data field is always empty, which fails parameter validation on the Chrome side). The easiest way to fix it was to make the parameter optional, since we never read it anyway. It is only there to hold the data so that the feedback custom bindings can move it into attachedFileBlobUrl.

This CL also fixes the sys info, which wasn't being sent due to code that was ported over from the UI incorrectly. The new UI does not need to wait on system information collection, since that is now done via a separate call, making passing around the sys_info variable completely redundant, and in this case wrong.

R=asargent@chromium.org
BUG=285942,285938
TEST=We are able to attach a file and system information with a feedback report.

Review URL: https://chromiumcodereview.appspot.com/23458031

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

(Windows XP issue should be fixed by usrsctp DEPS roll: https://codereview.chromium.org/23960004/)

BUG=

Review URL: https://chromiumcodereview.appspot.com/23766013

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

R=asargent@chromium.org
BUG=285413
TEST=Launching feedback in Incognito/Guest mode works and the browser doesn't crash.

Review URL: https://chromiumcodereview.appspot.com/23523033

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

Depends on: https://codereview.chromium.org/23499006/

R=vandebo
BUG=151701

Review URL: https://chromiumcodereview.appspot.com/23717027

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

The "loading..." shield on OSX had the currently focused UI element
"shine through" - i.e. the entire dialog was hidden behind it except
for the one focused element.

This change makes the loading screen first responder for the duration
of its visibility, preventing focus rings.

R=sail@chromium.org
BUG=none

Review URL: https://chromiumcodereview.appspot.com/23477044

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

Today, PepperFileSystemBrowserHost errnoeously calls Shutdown() on the
FileSystemOperationRunner used by the FileSystemContext it gets.

This is incorrect; other code in Chrome (or another
PepperFileSystemBrowserHost) could be using the same FileSystemOperationRunner,
and its operations will be cancelled.

BUG=225441

Review URL: https://chromiumcodereview.appspot.com/23466012

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

Rework the way that new actions are logged in the counting policy so
that it properly handles the case where there are duplicate records that
weren't properly coalesced.  (Duplicate records could arise from
clearing URL history, for example.)

This comes at the cost of additional queries--an update always takes 2
queries, instead 1 or 2 as before.  However, benchmarking shows the cost
to be not much different (worst case was ~5% slower, other cases were
indistinguishable or even a bit faster).

BUG=279465

Review URL: https://chromiumcodereview.appspot.com/23907004

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

DWM was introduced with Windows Vista. DwmFlush seems to be sufficient to warm it up before lowering the token. DWM is required to present to a window with Vista and later and this allows us to do so with the GPU process sandbox enabled.

Enabling the sandbox for this case is not in this patch. Other things are required, such as making the sandbox broker allow certain event objects to be opened from the GPU process.

Review URL: https://chromiumcodereview.appspot.com/23992006

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

This is one of the top crasher on some samsung devices
https://crash.corp.google.com/search?query=android.media.MediaCodec.flush
Capture some exceptions while we are trying to look into the problem

BUG=278464

Review URL: https://chromiumcodereview.appspot.com/23321004

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

BUG=282594
TEST=Run ChromeOS with flags:
     --force-fieldtrials="ShowProfileSwitcher/AlwaysShow/"
     --multi-profiles

Review URL: https://chromiumcodereview.appspot.com/23583037

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

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

http://build.chromium.org/f/chromium/perf/dashboard/ui/changelog_blink.html?url=/trunk&range=157369:157375&mode=html
TBR=
BUG=

Review URL: https://chromiumcodereview.appspot.com/24054002

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

Readback doesn't have all its steps synchronized properly with
impl side painting enabled. This patch prevents the readback
commit from being swapped to screen and avoids using non-readback
commits for readback by adding a SynchronousReadbackState.

This patch also gives forced draws due to checkerboarding its
own ForcedReadback state that behaves independently of readback.

Previously readback and forced draws shared drawing mechanisms,
but we want readbacks to occur ASAP while we want forced draws
to occur inline with the normal frame scheduling.

BUG=276082

Review URL: https://chromiumcodereview.appspot.com/23503003

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

Specifically, AccessibilityEvent.TYPE_WINDOW_CONTENT_CHANGED should
be fired on the appropriate node any time the accessibility tree changes
- previously we were only firing it in a few circumstances. That guarantees
tools like uiautomatorviewer won't see a stale accessibility tree.

BUG=286374

Review URL: https://chromiumcodereview.appspot.com/23481011

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

See https://codereview.chromium.org/18024002 for a similar example.

Note that this API doesn't work on UNIX, so we still need an in-app method of
opening a new window.

BUG=252881
R=lambroslambrou@chromium.org

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

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

BUG=282671

Review URL: https://chromiumcodereview.appspot.com/23890011

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