| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Implement a CreateApplicationShortcutView similar to the current Gears-based one;
- Add a few profile prefs to persist user's last checked shortcut locations
- Implement a web_app::CreateShortcut that stores icon under "<profile>/Web Applications"
in a similar layout as gears (i.e. <host>/<scheme_port>/<web_app_title>.ico) and calls
file_util code to creates shortcut on Windows;
- Add Win7 taskbar pin/unpin support function to file_util;
- Update TabContents to replace gears with new code;
Note:
- Gears dialog is modaless but this one is a modal dialog.
- Gear's icon store is not migrated because gears icons
could still be used by shortcuts created by gears and
thus we could not delete them even after migration. And we
are not using the local ico files in the dialog even if
they exists.
- New CreateApplicationShortcutView is included when
TOOLKIT_VIEW is defined. However on platforms other
than Windows, the actual CreateShortCut code is not
implemented. Right now it calls ShellIntegration's
CreateDesktopShort if OS_LINUX is defined and
NOTREACHED() for other platforms.
BUG=25528
TEST=Verify the new UI provides the same functionality as gears under XP/Vista and support pinning on Win 7.
Review URL: http://codereview.chromium.org/372012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32165 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
The patch gives non-webapp chrome an app id so that win7 taskbar group the icons correctly. That is, webapp icons are grouped together based on app name and all chrome windows are grouped together.
BUG=19358
TEST=verify fix for issue 19358
Review URL: http://codereview.chromium.org/385120
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32164 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=27685,27690
Review URL: http://codereview.chromium.org/398001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32163 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=27920
TEST=ToolbarControllerTest.LoadingState
TBR=dmaclach,erikkay
Review URL: http://codereview.chromium.org/403003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32162 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/387066
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32160 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32159 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
Will revert after looking at the Linux memory bot and perf cycler.
TBR=willchan
Review URL: http://codereview.chromium.org/403002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32158 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/397009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32156 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
this code the bookmark bar's widget gets sized to 1x1 for a
layout/painting step and we get all sorts of painting artifacts. I
don't understand why that happens, but forcing a layout fixes it. I've
scoped this just to chrome os for now.
BUG=27898
TEST=make sure tab switching on chrome os doesn't result in any visual
artifacts, especially when switching between ntp and non-ntp.
Review URL: http://codereview.chromium.org/397018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32152 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
When chrome is ran in kiosk mode, the fullscreen bubble should be hidden. As well, I found a major variable name mixup in browerview.
BUG=27308
TEST=None
Review URL: http://codereview.chromium.org/385146
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32151 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
BUG=27927
Review URL: http://codereview.chromium.org/400007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32149 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=27145,possibly others
TEST=See bug.
Review URL: http://codereview.chromium.org/399014
TBR=thakis@chromium.org
Review URL: http://codereview.chromium.org/399021
TBR=erikkay@chromium.org
Review URL: http://codereview.chromium.org/399024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32143 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
completely necessary, and as it turns out we don't always get matching
exit/enter events.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/395025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32142 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=27920
TEST=ToolbarControllerTest.Display
TBR=dmaclach
Review URL: http://codereview.chromium.org/399022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32139 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=27145,possibly others
TEST=See bug.
Review URL: http://codereview.chromium.org/399014
TBR=thakis@chromium.org
Review URL: http://codereview.chromium.org/399021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32137 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
the fly.
BUG=23400
TEST=none
Review URL: http://codereview.chromium.org/399006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32136 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
To do this, we have to initiate downloads on the UI thread and don't hold onto a reference in the file thread.
BUG=27667
Review URL: http://codereview.chromium.org/387055
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32129 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=21054
TEST=None
Review URL: http://codereview.chromium.org/391081
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32128 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
tab_contents and the
renderer_host sources.
Remove the AutomationProfileImpl class which wraps the Chrome profile for an external tab
container, which hosts ChromeFrame. This object was used to carry a custom URL request
context which was used to intercept HTTP requests and cookie requests issued by external
tabs.
However as the life time of the automation profile class depended on the lifetime of the external
tab container object this caused a number of crashes in objects which held on to the automation profile
pointer retrieved from the associated tab contents. This does not happen in a regualar Chrome browser
instance as the profile is deleted at the very end.
We can associate the automation URL request context with the underlying tab_contents which would
eventually percolate down to the resource message filter. Doing this would avoid the need for the
AutomationProfile class.
This CL achieves that.
Bug=27695,27662
Review URL: http://codereview.chromium.org/385117
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32127 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
chrome url request data sources. This will allow me to merge
IncognitoTabHTMLSource and NewTabHTMLSource and allow me clean
up DOMUIThemeSource, but those will be follow up changes.
Review URL: http://codereview.chromium.org/397019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32126 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=27145,possibly others
TEST=See bug.
Review URL: http://codereview.chromium.org/399014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32121 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
display of pop-ups for extensions viewed through a tab-contents view.I added a new class, ExtensionPopupHost. This class implements the necessary environment for managing child popup windows from either an ExtensionHost, or an ExtensionDOMUI. Note that this class is added as a member to ExtensionHost and ExtensionDOMUI.
I decided to take this approach to prevent multiple inheritance of the NotificationObserver class: Both ExtensionPopupHost and ExtensionHost must inherit from this class, and I was uncertain of how the system would behave wrt virtual inheritance. Please comment on if I should have used the inheritance approach.
I also removed the customHandler tag (in extension_api.json) that I had added in the initial submission. The arguments in the schema are now those that users of the API see. The nodocs tags were also removed.
The api experimental.popup.getAnchorWindow() has been renamed to popup.getParentWindow, as per a suggestion from Erik K.
BUG=none
TEST=extension_popup_apitest.cc
Review URL: http://codereview.chromium.org/385061
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32120 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
NtpBackgroundUtil::PaintBackgroundDetachedMode() expects to draw
the entire bookmark bar on every expose; all of its internal math
is based on the assumption that the incoming area is zero-based and
is the whole bookmark bar area.
TEST=(Repo instructions in bug)
BUG=25655
Review URL: http://codereview.chromium.org/401005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32115 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
is fixed.
BUG=16591
TEST = none
Review URL: http://codereview.chromium.org/384073
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32113 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
------
Revert 31875 to see whether it fixes reliability bot.
BUG=25677
TEST=None
------
TBR=huanr
Review URL: http://codereview.chromium.org/397017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32112 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
TEST=BrowserTest.SingleBeforeUnloadAfterWindowClose
TBR=idana
disable test on Linux due to crashes
Review URL: http://codereview.chromium.org/391080
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32108 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=27763
Review URL: http://codereview.chromium.org/400004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32106 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
http://codereview.chromium.org/401002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32105 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
races.
Review URL: http://codereview.chromium.org/391062
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32104 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=27833
TEST=See bug
Review URL: http://codereview.chromium.org/387064
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32103 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/399012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32096 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/389015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32095 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=24672
TEST=unit_tests
Patch by Thiago Farina <thiago.farina@gmail.com>.
Review URL: http://codereview.chromium.org/375016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32093 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Ensure correct shutdown of threads.
TEST=Covered by unit_tests.
BUG=27726
Review URL: http://codereview.chromium.org/384149
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32090 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
This fixes the error where PostTask would post the task to the file thread and release its reference before the constructor returned.
credit for fix goes to jam@ and thestig@
Review URL: http://codereview.chromium.org/399011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32089 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=27224
Review URL: http://codereview.chromium.org/385085
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32085 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
page-completedness
rather than the onload event. Extensions were moved from the pre-onload state
to running at document idle some time ago.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/397013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32076 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=25677
TEST=None
Review URL: http://codereview.chromium.org/397014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32074 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This should be sentence-case rather than title-case for non-Linux,
and it shouldn't have a trailing colon on Linux.
BUG=27766
Review URL: http://codereview.chromium.org/395019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32073 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=compiles
Review URL: http://codereview.chromium.org/384138
Patch from Thiago Farina <thiago.farina@gmail.com>.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32067 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/391074
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32066 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/399002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32056 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/385134
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32053 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32052 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=none
TBR=aa
Review URL: http://codereview.chromium.org/395016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32048 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=none
TBR=aa
Review URL: http://codereview.chromium.org/391071
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32047 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=none
TBR=aa
Review URL: http://codereview.chromium.org/384148
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32046 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also a bit of copyediting and cleanup.
Future work:
* improve these styles
* make these styles work well in IE
(the old styles didn't work well in IE, either)
* separate the codesite styles from ours (for easier update)
BUG=27434
TEST=none
TBR=aa
Review URL: http://codereview.chromium.org/398005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32043 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These aren't necessary now that the debugging tutorial
has been updated and the relevant examples are in the
new examples directory.
BUG=none
TEST=none
TBR=aa
Review URL: http://codereview.chromium.org/387059
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32039 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/384139
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32037 0039d316-1c4b-4281-b951-d872f2087c98
|