| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
| |
and browser_tests on windows. The blamelist is big, so this is a guess.
Original description:
Add SubmenuView::magic_token_ and checks.Add MenuHostRootView::destroyed_flag_ and check.May help identify the cause of crbug.com/78792.Minor refactoring / code consolidation.BUG=78792TEST=noneReview URL: http://codereview.chromium.org/6879059
TBR=msw@chromium.org
Review URL: http://codereview.chromium.org/6882117
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82466 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82463 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
TBR=timurrrr
Review URL: http://codereview.chromium.org/6879107
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82461 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
heap allocated objects as intentionally leaked ones.
Annotate the histograms produced by {Histogram,BooleanHistogram,LinearHistogram,CustomHistogram}::FactoryGet(), as leaked.
Rename StatsHistogram::StatsHistogramFactoryGet to StatsHistogram::FactoryGet, annotate the result as leaky, update the suppressions.
BUG=79322
Review URL: http://codereview.chromium.org/6869009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82460 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
malicious" in the warning message for malicious downloads.
Review URL: http://codereview.chromium.org/6879097
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82459 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
Remove the paranoid "set noparent" from tools/valgrind/OWNERS
TBR=timurrrr
Review URL: http://codereview.chromium.org/6882113
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82455 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This is breaking FormStructureBrowserTest.DataDrivenHeuristics
Original description:
Refine inferring label algorithm to Autofill correctly.1. Exclude <script> and <option> tags from the "Infer" traversals. 2. Exclude <!-- comment --> elements from the "Infer" traversals. 3. Change IDS_AUTOFILL_ZIP_CODE_RE to be ... |^cap$ ... to strengthen the match.BUG=64818TEST=1. Go to gmail.com and click on the "Create an account" button. 2. Double click first name field and try to trigger Autofill. 3. Choose an item from the Autofill window. 4. Make sure the fields are correctly filled.Review URL: http://codereview.chromium.org/6873030
TBR=Takano.Naoki@gmail.com
Review URL: http://codereview.chromium.org/6879106
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82452 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82450 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
incorrect if the widget is not realized yet.
This fixes an issue in Chrome OS where the omnibox is not set to the base UI font when it is overridden by a locale-specific gtkrc.
BUG=chromium-os:14133
TEST=verified the font is correct in Japanese UI and English UI on Chrome OS netbook, and verified font in omnibox and downloaded item widget (which also uses ForceFontPointSize) is the same on Chrome on Linux with and without GTK+ theme before and after this change
Review URL: http://codereview.chromium.org/6883004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82449 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. Exclude <script> and <option> tags from the "Infer" traversals. 2. Exclude <!-- comment --> elements from the "Infer" traversals. 3. Change IDS_AUTOFILL_ZIP_CODE_RE to be ... |^cap$ ... to strengthen the match.
BUG=64818
TEST=1. Go to gmail.com and click on the "Create an account" button. 2. Double click first name field and try to trigger Autofill. 3. Choose an item from the Autofill window. 4. Make sure the fields are correctly filled.
Review URL: http://codereview.chromium.org/6873030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82448 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
to do model association.
BUG=80124
TEST=
Review URL: http://codereview.chromium.org/6879099
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82447 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
ref: https://bugs.webkit.org/show_bug.cgi?id=58864
BUG=None
TEST=None
Review URL: http://codereview.chromium.org/6881078
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82446 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=dmikurube,kinuko
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6882112
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82445 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=chromium-os:8681
TEST=manually on the chrome notebook
Review URL: http://codereview.chromium.org/6881082
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82442 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
truncate.
A writing operation which increases the file size more than "allowed growth" results in PLATFORM_FILE_ERROR_NO_SPACE.
BUG=74841
TEST=FileWriterDelegateTest.*,QuotaFileUtilTest.*,FileSystemOperationTest.*
Review URL: http://codereview.chromium.org/6725021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82441 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
inspector/debugger/live-edit, and
inspector/console/console-object-constructor-name
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82434 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82432 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This handles a corner case where all broker resources are deleted before the channel created IPC response is handled or if it never occurs. This doesn't solve all potential problems, but it helps with the ones we can influence.
An example scenario that this does not fix is if another broker resource is created by the plugin in the same renderer but the broker process is hung or thinks it has a channel to this renderer (channel names are based on the renderer). Even though a new PpapiBrokerImpl will be created as expected, the broker will not establish the channel.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6883101
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82431 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
In the case of bookmarks which don't have any subfolder, importer failed to
insert prefix to bookmark item pathes and called erase() on empty vector.
BUG=79978
TEST=unit_tests --gtest_filter=FirefoxImporterTest.\*
Review URL: http://codereview.chromium.org/6877080
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82429 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1) All event listeners are now added in js code (not in html).
2) Some unnecessary listeners have been removed.
3) Per HTML element: There is only one listener for the same event.
BUG=NONE
TEST=Print preview tab should behave as before.
Review URL: http://codereview.chromium.org/6880090
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82428 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
These tests are hanging/failing on Lucid.
BUG=none
TEST=interactive_ui_tests passes
Review URL: http://codereview.chromium.org/6882109
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82427 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82426 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
ReplaceStringPlaceholders()- support more than 9 arguments ($1, $2, $3, ..., $9 and $10, $11, $12, ...) for ReplaceStringPlaceholders()- add the unittest named ReplaceStringPlaceholdersMoreThan9Replacements for testing more than 9 arguments for ReplaceStringPlaceholders()BUG=16963TEST=string_util_unittest:StringUtilTest.ReplaceStringPlaceholdersMoreThan9ReplacementsPatch by haraken@google.com.Review URL: http://codereview.chromium.org/6877008
TBR=tony@chromium.org
Review URL: http://codereview.chromium.org/6882108
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82423 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82420 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
"key" status. Fixes bug where they keystrokes started going to the browser window if you opened and closed a submenu from the keyboard.
R=mrossetti@chromium.org
BUG=80003
Review URL: http://codereview.chromium.org/6881089
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82419 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
R=mrossetti@chromium.org
BUG=80065
Review URL: http://codereview.chromium.org/6879089
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82418 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=chromium-os:14330
TEST=make sure file browser pops up when USB is inserted
Review URL: http://codereview.chromium.org/6881098
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82415 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=chromium-os:14328
TEST=NONE
Review URL: http://codereview.chromium.org/6880091
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82412 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
so it gets added only once so we dont do extra processing.
BUG=80049
TEST=
Review URL: http://codereview.chromium.org/6880082
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82411 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/6869043
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82409 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- support more than 9 arguments ($1, $2, $3, ..., $9 and $10, $11, $12, ...) for ReplaceStringPlaceholders()
- add the unittest named ReplaceStringPlaceholdersMoreThan9Replacements for testing more than 9 arguments for ReplaceStringPlaceholders()
BUG=16963
TEST=string_util_unittest:StringUtilTest.ReplaceStringPlaceholdersMoreThan9Replacements
Patch by haraken@google.com.
Review URL: http://codereview.chromium.org/6877008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82405 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
resulting in the RootView never getting sized.
http://crbug.com/79432
TEST=see bug
Review URL: http://codereview.chromium.org/6877104
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82403 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Make ExtensionInstallUI::ShowThemeInfoBar() private.
BUG=80062
TEST=Sync a theme to a computer. Make sure the install butterbar doesn't
show.
Review URL: http://codereview.chromium.org/6881093
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82402 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
It's ignored at the other end of the IPC, but I have tagged all
the places it needs to be obeyed with a link to the bug.
While I'm at it, switch UpdateTitle to using string16.
BUG=27094
Review URL: http://codereview.chromium.org/6880073
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82400 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- testAbleToFillFormAfterFormReset
- testDinstinguishMiddleInitialWithinName (M13 feature)
- testMultipleEmailAddressFilledByOneUserGesture
- testProfilesNotAggregatedWhenEmailFoundInNonEmailField (M13 feature)
Changed form id name for each html file for better consistency.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/6882071
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82399 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=28736
TEST=In a maximized window without the download shelf showing, click on a download
link. The status bubble should be moved above the download shelf that appears instead
of overlapping with it.
Review URL: http://codereview.chromium.org/6882040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82398 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
NSExceptions can cause the flag to get latched to YES, which messes
with window closing. A number of suspected exceptions shouldn't cause
too much other damage, so this patches over the problem for the short
term.
TBR=jam@chromium.org
BUG=45928
TEST=see bug
Review URL: http://codereview.chromium.org/6873081
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82397 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Bookmarks"
Call [self reconfigureBookmarkBar] from |nodeRemoved:| instead of from
|removeButton:|.
When it was called from |removeButton:|, the |noItemContainer| was still
hidden, so |adjustNoItemContainerWidthsForMaxX:| did nothing. Calling it
from |nodeRemoved:| fixes this.
Also, it is no longer necessary to call
|configureOffTheSideButtonContentsAndVisibility:| from |nodeRemoved:|,
since that is done by |reconfigureBookmarkBar| already.
BUG=44711
TEST=Add a bunch of items to the bookmarks bar. Resize the window to be
narrow. Open the bookmark manager and shift-select all items in the
bookmark bar. Delete the items through the context menu. The bookmark bar
should now contain the "Import bookmarks now..." text that does not overlap
with "Other bookmarks"
Review URL: http://codereview.chromium.org/6879010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82396 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Counter-intuitively, in gtk_dialog_init(), they use g_signal_connect() to attach
a delete-event handler which calls gtk_dialog_respond with DELETE_EVENT, but
then they return FALSE to fall through to the default handler, which calls destroy. The
widget_hide_on_delete handler that we're using here is inserted after the dialog
delete-event handler, but before the default handler, thus overriding the destroy.
BUG=79764
TEST=manual
Review URL: http://codereview.chromium.org/6880078
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82395 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82394 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
property)
BUG=none
TBR=vandebo@chromium.org
Review URL: http://codereview.chromium.org/6880080
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82393 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
tests.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82392 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
R=ben@chromium.org
TBR=ben@chromium.org
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6878085
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82391 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
memory column in task manager. This memory is
significant enough that we should be looking at it when analyzing memory of the browser process.
BUG=None
TEST=
1. Open up Task Manager,
2. Right-click and "enable" Javascript Memory
3. Observe that the "Browser" row reports the v8 heap
Note, like the v8-based proxy resolver this is DISABLED for single-process chrome because v8 is not thread safe.
Patch by Greg Simon <gregsimon@chromium.org>.
Review URL: http://codereview.chromium.org/6760003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82390 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6873135
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82387 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=green trees
Review URL: http://codereview.chromium.org/6880088
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82386 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add MenuHostRootView::destroyed_flag_ and check.
May help identify the cause of crbug.com/78792.
Minor refactoring / code consolidation.
BUG=78792
TEST=none
Review URL: http://codereview.chromium.org/6879059
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82385 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
R=ben@chromium.org
Review URL: http://codereview.chromium.org/6877064
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82384 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
properties marked "retain" or "copy" backed by synthesized instance variables.
BUG=none
TEST=base_unittests ObjCPropertyReleaserTest.SesameStreet
Review URL: http://codereview.chromium.org/6881091
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82383 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82377 0039d316-1c4b-4281-b951-d872f2087c98
|