| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
icon and the text to show the number of closed windows.
BUG=9022
TEST=With more than one window open close a window. Go to the new tab
page and make sure you see an entry for the closed window. The entry
should have the chrome/chromium logo and instead of 'Window' it
should say the number of tabs in the window.
Review URL: http://codereview.chromium.org/92022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14309 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/93060
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14305 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/95003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14301 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rebaselines:
http://trac.webkit.org/changeset/42722 resulted in
LayoutTests/editing/inserting/insert-3907422-fix.html
LayoutTests/editing/pasteboard/paste-text-015.html
LayoutTests/editing/style/font-family-with-space.html
http://trac.webkit.org/changeset/42723 resulted in
LayoutTests/editing/selection/select-all-iframe.html
LayoutTests/svg/custom/pointer-events-path.svg
http://trac.webkit.org/changeset/42716 resulted in
LayoutTests/fast/dom/HTMLSelectElement/named-options.html
http://trac.webkit.org/changeset/42725
Broke the close event behavior and resulted in disabling these ui tests:
* BrowserCloseBeforeUnloadOK and
* BrowserCloseBeforeUnloadCancel
Review URL: http://codereview.chromium.org/92051
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14291 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
not trigger its deletion.
BUG=10527
TEST=Open the New Tab Page. Click the "Remove thumbnails" link to enter edit mode. Right-click on a thumbnail. The context menu should be displayed and the thumbnail should not be removed.
Review URL: http://codereview.chromium.org/92018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14290 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When that disposition is used and no "normal" window is opened, it crashes.
"normal" means non POPUP, non APPLICATION window type. This happens because
background disposition doens't focus or activate the window unless selected. A
crash happens because GetSelectedTabContents() is null since nothing initially
there.
This fix might have stopped it crash, but would be best to see if its a "good"
approach.
I fixed some lints such as, header file already exists, extra white space, etc)
BUG=10591 ( http://crbug.com/10591 )
Original patch by Mohamed Mansour (see http://codereview.chromium.org/67205/show), r=me,pkasting
Review URL: http://codereview.chromium.org/93058
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14288 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change adds a new test "RenderViewTest.OnHandleKeyboardEvent" to verify we can receive proper DOM events when we send platform-specific keyboard events.
This test is mainly for verifying our WebKit-glue code can convert platform-specific keyboard events to keyboard events of WebKit without any problems, which is hard to implement with LayoutTests.
To emulate non-US layouts, this test loads the following keyboard-layout drivers:
* Arabic (qwerty);
* United States (qwerty);
* French (azerty);
* Hebrew, and;
* Canadian French (qwerty).
Review URL: http://codereview.chromium.org/75034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14287 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
it too.
Review URL: http://codereview.chromium.org/87007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14284 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
change needs to change all open windows with the same preferences,
not just the current one).
Improve unit testing.
Limit bookmark menu size width (1st pass).
Cleanup/delete of code which no longer does much.
Review URL: http://codereview.chromium.org/79068
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14282 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Tests.
Log of hanging tests:
http://build.chromium.org/buildbot/waterfall/builders/XP%20Tests%20(dbg)(2)/builds/1246/steps/unit_tests/logs/stdio
TBR=http://codereview.chromium.org/92055
Review URL: http://codereview.chromium.org/92055
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14281 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Use HTML5 placeholder instead of js code for the text input place holders.
Use HTML5 doctype in all html files.
Skip optional type in script and style tags.
Some small optimizations in LocalStrings.
Review URL: http://codereview.chromium.org/88072
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14278 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
first-run dialog (ta,kn,te,ml)
BUG=10687
Review URL: http://codereview.chromium.org/88017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14277 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
breaking horribly. When it _is_ small, draw the frame correctly.This is an incomplete fix; only non-Aero main/popup/app windows are handled so far, but this was becoming hairy enough I wanted to checkpoint it.BUG=9885
Review URL: http://codereview.chromium.org/88069
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14276 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
[Retry of 14260 with fixed unit test.]
1) Moves renderer_initialized_ from RenderWidgetView into RenderWidgetHost (its parent). Should be identical semantics to before.
2) Test renderer_initialized_ in RWH::WasResized().
This also reverts r13725, which was another attempt at fixing this problem.
BUG=9830,10659
TEST=From the Linux start page, click a link then click back -- should not get a gray page. From Google reader, click a link (which spawns a new tab within the same process) -- should not get a gray page.
Review URL: http://codereview.chromium.org/93038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14273 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
vista. I believe the bug happens because we're in a nested loop with a
Dispatcher, so that even though we're posting two quits, they both end
up quitting the same loop. Posting a WM_USER event between these two
tickles exitting out of the nested loop, then quitting the outer loop.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/93049
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14272 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
until it's reactivated.
BUG=8945
Review URL: http://codereview.chromium.org/93048
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14270 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
This reverts commit r14260.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14267 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- add concept of default insets to view which get added to any other insets provided by the user. used by label to provide room for a focus border.
- provide the ability for the label to paint its focus border even if it isn't focused. needed because the outer container (the checkbox) gets focus but the inner label does not, however the label knows best the location of its text around which the focus border must be drawn.
please note:
- also make it easier to click checkboxes by not resetting mouse_pressed_handler_ in RootView when a view decides it doesn't want to handle a drag. this is so we can still receive mousereleased notifications when the mouse is released... it's "difficult" to click checkboxes and radio buttons when you accidentally drag a little on their label. (this is the root view change).
- fix slight alignment issue on the general page of options.
Also fix a slight error in my last radio checkbox - clicking on a checked radio button should still focus it.
http://crbug.com/10834
TEST=visit options, Minor Tweaks. click the "always ask before downloading" checkbox and observe that the focus rect tightly surrounds the text label instead of stretching to the right side of the dialog.
Visit options, click an already-checked radio button. observe that it takes focus.
Visit options, click on any checkbox or radio button, drag slightly then release (still within the bounds of the item). note the item is now toggled or selected. click down then drag out and release, note that it is not toggled or selected.
Review URL: http://codereview.chromium.org/92004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14265 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
Without this, GetClientRect would return a size of 0,0 for the view, which would prevent the tooltip machinery from working properly.
Review URL: http://codereview.chromium.org/87058
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14264 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1) Moves renderer_initialized_ from RenderWidgetView into RenderWidgetHost (its parent). Should be identical semantics to before.
2) Test renderer_initialized_ in RWH::WasResized().
This also reverts r13725, which was another attempt at fixing this problem.
BUG=9830,10659
TEST=From the Linux start page, click a link then click back -- should not get a
gray page. From Google reader, click a link (which spawns a new tab within the
same process) -- should not get a gray page.
Review URL: http://codereview.chromium.org/93038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14260 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
Needed to remove the anonymous namespace because Mac's gcc complains
about friend trying to poke through anonymous namespaces.
Review URL: http://codereview.chromium.org/93043
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14259 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added a new field into ContextMenuParam only for this purpose. The new field
won't be validated in frontend processes. In this way, even if renderer
processes are going to mad, the frontend would be OK if the frontend uses this
new field only for copying into the clipboard.
Fix checked in for Shinichiro Hamaji
Original code review: http://codereview.chromium.org/91002
BUG=2725
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14256 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/88053
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14255 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
With my ownership changes, the TabContents owns the RVHManager which owns the
DOMUI. This means that the DOM UI is deleted from ~TabContents and it will
crash during destruction if we access the TabContents to delete the property.
Review URL: http://codereview.chromium.org/92042
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14253 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Removed anonymous namespace, Mac's gcc won't allow friend to reach
into anonymous namespaces.
Noticed that UserScriptMasterTest.NoScripts hanged it DirectoryWatcher
was not implemented. Added a timeout event so that now it fails
slowly.
Then I disabled that test for non-Windows. Other tests pass, lets get
those in the system!
Review URL: http://codereview.chromium.org/88075
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14252 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Enable extension_messages_unittest.cc for Mac and Linux. Had been
added for Windows, but not to chrome.gyp.
Drop extension_content_script_inject_unittest.cc because it no longer
exists.
Review URL: http://codereview.chromium.org/87070
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14251 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
* Link agasint Breakpad from public svn repo.
* Bump DEPS to new svn rev of Breakpad with fixes for 10.5 compilation.
* Fill in code to differentiate between processes types in crash report.
Review URL: http://codereview.chromium.org/88043
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14245 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/87072
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14243 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
from being the frontmost app during the course of a ui_tests run. This way
errors will show up in more carefully controlled local tests, but won't turn
the bots red.
BUG=5278
TEST=make sure TabRestoreUiTest.* don't fail due to is_active being false
Review URL: http://codereview.chromium.org/92038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14242 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=pinkerton
Review URL: http://codereview.chromium.org/93037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14241 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
download code. Get other tests compiling even if disabled. Mark why tests are failing in gyp file.
Review URL: http://codereview.chromium.org/92033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14239 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
That DCHECK might be a little overzealous, but is probably useful for most situations, so I'm leaving it.
TBR=tony
Review URL: http://codereview.chromium.org/92036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14237 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
came from so they're restored together (into the same new window) when
appropriate.
Fix safety check on tab index when restoring: make it check the correct
browser.
Change some ASSERTs to EXPECTs in the unit test for greater coverage.
BUG=5278
TEST=Open a window with two tabs, close both (closing the window), then
restore both. Make sure both restored tabs are in the same window. Open
a window with multiple tabs, close a tab, then close the window using
its close box. Restore both and make sure the tab goes back into the window.
Review URL: http://codereview.chromium.org/92001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14234 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
merging the two. This is extremely straightforward except for the
GearsCreateShortcutCallbackFunctor which I had to move to the header (with a
TODO and its associated function OnGearsCreateShortcutDone) and some of the
constructors for the members which took a WebContents that I had to cast.
I tried to reorganize the data to have some kind of groupings.
I made the member initializers all explicit for TabContents because the
extreme number of them makes them difficult to keep track of.
Review URL: http://codereview.chromium.org/88021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14230 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/87031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14227 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14225 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
TBR=tony
Review URL: http://codereview.chromium.org/88045
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14222 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
time because they didn't pass.
Review URL: http://codereview.chromium.org/88056
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14221 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
Also, fix duration and tween type for download shelf open animation.
Review URL: http://codereview.chromium.org/88064
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14220 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/93028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14214 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
Add a resource for tabs with no title on mac. Fix window title reporting. Remove the Browser dependency from the tab strip (only needs a TabStripModel).
Review URL: http://codereview.chromium.org/93025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14212 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
flusing the message queue (containing the delete message).
BUG=10782
Review URL: http://codereview.chromium.org/92023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14208 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(r14075 take two)
Currently we perform several X operations on the IO thread including
geometry and clipboard work. This is causing races inside Xlib and
crashing the browser.
These are the result of synchronous calls from the renderer, so we
cannot route these requests to the UI thread without risking deadlock.
Thus we introduce the UI2 thread. This thread has a second connection
to the X server and can perform X operations safely the without UI
thread.
Work remains to be done:
Since we still have the hack where we pass GtkWidget pointers into the
renderer and back, we still have to access these structures from the
IO and UI2 threads. This still needs to be fixed, but this is not the
patch for it.
Also, not all the X calls from the IO thread have been moved over in
this patch; just a few small ones.
http://codereview.chromium.org/67145
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14206 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
Issue=8934
Review URL: http://codereview.chromium.org/89006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14204 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
doesn't show existing files).
BUG=10734
Review URL: http://codereview.chromium.org/93019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14202 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=10843
TBR=mbelshe
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14200 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
This reverts commit 8269e3cc8929f8348d0ed1ec13cb69f90cc928f6.
Review URL: http://codereview.chromium.org/93020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14190 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/92016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14189 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14187 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/90007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14183 0039d316-1c4b-4281-b951-d872f2087c98
|