GitLab

BUG=none
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=49298

Review URL: http://codereview.chromium.org/2749004

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

a 'minimal' set of rectangles that covers the changed region.

BUG=none
TEST=new unittests added

Review URL: http://codereview.chromium.org/2714007

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

Calling NaClAppDtor() is unnecessary because the process is about to
exit anyway, and it happens only in the error path because
NaClAppDtor() is not able to shut down running NaCl threads safely.
There is a TODO for its removal in the NaCl tree.

This is related to http://codereview.chromium.org/2659002.

BUG=http://code.google.com/p/nativeclient/issues/detail?id=560
TEST=nacl_ui_test

Review URL: http://codereview.chromium.org/2753002

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

Also added paths to DirectX SDK to common.gypi because ANGLE requires headers and libraries not part of the Windows Platform SDK.

TEST=trybots
BUG=none

Review URL: http://codereview.chromium.org/2685002

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

Removed redundent version from update string.
Added a line break to the ChromeOS version on test builds to improve appearence
(select and drag to see complete version).
Added margins to multiline edit fields without borders to make them match
single line edit fields in positioning.
Fixed an issue with setting the initial focus for GTK based windows. This
fixes the problem of the cursor showing up in the version field.

BUG=chromium-os:3571 1589 2967
TEST=none

Review URL: http://codereview.chromium.org/2620003

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

BUG=none
TEST=Build succeeds using 10.6 SDK

Review URL: http://codereview.chromium.org/2790005

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

It was reverted due to breaking a valgrind test which has since been disabled.

Review URL: http://codereview.chromium.org/2763004

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

I got it to a point where we show HTML content from the extension within an infobar, but there are a few things missing.

Missing pieces:
- Infobar does not size based on the height of its content (between 36px and 72px).
- Infobar content always has a fixed width, but needs to fill out into the available infobar area.
- I've put in code to asynchronously fetch the image for the Infobar, but it needs to be shown and hooked up to the ExtensionAction context menu (same context menu we use for the browser action container icons).
- The background for HTML content should have a gradient, but is just solid blue at the moment.

BUG=39916
TEST=Infobars should show up on GTK.

Review URL: http://codereview.chromium.org/2753005

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

it crashes randomly.
BUG=none
TEST=dev platform

Review URL: http://codereview.chromium.org/2765005

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

Modify bsdiff 4.3 to create goobsdiff, which is appropriate for use as the Mac
binary differ/patcher.

BUG=45017
TEST=none


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

binary differ/patcher.

BUG=45017
TEST=none
Review URL: http://codereview.chromium.org/2716005

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

The reason I'm collapsing all the "create left/center/right sections" functions into one is because in the upcoming patch that moves "stop" from "go" onto "reload", I'll need to reorder bits within this unified block, and that change will be easier to review if this mechanical change lands first.

BUG=none
TEST=none
Review URL: http://codereview.chromium.org/2781003

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

 instead of adding window_contents_ as grab widget.
 Explicitly adding grab widget was confusing gtk and the event has been handled by grab widget rather than by the window the event is forwarded to.
 This CL fixes it by removing all grab widget and let gtk propagage event in target window.

BUG=none
TEST=open screen locker, type wrong password to show message bubble,
then close it by clicking close (x) button.

Review URL: http://codereview.chromium.org/2788001

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

BUG=46186
TEST=Mac Valgrind UI 3

Review URL: http://codereview.chromium.org/2736002

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

displaying in a tab.  This could happen when the user types in a new query before
the AlternateNavURLFetcher was able to resolve the URL.

BUG=43378
TEST=Load two single word domains within quick succession in the same tab.
Review URL: http://codereview.chromium.org/2747006

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

Review URL: http://codereview.chromium.org/2748005

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

BUG=chromium-os:3462
TEST=none
Review URL: http://codereview.chromium.org/2758004

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

BUG=none
TEST=trybots

Review URL: http://codereview.chromium.org/2770005

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

Review URL: http://codereview.chromium.org/2729002

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

