| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
| |
BUG=8042
TEST=None
Review URL: http://codereview.chromium.org/567033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38016 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=27526
TEST=Unit tests BrowserActionApiTest.BrowserAction*Popup .
Review URL: http://codereview.chromium.org/552263
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38015 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
bots due to newer python version
BUG=27879
TEST=none
Review URL: http://codereview.chromium.org/570004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38013 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=27132
TEST=none
Review URL: http://codereview.chromium.org/567019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38012 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
not supported by the translation server.
BUG=34365
TEST=Visit http://www.breizh.net/brezhoneg/, no translation
infobar should be shown.
Review URL: http://codereview.chromium.org/568028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38010 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
by the extension and not for images that were autogenerated.
Since this adds data to the theme pack file, bump the version number.
BUG=34078
TEST=BrowserThemePackTest.TestHasCustomImage
Review URL: http://codereview.chromium.org/562025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38006 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/553153
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38004 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=34416
TEST=try it
Review URL: http://codereview.chromium.org/569031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38003 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Modify CookiePolicy to work asynchronously
This change will enable us to prompt the user before setting a cookie. While we only need to prompt before setting, we actually need to make both CanSetCookie and CanGetCookies asynchronous. This is necessary in order to preserve FIFO ordering since the value returned by GetCookies depends on the changes made to the cookie DB by SetCookie.
This change also includes some simplification of CookieStore. Instead of N virtual functions, I distilled it down to only 4. The remaining functions are instead expressed in terms of those.
While studying all the places where we currently use CookiePolicy, I found that some of them were not appropriate. After discussing with Amit, I decided to remove the policy checks in URLRequestAutomationJob. See the comments in the code regarding this.
I changed the signature of CookieMonster::GetRawCookies to GetAllCookiesForURL to better match GetAllCookies. I also filed a bug about making it even closer in functionality. Related to this change webkit/glue/webcookie.h grows a constructor that takes a CanonicalCookie to help clean up some code.
On the Chrome side, ChromeURLRequestContext now has a ChromeCookiePolicy object. That object is threadsafe ref counted because it is passed between the UI and IO threads. It is responsible for implementing the queuing logic described above. It will also in the future trigger the Chrome UI code to actually show the setcookie prompt.
Please review the state machinery changes in URLRequestHttpJob carefully.
R=eroman
BUG=34331
TEST=no tests yet for prompting.
Review URL: http://codereview.chromium.org/567015
TBR=darin@chromium.org
Review URL: http://codereview.chromium.org/562037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38002 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix ChromiumOS build.
TBR=eroman
Review URL: http://codereview.chromium.org/569030
TBR=darin@chromium.org
Review URL: http://codereview.chromium.org/565036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38001 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=eroman
Review URL: http://codereview.chromium.org/569030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38000 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Separated objects construction and initialization.
BUG=34374
TEST=Chrome based login now should have status area buttons in the right-top corner.
Review URL: http://codereview.chromium.org/564027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37999 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change will enable us to prompt the user before setting a cookie. While we only need to prompt before setting, we actually need to make both CanSetCookie and CanGetCookies asynchronous. This is necessary in order to preserve FIFO ordering since the value returned by GetCookies depends on the changes made to the cookie DB by SetCookie.
This change also includes some simplification of CookieStore. Instead of N virtual functions, I distilled it down to only 4. The remaining functions are instead expressed in terms of those.
While studying all the places where we currently use CookiePolicy, I found that some of them were not appropriate. After discussing with Amit, I decided to remove the policy checks in URLRequestAutomationJob. See the comments in the code regarding this.
I changed the signature of CookieMonster::GetRawCookies to GetAllCookiesForURL to better match GetAllCookies. I also filed a bug about making it even closer in functionality. Related to this change webkit/glue/webcookie.h grows a constructor that takes a CanonicalCookie to help clean up some code.
On the Chrome side, ChromeURLRequestContext now has a ChromeCookiePolicy object. That object is thread-safe ref counted because it is passed between the UI and IO threads. It is responsible for implementing the queuing logic described above. It will also in the future trigger the Chrome UI code to actually show the set-cookie prompt.
Please review the state machinery changes in URLRequestHttpJob carefully.
R=eroman
BUG=34331
TEST=no tests yet for prompting.
Review URL: http://codereview.chromium.org/567015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37998 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
regression bugs.
R=arv
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/563020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37997 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
states. Basically adding error catching.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/562005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37996 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TEST=None
BUG=None
Review URL: http://codereview.chromium.org/565020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37995 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix breakage caused by <http://src.chromium.org/viewvc/chrome?view=rev&revision=37819>.
Changed tags for File->{Save Page As..., Email Page Location, Print...} and View->Text Encoding.
BUG=34463
TEST=Check that File->Save Page As... (and Cmd-S), File->Email Page Location, and View->Text Encoding work properly.
Review URL: http://codereview.chromium.org/566033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37994 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
to OS X support, so this CL simply tweaks the #ifdefs.
BUG=32602
TEST=none
Review URL: http://codereview.chromium.org/569018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37993 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Original change:
Add tests for the geolocation network provider.
Also some small tidy up a few other files.
BUG=http://crbug.com/11246
TEST=unit_tests.exe gtest_filter=NetworkLocationProvider* gtest_break_on_failure
Review URL: http://codereview.chromium.org/556106
TBR=joth@chromium.org
Review URL: http://codereview.chromium.org/570006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37992 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
gcf: protocol that's now disabled.
TBR=ananta
Review URL: http://codereview.chromium.org/560035
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37990 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Also some small tidy up a few other files.
BUG=http://crbug.com/11246
TEST=unit_tests.exe --gtest_filter=NetworkLocationProvider* --gtest_break_on_failure
Review URL: http://codereview.chromium.org/556106
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37989 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
testing::AnyNumber() to be followed by WillRepeatedly(...)
instead of WillOnce
TBR=ananta
Review URL: http://codereview.chromium.org/563019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37988 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
GC upon raw cookies request.
TBR=yurys
Review URL: http://codereview.chromium.org/564039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37985 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
I added code in r36144 so that we wouldn't create extra ConnectJobs if we were still releasing sockets. This code did not handle the case where we had multiple disconnected releasing sockets. I've added a test and code to cover this case now.
BUG=34123
Review URL: http://codereview.chromium.org/557089
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37984 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=34391
Review URL: http://codereview.chromium.org/560026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37982 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Well, almost. cf: is now changed to gcf: protocol. gcf: protocol
now is supported only for the following cases:
1. gcf:attach_external_tabXXX urls. These are URLs used internally
by Chrome Frame to implement window.open
2. gcf:about:xxx
3. gcf:view-source:xxx
4 For any other URLs ONLY if it is enabled by setting a registry
value 'EnableGCFProtocol' to 1 in HKCU\Software\Google\ChromeFrame
BUG=22721,23006,23175,29350
TEST=changed existing cf: tests to new gcf: tests, added a new test for cf:view-source
Review URL: http://codereview.chromium.org/562008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37981 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=None
Review URL: http://codereview.chromium.org/560023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37980 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
- use index of selected language menu item (which corresponds to index in array of languages) to check if selection has changed; was retrieving, converting and comparing text, which is more expensive and slower
- only call backend to translate when source and target languages are different
BUG=31297
TEST=verify that selection of languages in translate infobars still work as expected.
Review URL: http://codereview.chromium.org/563018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37979 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/565035
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37978 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
just compiling.
BUG=26099
Review URL: http://codereview.chromium.org/552240
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37977 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
from Jungshik review:
http://codereview.chromium.org/552216/show
TEST=None
BUG=None
Review URL: http://codereview.chromium.org/566024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37976 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Pepper APIs in Chrome. Specifically, we need to be able to get the base::SharedMemory
or TransportDIB memory regions used to communicate with devices. To enable doing this,
especially in 64-bit Chrome, I needed to change the GetStateContext and SetStateContext
functions to manipulate 64-bit values. This CL does not include the Native Client
hookup to the new APIs, but is needed to do so.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/569004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37975 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=22360
TEST=none
Review URL: http://codereview.chromium.org/564038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37974 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
checkmark icon next to Pause to alternating the words "Pause" and "Resume".
XIB change: unset the 'On Image' on the Pause button.
BUG=23069
TEST=Download large file and verify that the menu item says "Pause." Click "Pause" and the menu item should now say "Resume".
Review URL: http://codereview.chromium.org/563001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37973 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
http://codereview.chromium.org/569017
BUG=0
TEST=Run the test for both Debug and Release builds
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37972 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This version contains a number of bugfixes including wrapping
of values when calling userdefined functions and a stricter
JSON parser.
BUG=32191,34221
Review URL: http://codereview.chromium.org/564037
TBR=ager@chromium.org
Review URL: http://codereview.chromium.org/562035
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37971 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Without this change, items in submenus of a context menu where always enabled. With this change, the value we use for [NSMenuItem setEnabled:] is respected.
This change is required for an upcoming change that needs this to work.
BUG=none
TEST=Right-click on a text input field, Spell Checker Options->Show spelling panel shouldn't be disabled.
Review URL: http://codereview.chromium.org/567031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37970 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=senorblanco
TEST=trybots
BUG=none
Review URL: http://codereview.chromium.org/561046
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37969 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This version contains a number of bugfixes including wrapping
of values when calling user-defined functions and a stricter
JSON parser.
BUG=32191,34221
Review URL: http://codereview.chromium.org/564037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37968 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37965 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=mpcomplete
TEST=trybots
BUG=none
Review URL: http://codereview.chromium.org/560029
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37964 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=senorblanco
TEST=trybots
BUG=none
Review URL: http://codereview.chromium.org/569025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37963 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=senorblanco
TEST=trybots
BUG=none
Review URL: http://codereview.chromium.org/561044
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37962 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
month and year]
BUG=33026
TEST=should appear like in mocks, when dialog is finished
Review URL: http://codereview.chromium.org/566027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37961 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=31104
TEST=Visit http://www.google.com/search?ie=UTF-8&q=上地 and select entire text in Omnibox, then copy the text.
Paste the copied text into other applications such as notepad and check if the pasted text contains %E4%B8%8A%E5%9C%B0, not 上地.
Paste the copied text into Safari's URL bar and type return. Check if you are navigated to the same website.
Paste the copied text into textareas in Chrome and check if the pasted text contains %E4%B8%8A%E5%9C%B0.
Paste the copied text into Gmail's rich text editing mode and check if the pasted text contains 上地.
Right click the link you pasted in Gmail and check if you see "Go to: http://www.google.com/...%E4%B8%8A%E5%9C%B0".
Paste the copied text into TextEdit and check if the pasted text isn't garbled and correctly contains 上地.
Check if the link in TextEdit is pointing to encoded URL (http://www.google.com/...%E4%B8%8A%E5%9C%B0).
TEST=Visit http://www.google.com/search?ie=UTF-8&q=上地 again and select "q=上地" in Omnibox, then copy the text.
Paste it into somewhere and check if you see "q=上地".
TEST=Copy "上地" from somewhere and paste it into Omnibox.
Copy from Omnibox and paste it into notepad. Then check if the pasted text is NOT encoded.
Review URL: http://codereview.chromium.org/549172
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37955 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TBR=darin
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/561043
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37953 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
chromeos::BrowserView*
Now has all build fixes integrated.
BUG=none
TEST=none
TBR=oshima
Review URL: http://codereview.chromium.org/568030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37952 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=
TEST=Go to a page with blocked stuff. Icons should appear in omnibox.
Review URL: http://codereview.chromium.org/565010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37951 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
caret if there is no selected text.
See http://webkit.org/b/33364
BUG=8841
TEST=Open www.google.com on Windows or Linux (not Mac OS)
type "qeury"
Right-click on the text field
Select "query"
Confirm that qeury is replaced with "query"
Review URL: http://codereview.chromium.org/562014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37939 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=32719
TEST=none
Review URL: http://codereview.chromium.org/560024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37936 0039d316-1c4b-4281-b951-d872f2087c98
|