Since some work is needed on Linux and Mac to use the new translate infobar delegate, I created a new version of the refactored classes instead of replacing them. Once Linux and Mac use the new classes, we can make the swicth.
The TranslateInfoBarDelegate now contains all states, so there is no more logic on the infobar classes.
I broke down the single infobar class into multiple ones (there is now an infobar for each state: before translate, translating, after translate, error): it makes the code simpler.
I had to fix ReplaceInfoBar on Windows as it was not working properly.

BUG=40828
TEST=Test thoroughly the translate feature.

Review URL: http://codereview.chromium.org/2602003

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

Questions:
- Is deltaY the correct field in which to store this event value?
- Is the 'wheel' addEventListener hook broken in the first place? Doesn't work for me on OSX, but might be working in Windows/Linux and don't know if this change would break it for those.
- It is necessary to cancel event bubbling for the onmousewheel event?
Review URL: http://codereview.chromium.org/2456004

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

Review URL: http://codereview.chromium.org/2718003

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

BUG=none
TEST=none
TBR=hamaji

Review URL: http://codereview.chromium.org/2724011

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

BUG=46111
TEST=none

Review URL: http://codereview.chromium.org/2785001

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

the comment, I suspect it used to be private but got moved around and now it's
public.

TEST=none
BUG=none
Review URL: http://codereview.chromium.org/2740008

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

BUG=none
TEST=none

Review URL: http://codereview.chromium.org/2749004

TBR=sergeyu@chromium.org
Review URL: http://codereview.chromium.org/2724010

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

RFC 2744 does not require string types to be NULL terminated, because their length is explicitly passed as part of the gss_buffer_desc (see Section 3.2.). As such, when printing error strings, the length should be explicitly stated. This is shown in the example code of gss_display_status in Section 5.11. While in practice this is the case (at least from checking MIT and Heimdal's error handling code), it doesn't hurt to be defensive.

In addition, there are some conditions where value may be NULL or length may be 0, so make sure to check for these prior to calling StringPrintf, so as not to crash. Finally, for the extreme defensive case, make sure that the length (which is a size_t) is capped at INT_MAX prior to printing.

Contributed by ryan.sleevi@gmail.com

BUG=33033
TEST=None

Review URL: http://codereview.chromium.org/2646004

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

BUG=none
TEST=none

Review URL: http://codereview.chromium.org/2749004

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

BUG=46177
TEST=CrOs Valgrind 3 goes green.

Review URL: http://codereview.chromium.org/2739004

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

Before we carefully used the app (Chrome/Chromium) name, but:
1) these are for internal debugging use only, so why compute this name
2) on Linux the thread ids are limited to 16 characters, and "Chromium"
   is too long

Review URL: http://codereview.chromium.org/2741003

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

2. Timed updates of the database based on the number of sites changed.

BUG=None
TEST=TopSitesTest

Review URL: http://codereview.chromium.org/2746002

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

for an invalid program. The check is already in
UseProgram. If that check passes the program is
"installed" at which point it, even if LinkProgram
is called and fails, marking the program as invalid,
the previously installed program is still in use
until UseProgram is called.

TEST=none
BUG=none

Review URL: http://codereview.chromium.org/2770007

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

Review URL: http://codereview.chromium.org/2746005

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

Change the layout of content and general page. Use the horizontal title and content layout for each option group.

BUG=18949
TEST=Open the option window, check the content and general pages.

Patch by Ningxin Hu <ningxin.hu@intel.com>.

Review URL: http://codereview.chromium.org/1969006

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

BUG=21149

TBR=agl@chromium.org
Review URL: http://codereview.chromium.org/2711008

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

BUG=46114

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

Convert from using NSBitmapImageRep plus context manipulation to
-[NSImage lockFocus].

BUG=none
TEST=Browser actions look the same as before.

Review URL: http://codereview.chromium.org/2635005

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

Excluding them individually appeared to be too hard...
BUG=38575
Review URL: http://codereview.chromium.org/2779006

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

BUG=None
TEST=net_unittests, and ran chrome with net_internals DNS.

Review URL: http://codereview.chromium.org/2721005

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

A DictionaryValue we need to keep was being autoreleased in the Mac message loop. The import process uses only C++ code, so we just make a deep copy of the (very small, 5 strings) Dictionary when it's passed into the bridge.

BUG=46003
TEST=import works on Mac OS 10.6 and 10.5
Review URL: http://codereview.chromium.org/2715006

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